Patch 7.3.319 (after 7.3.311)
Problem:    Redobuff doesn't always include changes of the completion leader.
Solution:   Insert backspaces as needed. (idea by Taro Muraoka)
Files:      src/edit.c


*** ../vim-7.3.318/src/edit.c   2011-09-14 16:52:02.000000000 +0200
--- src/edit.c  2011-09-21 17:59:10.000000000 +0200
***************
*** 163,168 ****
--- 163,169 ----
  static void ins_compl_set_original_text __ARGS((char_u *str));
  static void ins_compl_addfrommatch __ARGS((void));
  static int  ins_compl_prep __ARGS((int c));
+ static void ins_compl_fixRedoBufForLeader __ARGS((char_u *ptr_arg));
  static buf_T *ins_compl_next_buf __ARGS((buf_T *buf, int flag));
  #if defined(FEAT_COMPL_FUNC) || defined(FEAT_EVAL)
  static void ins_compl_add_list __ARGS((list_T *list));
***************
*** 3713,3721 ****
             * memory that was used, and make sure we can redo the insert. */
            if (compl_curr_match != NULL || compl_leader != NULL || c == Ctrl_E)
            {
-               char_u  *p;
-               int     temp = 0;
- 
                /*
                 * If any of the original typed text has been changed, eg when
                 * ignorecase is set, we must add back-spaces to the redo
--- 3714,3719 ----
***************
*** 3726,3750 ****
                 */
                if (compl_curr_match != NULL && compl_used_match && c != Ctrl_E)
                    ptr = compl_curr_match->cp_str;
-               else if (compl_leader != NULL)
-                   ptr = compl_leader;
                else
!                   ptr = compl_orig_text;
!               if (compl_orig_text != NULL)
!               {
!                   p = compl_orig_text;
!                   for (temp = 0; p[temp] != NUL && p[temp] == ptr[temp];
!                                                                      ++temp)
!                       ;
! #ifdef FEAT_MBYTE
!                   if (temp > 0)
!                       temp -= (*mb_head_off)(compl_orig_text, p + temp);
! #endif
!                   for (p += temp; *p != NUL; mb_ptr_adv(p))
!                       AppendCharToRedobuff(K_BS);
!               }
!               if (ptr != NULL)
!                   AppendToRedobuffLit(ptr + temp, -1);
            }
  
  #ifdef FEAT_CINDENT
--- 3724,3732 ----
                 */
                if (compl_curr_match != NULL && compl_used_match && c != Ctrl_E)
                    ptr = compl_curr_match->cp_str;
                else
!                   ptr = NULL;
!               ins_compl_fixRedoBufForLeader(ptr);
            }
  
  #ifdef FEAT_CINDENT
***************
*** 3834,3839 ****
--- 3816,3859 ----
  }
  
  /*
+  * Fix the redo buffer for the completion leader replacing some of the typed
+  * text.  This inserts backspaces and appends the changed text.
+  * "ptr" is the known leader text or NUL.
+  */
+     static void
+ ins_compl_fixRedoBufForLeader(ptr_arg)
+     char_u *ptr_arg;
+ {
+     int           len;
+     char_u  *p;
+     char_u  *ptr = ptr_arg;
+ 
+     if (ptr == NULL)
+     {
+       if (compl_leader != NULL)
+           ptr = compl_leader;
+       else
+           return;  /* nothing to do */
+     }
+     if (compl_orig_text != NULL)
+     {
+       p = compl_orig_text;
+       for (len = 0; p[len] != NUL && p[len] == ptr[len]; ++len)
+           ;
+ #ifdef FEAT_MBYTE
+       if (len > 0)
+           len -= (*mb_head_off)(p, p + len);
+ #endif
+       for (p += len; *p != NUL; mb_ptr_adv(p))
+           AppendCharToRedobuff(K_BS);
+     }
+     else
+       len = 0;
+     if (ptr != NULL)
+       AppendToRedobuffLit(ptr + len, -1);
+ }
+ 
+ /*
   * Loops through the list of windows, loaded-buffers or non-loaded-buffers
   * (depending on flag) starting from buf and looking for a non-scanned
   * buffer (other than curbuf).        curbuf is special, if it is called with
***************
*** 5241,5246 ****
--- 5261,5270 ----
        else
            edit_submode = (char_u *)_(CTRL_X_MSG(ctrl_x_mode));
  
+       /* If any of the original typed text has been changed we need to fix
+        * the redo buffer. */
+       ins_compl_fixRedoBufForLeader(NULL);
+ 
        /* Always add completion for the original text. */
        vim_free(compl_orig_text);
        compl_orig_text = vim_strnsave(line + compl_col, compl_length);
*** ../vim-7.3.318/src/version.c        2011-09-21 17:33:49.000000000 +0200
--- src/version.c       2011-09-21 18:21:07.000000000 +0200
***************
*** 711,712 ****
--- 711,714 ----
  {   /* Add new patch number below this line */
+ /**/
+     319,
  /**/

-- 
hundred-and-one symptoms of being an internet addict:
27. You refer to your age as 3.x.

 /// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

Raspunde prin e-mail lui