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 / old / 5.4.27 < prev    next >
Encoding:
Internet Message Format  |  1999-08-12  |  3.8 KB
  1. To: Mike Steed <mrsteed@usa.net>
  2. In-Reply-To: <19990813155859.14060.qmail@www0r.netaddress.usa.net>
  3. Cc: vim-dev@vim.org
  4. Subject: patch 5.4.27 (was: 5.4 popup menu bug (Win32))
  5. Fcc: outbox
  6. From: Bram Moolenaar <Bram@moolenaar.net>
  7. ------------
  8.  
  9. Mike Steed wrote:
  10.  
  11. >    Here's a problem (unpatched Vim-5.4, pardon me if this has been reported):
  12. > 
  13. >    gvim -u NONE -U NONE
  14. >    :amenu PopUp.Hello :echo 'Hello'^M
  15. >    (popup menu item works)
  16. > 
  17. >    :aunmenu PopUp.Hello
  18. >    (popup menu item still shows up in the menu and works)
  19. > 
  20. >    :aunmenu PopUp.Hello
  21. >    Vim says "No menu of that name".
  22. > 
  23. >    I hope this isn't yet another Win32 oddity that depends on my particular
  24. > version of IE, comctl*, etc., etc.
  25.  
  26. As Charles Campbell noted, there also is a problem on Motif.  I can reproduce
  27. both.
  28.  
  29. Hmm, I have it fixed now for Motif, but the menu is still there for Win32.
  30. Perhaps this has something to do with the tear-off items.  Or deleting PopUp
  31. menu items just doesn't work.
  32.  
  33.  
  34. Patch 5.4.27
  35. Problem:    Entries in the PopUp[nvic] menus were added for several modes, but
  36.         only deleted for the mode they were used for.  This resulted in
  37.         the  entry remaining in the PopUp menu.
  38.         When removing a PopUp[nvic] menu, the name had been truncated,
  39.         could result in greying-out the whole PopUp menu.
  40. Solution:   Remove entries for all modes from the PopUp[nvic] menus.  Remove
  41.         the PopUp[nvic] menu entries first, before the name is changed.
  42. Files:        src/menu.c
  43.  
  44.  
  45. *** ../vim-5.4.26/src/menu.c    Tue Aug 10 16:09:26 1999
  46. --- src/menu.c    Fri Aug 13 22:09:30 1999
  47. ***************
  48. *** 139,145 ****
  49.       {
  50.       if (STRCMP(menu_path, "*") == 0)    /* meaning: remove all menus */
  51.           menu_path = (char_u *)"";
  52. -     remove_menu(&root_menu, menu_path, modes, FALSE);
  53.   
  54.       /*
  55.        * For the PopUp menu, remove a menu for each mode separately.
  56. --- 139,144 ----
  57. ***************
  58. *** 152,162 ****
  59.               p = popup_mode_name(menu_path, i);
  60.               if (p != NULL)
  61.               {
  62. !             remove_menu(&root_menu, p, 1 << i, TRUE);
  63.               vim_free(p);
  64.               }
  65.           }
  66.       }
  67.       }
  68.       else
  69.       {
  70. --- 151,164 ----
  71.               p = popup_mode_name(menu_path, i);
  72.               if (p != NULL)
  73.               {
  74. !             remove_menu(&root_menu, p, MENU_ALL_MODES, TRUE);
  75.               vim_free(p);
  76.               }
  77.           }
  78.       }
  79. + 
  80. +     /* Careful: remove_menu() changes menu_path */
  81. +     remove_menu(&root_menu, menu_path, modes, FALSE);
  82.       }
  83.       else
  84.       {
  85. *** ../vim-5.4.26/src/version.h    Fri Aug 13 13:12:01 1999
  86. --- src/version.h    Fri Aug 13 22:07:14 1999
  87. ***************
  88. *** 19,26 ****
  89.   #define VIM_VERSION_MINOR_STR        "4"
  90.   #define VIM_VERSION_BUILD         57
  91.   #define VIM_VERSION_BUILD_STR        "57"
  92. ! #define VIM_VERSION_PATCHLEVEL         26
  93. ! #define VIM_VERSION_PATCHLEVEL_STR    "26"
  94.   
  95.   /*
  96.    * VIM_VERSION_NODOT is used for the runtime directory name.
  97. --- 19,26 ----
  98.   #define VIM_VERSION_MINOR_STR        "4"
  99.   #define VIM_VERSION_BUILD         57
  100.   #define VIM_VERSION_BUILD_STR        "57"
  101. ! #define VIM_VERSION_PATCHLEVEL         27
  102. ! #define VIM_VERSION_PATCHLEVEL_STR    "27"
  103.   
  104.   /*
  105.    * VIM_VERSION_NODOT is used for the runtime directory name.
  106. ***************
  107. *** 30,35 ****
  108.    */
  109.   #define VIM_VERSION_NODOT    "vim54"
  110.   #define VIM_VERSION_SHORT    "5.4"
  111. ! #define VIM_VERSION_MEDIUM    "5.4.26"
  112. ! #define VIM_VERSION_LONG    "VIM - Vi IMproved 5.4.26 (1999 Aug 13)"
  113. ! #define VIM_VERSION_LONG_DATE    "VIM - Vi IMproved 5.4.26 (1999 Aug 13, compiled "
  114. --- 30,35 ----
  115.    */
  116.   #define VIM_VERSION_NODOT    "vim54"
  117.   #define VIM_VERSION_SHORT    "5.4"
  118. ! #define VIM_VERSION_MEDIUM    "5.4.27"
  119. ! #define VIM_VERSION_LONG    "VIM - Vi IMproved 5.4.27 (1999 Aug 13)"
  120. ! #define VIM_VERSION_LONG_DATE    "VIM - Vi IMproved 5.4.27 (1999 Aug 13, compiled "
  121.  
  122. --
  123. hundred-and-one symptoms of being an internet addict:
  124. 84. Books in your bookcase bear the names Bongo, WinSock and Inside OLE
  125.  
  126. --/-/---- Bram Moolenaar ---- Bram@moolenaar.net ---- Bram@vim.org ---\-\--
  127.   \ \    www.vim.org/iccf      www.moolenaar.net       www.vim.org    / /
  128.