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.3 / 6.3.055 < prev    next >
Encoding:
Internet Message Format  |  2005-01-12  |  8.6 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.3.055
  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.3.055 (after 6.3.013)
  11. Problem:    Can't use getcmdline(), getcmdpos() or setcmdpos() with <C-R>=
  12.         when editing a command line.  Using <C-\>e may crash Vim. (Peter
  13.         Winters)
  14. Solution:   When moving ccline out of the way for recursive use, make it
  15.         available to the functions that need it.  Also save and restore
  16.         ccline when calling get_expr_line().  Make ccline.cmdbuf NULL at
  17.         the end of getcmdline().
  18. Files:        src/ex_getln.c
  19.  
  20.  
  21. *** ../vim-6.3.054/src/ex_getln.c    Fri Oct 22 11:45:17 2004
  22. --- src/ex_getln.c    Thu Jan 13 14:06:56 2005
  23. ***************
  24. *** 80,85 ****
  25. --- 80,87 ----
  26.   static void    alloc_cmdbuff __ARGS((int len));
  27.   static int    realloc_cmdbuff __ARGS((int len));
  28.   static void    draw_cmdline __ARGS((int start, int len));
  29. + static void    save_cmdline __ARGS((struct cmdline_info *ccp));
  30. + static void    restore_cmdline __ARGS((struct cmdline_info *ccp));
  31.   static int    cmdline_paste __ARGS((int regname, int literally));
  32.   #if defined(FEAT_XIM) && defined(FEAT_GUI_GTK)
  33.   static void    redrawcmd_preedit __ARGS((void));
  34. ***************
  35. *** 589,596 ****
  36.   #ifdef FEAT_EVAL
  37.           else if (c == 'e')
  38.           {
  39. !         struct cmdline_info        save_ccline;
  40. !         char_u            *p;
  41.   
  42.           /*
  43.            * Replace the command line with the result of an expression.
  44. --- 591,598 ----
  45.   #ifdef FEAT_EVAL
  46.           else if (c == 'e')
  47.           {
  48. !         struct cmdline_info save_ccline;
  49. !         char_u            *p = NULL;
  50.   
  51.           /*
  52.            * Replace the command line with the result of an expression.
  53. ***************
  54. *** 601,614 ****
  55.               new_cmdpos = 99999;    /* keep it at the end */
  56.           else
  57.               new_cmdpos = ccline.cmdpos;
  58. !         save_ccline = ccline;
  59. !         ccline.cmdbuff = NULL;
  60. !         ccline.cmdprompt = NULL;
  61.           c = get_expr_register();
  62. !         ccline = save_ccline;
  63.           if (c == '=')
  64.           {
  65.               p = get_expr_line();
  66.               if (p != NULL
  67.                    && realloc_cmdbuff((int)STRLEN(p) + 1) == OK)
  68.               {
  69. --- 603,618 ----
  70.               new_cmdpos = 99999;    /* keep it at the end */
  71.           else
  72.               new_cmdpos = ccline.cmdpos;
  73. ! 
  74. !         save_cmdline(&save_ccline);
  75.           c = get_expr_register();
  76. !         restore_cmdline(&save_ccline);
  77.           if (c == '=')
  78.           {
  79. +             save_cmdline(&save_ccline);
  80.               p = get_expr_line();
  81. +             restore_cmdline(&save_ccline);
  82. + 
  83.               if (p != NULL
  84.                    && realloc_cmdbuff((int)STRLEN(p) + 1) == OK)
  85.               {
  86. ***************
  87. *** 1027,1037 ****
  88.               }
  89.               else
  90.               {
  91. !             save_ccline = ccline;
  92. !             ccline.cmdbuff = NULL;
  93. !             ccline.cmdprompt = NULL;
  94.               c = get_expr_register();
  95. !             ccline = save_ccline;
  96.               }
  97.           }
  98.   #endif
  99. --- 1031,1039 ----
  100.               }
  101.               else
  102.               {
  103. !             save_cmdline(&save_ccline);
  104.               c = get_expr_register();
  105. !             restore_cmdline(&save_ccline);
  106.               }
  107.           }
  108.   #endif
  109. ***************
  110. *** 1723,1729 ****
  111.       ui_cursor_shape();        /* may show different cursor shape */
  112.   #endif
  113.   
  114. !     return ccline.cmdbuff;
  115.   }
  116.   
  117.   #if (defined(FEAT_CRYPT) || defined(FEAT_EVAL)) || defined(PROTO)
  118. --- 1725,1737 ----
  119.       ui_cursor_shape();        /* may show different cursor shape */
  120.   #endif
  121.   
  122. !     {
  123. !     char_u *p = ccline.cmdbuff;
  124. ! 
  125. !     /* Make ccline empty, getcmdline() may try to use it. */
  126. !     ccline.cmdbuff = NULL;
  127. !     return p;
  128. !     }
  129.   }
  130.   
  131.   #if (defined(FEAT_CRYPT) || defined(FEAT_EVAL)) || defined(PROTO)
  132. ***************
  133. *** 1743,1754 ****
  134.       struct cmdline_info    save_ccline;
  135.       int            msg_col_save = msg_col;
  136.   
  137. !     save_ccline = ccline;
  138. !     ccline.cmdbuff = NULL;
  139.       ccline.cmdprompt = prompt;
  140.       ccline.cmdattr = attr;
  141.       s = getcmdline(firstc, 1L, 0);
  142. !     ccline = save_ccline;
  143.       /* Restore msg_col, the prompt from input() may have changed it. */
  144.       msg_col = msg_col_save;
  145.   
  146. --- 1751,1761 ----
  147.       struct cmdline_info    save_ccline;
  148.       int            msg_col_save = msg_col;
  149.   
  150. !     save_cmdline(&save_ccline);
  151.       ccline.cmdprompt = prompt;
  152.       ccline.cmdattr = attr;
  153.       s = getcmdline(firstc, 1L, 0);
  154. !     restore_cmdline(&save_ccline);
  155.       /* Restore msg_col, the prompt from input() may have changed it. */
  156.       msg_col = msg_col_save;
  157.   
  158. ***************
  159. *** 2537,2542 ****
  160. --- 2544,2583 ----
  161.       return retval;
  162.   }
  163.   
  164. + static struct cmdline_info  prev_ccline;
  165. + static int            prev_ccline_used = FALSE;
  166. + 
  167. + /*
  168. +  * Save ccline, because obtaining the "=" register may execute "normal :cmd"
  169. +  * and overwrite it.  But get_cmdline_str() may need it, thus make it
  170. +  * available globally in prev_ccline.
  171. +  */
  172. +     static void
  173. + save_cmdline(ccp)
  174. +     struct cmdline_info *ccp;
  175. + {
  176. +     if (!prev_ccline_used)
  177. +     {
  178. +     vim_memset(&prev_ccline, 0, sizeof(struct cmdline_info));
  179. +     prev_ccline_used = TRUE;
  180. +     }
  181. +     *ccp = prev_ccline;
  182. +     prev_ccline = ccline;
  183. +     ccline.cmdbuff = NULL;
  184. +     ccline.cmdprompt = NULL;
  185. + }
  186. + 
  187. + /*
  188. +  * Resture ccline after it has been saved with save_cmdline().
  189. +  */
  190. +     static void
  191. + restore_cmdline(ccp)
  192. +     struct cmdline_info *ccp;
  193. + {
  194. +     ccline = prev_ccline;
  195. +     prev_ccline = *ccp;
  196. + }
  197. + 
  198.   /*
  199.    * paste a yank register into the command line.
  200.    * used by CTRL-R command in command-line mode
  201. ***************
  202. *** 2571,2583 ****
  203.       regname = may_get_selection(regname);
  204.   #endif
  205.   
  206. !     /* Need to save and restore ccline, because obtaining the "=" register may
  207. !      * execute "normal :cmd" and overwrite it. */
  208. !     save_ccline = ccline;
  209. !     ccline.cmdbuff = NULL;
  210. !     ccline.cmdprompt = NULL;
  211.       i = get_spec_reg(regname, &arg, &allocated, TRUE);
  212. !     ccline = save_ccline;
  213.   
  214.       if (i)
  215.       {
  216. --- 2612,2621 ----
  217.       regname = may_get_selection(regname);
  218.   #endif
  219.   
  220. !     /* Need to save and restore ccline. */
  221. !     save_cmdline(&save_ccline);
  222.       i = get_spec_reg(regname, &arg, &allocated, TRUE);
  223. !     restore_cmdline(&save_ccline);
  224.   
  225.       if (i)
  226.       {
  227. ***************
  228. *** 4541,4546 ****
  229. --- 4579,4602 ----
  230.       return history[histype][hisidx[histype]].hisnum;
  231.   }
  232.   
  233. + static struct cmdline_info *get_ccline_ptr __ARGS((void));
  234. + 
  235. + /*
  236. +  * Get pointer to the command line info to use. cmdline_paste() may clear
  237. +  * ccline and put the previous value in prev_ccline.
  238. +  */
  239. +     static struct cmdline_info *
  240. + get_ccline_ptr()
  241. + {
  242. +     if ((State & CMDLINE) == 0)
  243. +     return NULL;
  244. +     if (ccline.cmdbuff != NULL)
  245. +     return &ccline;
  246. +     if (prev_ccline_used && prev_ccline.cmdbuff != NULL)
  247. +     return &prev_ccline;
  248. +     return NULL;
  249. + }
  250. + 
  251.   /*
  252.    * Get the current command line in allocated memory.
  253.    * Only works when the command line is being edited.
  254. ***************
  255. *** 4549,4557 ****
  256.       char_u *
  257.   get_cmdline_str()
  258.   {
  259. !     if (ccline.cmdbuff == NULL || (State & CMDLINE) == 0)
  260.       return NULL;
  261. !     return vim_strnsave(ccline.cmdbuff, ccline.cmdlen);
  262.   }
  263.   
  264.   /*
  265. --- 4605,4615 ----
  266.       char_u *
  267.   get_cmdline_str()
  268.   {
  269. !     struct cmdline_info *p = get_ccline_ptr();
  270. ! 
  271. !     if (p == NULL)
  272.       return NULL;
  273. !     return vim_strnsave(p->cmdbuff, p->cmdlen);
  274.   }
  275.   
  276.   /*
  277. ***************
  278. *** 4563,4571 ****
  279.       int
  280.   get_cmdline_pos()
  281.   {
  282. !     if (ccline.cmdbuff == NULL || (State & CMDLINE) == 0)
  283.       return -1;
  284. !     return ccline.cmdpos;
  285.   }
  286.   
  287.   /*
  288. --- 4621,4631 ----
  289.       int
  290.   get_cmdline_pos()
  291.   {
  292. !     struct cmdline_info *p = get_ccline_ptr();
  293. ! 
  294. !     if (p == NULL)
  295.       return -1;
  296. !     return p->cmdpos;
  297.   }
  298.   
  299.   /*
  300. ***************
  301. *** 4577,4583 ****
  302.   set_cmdline_pos(pos)
  303.       int        pos;
  304.   {
  305. !     if (ccline.cmdbuff == NULL || (State & CMDLINE) == 0)
  306.       return 1;
  307.   
  308.       /* The position is not set directly but after CTRL-\ e or CTRL-R = has
  309. --- 4637,4645 ----
  310.   set_cmdline_pos(pos)
  311.       int        pos;
  312.   {
  313. !     struct cmdline_info *p = get_ccline_ptr();
  314. ! 
  315. !     if (p == NULL)
  316.       return 1;
  317.   
  318.       /* The position is not set directly but after CTRL-\ e or CTRL-R = has
  319. *** ../vim-6.3.054/src/version.c    Wed Jan  5 11:17:36 2005
  320. --- src/version.c    Thu Jan 13 14:08:12 2005
  321. ***************
  322. *** 643,644 ****
  323. --- 643,646 ----
  324.   {   /* Add new patch number below this line */
  325. + /**/
  326. +     55,
  327.   /**/
  328.  
  329. -- 
  330. ARTHUR:  Well, I AM king...
  331. DENNIS:  Oh king, eh, very nice.  An' how'd you get that, eh?  By exploitin'
  332.          the workers -- by 'angin' on to outdated imperialist dogma which
  333.          perpetuates the economic an' social differences in our society!  If
  334.          there's ever going to be any progress--
  335.                                   The Quest for the Holy Grail (Monty Python)
  336.  
  337.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  338. ///        Sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  339. \\\              Project leader for A-A-P -- http://www.A-A-P.org        ///
  340.  \\\     Buy LOTR 3 and help AIDS victims -- http://ICCF.nl/lotr.html   ///
  341.