home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 6.2.267 < prev    next >
Encoding:
Internet Message Format  |  2004-02-16  |  10.8 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.2.267 (extra)
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. Mime-Version: 1.0
  6. Content-Type: text/plain; charset=ISO-8859-1
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 6.2.267 (extra)
  11. Problem:    Win32: "&&" in a tearoff menu is not shown. (Luc Hermitte)
  12. Solution:   Use the "name" item from the menu instead of the "dname" item.
  13. Files:        src/gui_w32.c, src/menu.c
  14.  
  15.  
  16. *** ../vim-6.2.266/src/gui_w32.c    Sun Jan 25 20:24:03 2004
  17. --- src/gui_w32.c    Mon Feb 16 12:19:06 2004
  18. ***************
  19. *** 2965,2971 ****
  20.       HFONT    font, oldFont;
  21.       int        col, spaceWidth, len;
  22.       int        columnWidths[2];
  23. !     char_u    *label, *text, *end, *acEnd = NULL;
  24.       int        padding0, padding1, padding2 = 0;
  25.       int        sepPadding=0;
  26.   #ifdef USE_SYSMENU_FONT
  27. --- 2965,2973 ----
  28.       HFONT    font, oldFont;
  29.       int        col, spaceWidth, len;
  30.       int        columnWidths[2];
  31. !     char_u    *label, *text;
  32. !     int        acLen;
  33. !     int        nameLen;
  34.       int        padding0, padding1, padding2 = 0;
  35.       int        sepPadding=0;
  36.   #ifdef USE_SYSMENU_FONT
  37. ***************
  38. *** 2974,2984 ****
  39.   #endif
  40.   
  41.       /*
  42. !      * If this menu is already torn off, then don't
  43. !      * tear it off again, but move the existing tearoff
  44. !      * to the mouse position.
  45.        */
  46. - 
  47.       if (IsWindow(menu->tearoff_handle))
  48.       {
  49.       POINT mp;
  50. --- 2976,2983 ----
  51.   #endif
  52.   
  53.       /*
  54. !      * If this menu is already torn off, move it to the mouse position.
  55.        */
  56.       if (IsWindow(menu->tearoff_handle))
  57.       {
  58.       POINT mp;
  59. ***************
  60. *** 2991,3003 ****
  61.       }
  62.   
  63.       /*
  64. !      * Otherwise, create a new tearoff
  65.        */
  66. - 
  67.       if (*title == MNU_HIDDEN_CHAR)
  68.       title++;
  69.   
  70. !     /* Allocate some memory to play with.  It's made bigger when needed. */
  71.       template_len = DLG_ALLOC_SIZE;
  72.       pdlgtemplate = p = (WORD *)LocalAlloc(LPTR, template_len);
  73.       if (p == NULL)
  74. --- 2990,3002 ----
  75.       }
  76.   
  77.       /*
  78. !      * Create a new tearoff.
  79.        */
  80.       if (*title == MNU_HIDDEN_CHAR)
  81.       title++;
  82.   
  83. !     /* Allocate memory to store the dialog template.  It's made bigger when
  84. !      * needed. */
  85.       template_len = DLG_ALLOC_SIZE;
  86.       pdlgtemplate = p = (WORD *)LocalAlloc(LPTR, template_len);
  87.       if (p == NULL)
  88. ***************
  89. *** 3019,3051 ****
  90.       oldFont = SelectFont(hdc, font);
  91.       else
  92.       oldFont = SelectFont(hdc, GetStockObject(SYSTEM_FONT));
  93. !     /*
  94. !      * Calculate width of a single space.  Used for padding columns to the
  95. !      * right width.
  96. !      */
  97.       spaceWidth = GetTextWidth(hdc, " ", 1);
  98.   
  99.       submenuWidth = 0;
  100. -     /* Figure out widths for each column. */
  101.       for (col = 0; col < 2; col++)
  102.       {
  103.       columnWidths[col] = 0;
  104.       for (pmenu = menu->children; pmenu != NULL; pmenu = pmenu->next)
  105.       {
  106.           text = (col == 0) ? pmenu->dname : pmenu->actext;
  107. -         if (pmenu->children != NULL)
  108. -         submenuWidth = TEAROFF_COLUMN_PADDING * spaceWidth;
  109.           if (text != NULL && *text != NUL)
  110.           {
  111. !         end = text + strlen(text);
  112. !         textWidth = GetTextWidth(hdc, text, (int)(end - text));
  113.           if (textWidth > columnWidths[col])
  114.               columnWidths[col] = textWidth;
  115.           }
  116.       }
  117.       }
  118.       if (columnWidths[1] == 0)
  119.       {
  120.       if (submenuWidth != 0)
  121.           columnWidths[0] += submenuWidth;
  122.       else
  123. --- 3018,3051 ----
  124.       oldFont = SelectFont(hdc, font);
  125.       else
  126.       oldFont = SelectFont(hdc, GetStockObject(SYSTEM_FONT));
  127. ! 
  128. !     /* Calculate width of a single space.  Used for padding columns to the
  129. !      * right width. */
  130.       spaceWidth = GetTextWidth(hdc, " ", 1);
  131.   
  132. +     /* Figure out max width of the text column, the accelerator column and the
  133. +      * optional submenu column. */
  134.       submenuWidth = 0;
  135.       for (col = 0; col < 2; col++)
  136.       {
  137.       columnWidths[col] = 0;
  138.       for (pmenu = menu->children; pmenu != NULL; pmenu = pmenu->next)
  139.       {
  140. +         /* Use "dname" here to compute the width of the visible text. */
  141.           text = (col == 0) ? pmenu->dname : pmenu->actext;
  142.           if (text != NULL && *text != NUL)
  143.           {
  144. !         textWidth = GetTextWidth(hdc, text, (int)STRLEN(text));
  145.           if (textWidth > columnWidths[col])
  146.               columnWidths[col] = textWidth;
  147.           }
  148. +         if (pmenu->children != NULL)
  149. +         submenuWidth = TEAROFF_COLUMN_PADDING * spaceWidth;
  150.       }
  151.       }
  152.       if (columnWidths[1] == 0)
  153.       {
  154. +     /* no accelerators */
  155.       if (submenuWidth != 0)
  156.           columnWidths[0] += submenuWidth;
  157.       else
  158. ***************
  159. *** 3053,3071 ****
  160.       }
  161.       else
  162.       {
  163.       columnWidths[0] += TEAROFF_COLUMN_PADDING * spaceWidth;
  164.       columnWidths[1] += submenuWidth;
  165.       }
  166.       /*
  167. !      * Now find the width of our 'menu'.
  168.        */
  169. !     textWidth = 0;
  170. !     for (col = 0; col < 2; col++)
  171. !     textWidth += columnWidths[col];
  172.       if (submenuWidth != 0)
  173.       {
  174.       submenuWidth = GetTextWidth(hdc, TEAROFF_SUBMENU_LABEL,
  175. !                   (int)STRLEN(TEAROFF_SUBMENU_LABEL));
  176.       textWidth += submenuWidth;
  177.       }
  178.       dlgwidth = GetTextWidth(hdc, title, (int)STRLEN(title));
  179. --- 3053,3071 ----
  180.       }
  181.       else
  182.       {
  183. +     /* there is an accelerator column */
  184.       columnWidths[0] += TEAROFF_COLUMN_PADDING * spaceWidth;
  185.       columnWidths[1] += submenuWidth;
  186.       }
  187. + 
  188.       /*
  189. !      * Now find the total width of our 'menu'.
  190.        */
  191. !     textWidth = columnWidths[0] + columnWidths[1];
  192.       if (submenuWidth != 0)
  193.       {
  194.       submenuWidth = GetTextWidth(hdc, TEAROFF_SUBMENU_LABEL,
  195. !                       (int)STRLEN(TEAROFF_SUBMENU_LABEL));
  196.       textWidth += submenuWidth;
  197.       }
  198.       dlgwidth = GetTextWidth(hdc, title, (int)STRLEN(title));
  199. ***************
  200. *** 3105,3113 ****
  201.       *p++ = 0;        // Class
  202.   
  203.       /* copy the title of the dialog */
  204. !     nchar = nCopyAnsiToWideChar(p, ((*title) ?
  205. !                     (LPSTR)title :
  206. !                     (LPSTR)("Vim "VIM_VERSION_MEDIUM)));
  207.       p += nchar;
  208.   
  209.       if (s_usenewlook)
  210. --- 3105,3113 ----
  211.       *p++ = 0;        // Class
  212.   
  213.       /* copy the title of the dialog */
  214. !     nchar = nCopyAnsiToWideChar(p, ((*title)
  215. !                     ? (LPSTR)title
  216. !                     : (LPSTR)("Vim "VIM_VERSION_MEDIUM)));
  217.       p += nchar;
  218.   
  219.       if (s_usenewlook)
  220. ***************
  221. *** 3130,3141 ****
  222.       p += nchar;
  223.       }
  224.   
  225. !     /* Don't include tearbar in tearoff menu */
  226.       if (STRCMP(menu->children->name, TEAR_STRING) == 0)
  227.       menu = menu->children->next;
  228.       else
  229.       menu = menu->children;
  230. - 
  231.       for ( ; menu != NULL; menu = menu->next)
  232.       {
  233.       if (menu->modes == 0)    /* this menu has just been deleted */
  234. --- 3130,3143 ----
  235.       p += nchar;
  236.       }
  237.   
  238. !     /*
  239. !      * Loop over all the items in the menu.
  240. !      * But skip over the tearbar.
  241. !      */
  242.       if (STRCMP(menu->children->name, TEAR_STRING) == 0)
  243.       menu = menu->children->next;
  244.       else
  245.       menu = menu->children;
  246.       for ( ; menu != NULL; menu = menu->next)
  247.       {
  248.       if (menu->modes == 0)    /* this menu has just been deleted */
  249. ***************
  250. *** 3166,3188 ****
  251.           }
  252.       }
  253.   
  254. !     /* Figure out length of this menu label */
  255. !     len = (int)STRLEN(menu->dname);
  256. !     end = menu->dname + STRLEN(menu->dname);
  257.       padding0 = (columnWidths[0] - GetTextWidth(hdc, menu->dname,
  258. !             (int)(end - menu->dname))) / spaceWidth;
  259.       len += padding0;
  260.       if (menu->actext != NULL)
  261.       {
  262. !         len += (int)STRLEN(menu->actext);
  263. !         acEnd = menu->actext + STRLEN(menu->actext);
  264. !         textWidth = GetTextWidth(hdc, menu->actext, (int)(acEnd - menu->actext));
  265.       }
  266.       else
  267.           textWidth = 0;
  268. - 
  269.       padding1 = (columnWidths[1] - textWidth) / spaceWidth;
  270.       len += padding1;
  271.       if (menu->children == NULL)
  272.       {
  273.           padding2 = submenuWidth / spaceWidth;
  274. --- 3168,3192 ----
  275.           }
  276.       }
  277.   
  278. !     /* Figure out minimal length of this menu label.  Use "name" for the
  279. !      * actual text, "dname" for estimating the displayed size.  "name"
  280. !      * has "&a" for mnemonic and includes the accelerator. */
  281. !     len = nameLen = (int)STRLEN(menu->name);
  282.       padding0 = (columnWidths[0] - GetTextWidth(hdc, menu->dname,
  283. !                       (int)STRLEN(menu->dname))) / spaceWidth;
  284.       len += padding0;
  285. + 
  286.       if (menu->actext != NULL)
  287.       {
  288. !         acLen = (int)STRLEN(menu->actext);
  289. !         len += acLen;
  290. !         textWidth = GetTextWidth(hdc, menu->actext, acLen);
  291.       }
  292.       else
  293.           textWidth = 0;
  294.       padding1 = (columnWidths[1] - textWidth) / spaceWidth;
  295.       len += padding1;
  296. + 
  297.       if (menu->children == NULL)
  298.       {
  299.           padding2 = submenuWidth / spaceWidth;
  300. ***************
  301. *** 3199,3212 ****
  302.       text = label = alloc((unsigned)len + 1);
  303.       if (label == NULL)
  304.           break;
  305. !     STRNCPY(text, menu->dname, end - menu->dname);
  306. !     text += end - menu->dname;
  307.       while (padding0-- > 0)
  308.           *text++ = ' ';
  309.       if (menu->actext != NULL)
  310.       {
  311. !         STRNCPY(text, menu->actext, acEnd - menu->actext);
  312. !         text += acEnd - menu->actext;
  313.       }
  314.       while (padding1-- > 0)
  315.           *text++ = ' ';
  316. --- 3203,3219 ----
  317.       text = label = alloc((unsigned)len + 1);
  318.       if (label == NULL)
  319.           break;
  320. ! 
  321. !     STRNCPY(text, menu->name, nameLen);
  322. !     text = vim_strchr(text, TAB);        /* stop at TAB before actext */
  323. !     if (text == NULL)
  324. !         text = label + nameLen;        /* no actext, use whole name */
  325.       while (padding0-- > 0)
  326.           *text++ = ' ';
  327.       if (menu->actext != NULL)
  328.       {
  329. !         STRNCPY(text, menu->actext, acLen);
  330. !         text += acLen;
  331.       }
  332.       while (padding1-- > 0)
  333.           *text++ = ' ';
  334. ***************
  335. *** 3221,3227 ****
  336.           *text++ = ' ';
  337.       }
  338.       *text = NUL;
  339. -     *end = NUL;
  340.   
  341.       /*
  342.        * BS_LEFT will just be ignored on Win32s/NT3.5x - on
  343. --- 3228,3233 ----
  344. *** ../vim-6.2.266/src/menu.c    Sun Feb 15 13:33:50 2004
  345. --- src/menu.c    Sun Feb 15 22:06:06 2004
  346. ***************
  347. *** 1596,1607 ****
  348.       }
  349.       else
  350.       text = vim_strsave(str);
  351. !     if (text != NULL)
  352.       {
  353. !     p = vim_strchr(text, '&');
  354.       if (p != NULL)
  355.       {
  356. !         if (mnemonic != NULL)
  357.   #if !defined(__MVS__) || defined(MOTIF390_MNEMONIC_FIXED)
  358.           *mnemonic = p[1];
  359.   #else
  360. --- 1596,1611 ----
  361.       }
  362.       else
  363.       text = vim_strsave(str);
  364. ! 
  365. !     /* Find mnemonic characters "&a" and reduce "&&" to "&". */
  366. !     for (p = text; p != NULL; )
  367.       {
  368. !     p = vim_strchr(p, '&');
  369.       if (p != NULL)
  370.       {
  371. !         if (p[1] == NUL)        /* trailing "&" */
  372. !         break;
  373. !         if (mnemonic != NULL && p[1] != '&')
  374.   #if !defined(__MVS__) || defined(MOTIF390_MNEMONIC_FIXED)
  375.           *mnemonic = p[1];
  376.   #else
  377. ***************
  378. *** 1618,1623 ****
  379. --- 1622,1628 ----
  380.           }
  381.   #endif
  382.           mch_memmove(p, p + 1, STRLEN(p));
  383. +         p = p + 1;
  384.       }
  385.       }
  386.       return text;
  387. *** ../vim-6.2.266/src/version.c    Tue Feb 17 20:44:14 2004
  388. --- src/version.c    Tue Feb 17 21:01:21 2004
  389. ***************
  390. *** 639,640 ****
  391. --- 639,642 ----
  392.   {   /* Add new patch number below this line */
  393. + /**/
  394. +     267,
  395.   /**/
  396.  
  397. -- 
  398. hundred-and-one symptoms of being an internet addict:
  399. 170. You introduce your wife as "my_lady@home.wife" and refer to your
  400.      children as "forked processes."
  401.  
  402.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  403. ///        Sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  404. \\\              Project leader for A-A-P -- http://www.A-A-P.org        ///
  405.  \\\  Help AIDS victims, buy here: http://ICCF-Holland.org/click1.html  ///
  406.