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.1.165 < prev    next >
Encoding:
Internet Message Format  |  2002-11-04  |  6.2 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.1.165
  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.1.165
  11. Problem:    Making changes in several lines and then a change in one of these
  12.         lines that splits it in two or more lines, undo information was
  13.         corrupted.  May cause a crash. (Dave Fishburn)
  14. Solution:   When skipping to save a line for undo because it was already
  15.         saved, move it to become the last saved line, so that when the
  16.         command changes the line count other saved lines are not involved.
  17. Files:        src/undo.c
  18.  
  19.  
  20. *** ../vim61.164/src/undo.c    Tue Apr  9 23:19:52 2002
  21. --- src/undo.c    Sat Aug 24 23:11:19 2002
  22. ***************
  23. *** 155,160 ****
  24. --- 155,161 ----
  25.       long        i;
  26.       struct u_header    *uhp;
  27.       u_entry_T        *uep;
  28. +     u_entry_T        *prev_uep;
  29.       long        size;
  30.   
  31.       /*
  32. ***************
  33. *** 258,310 ****
  34.       if (size == 1)
  35.       {
  36.           uep = u_get_headentry();
  37.           for (i = 0; i < 10; ++i)
  38.           {
  39.           if (uep == NULL)
  40.               break;
  41.   
  42. !         /* If lines have been inserted/deleted we give up. */
  43. !         if (curbuf->b_u_newhead->uh_getbot_entry != uep
  44. !             ? (uep->ue_top + uep->ue_size + 1
  45. !                 != (uep->ue_bot == 0
  46. !                 ? curbuf->b_ml.ml_line_count + 1
  47. !                 : uep->ue_bot))
  48. !             : uep->ue_lcount != curbuf->b_ml.ml_line_count)
  49.               break;
  50.   
  51.           /* If it's the same line we can skip saving it again. */
  52.           if (uep->ue_size == 1 && uep->ue_top == top)
  53.           {
  54. -             /* If it's not the last entry, get ue_bot for the last
  55. -              * entry now.  Following deleted/inserted lines go to the
  56. -              * re-used entry. */
  57.               if (i > 0)
  58.               {
  59.               u_getbot();
  60.               curbuf->b_u_synced = FALSE;
  61.               }
  62.   
  63. !             /* The line count might change afterwards. */
  64.               if (newbot != 0)
  65. -             {
  66. -             /* When changing the line count and it's not the
  67. -              * newest entry, must adjust the line numbers of older
  68. -              * entries. */
  69. -             if (uep != curbuf->b_u_newhead->uh_entry
  70. -                              && uep->ue_bot != newbot)
  71. -             {
  72. -                 u_entry_T    *p;
  73. - 
  74. -                 for (p = curbuf->b_u_newhead->uh_entry;
  75. -                              p != uep; p = p->ue_next)
  76. -                 {
  77. -                 if (p->ue_bot != 0)
  78. -                     p->ue_bot -= uep->ue_bot - newbot;
  79. -                 p->ue_top -= uep->ue_bot - newbot;
  80. -                 }
  81. -             }
  82.               uep->ue_bot = newbot;
  83. -             }
  84.               else if (bot > curbuf->b_ml.ml_line_count)
  85.               uep->ue_bot = 0;
  86.               else
  87. --- 259,308 ----
  88.       if (size == 1)
  89.       {
  90.           uep = u_get_headentry();
  91. +         prev_uep = NULL;
  92.           for (i = 0; i < 10; ++i)
  93.           {
  94.           if (uep == NULL)
  95.               break;
  96.   
  97. !         /* If lines have been inserted/deleted we give up.
  98. !          * Also when the line was included in a multi-line save. */
  99. !         if ((curbuf->b_u_newhead->uh_getbot_entry != uep
  100. !                 ? (uep->ue_top + uep->ue_size + 1
  101. !                 != (uep->ue_bot == 0
  102. !                     ? curbuf->b_ml.ml_line_count + 1
  103. !                     : uep->ue_bot))
  104. !                 : uep->ue_lcount != curbuf->b_ml.ml_line_count)
  105. !             || (uep->ue_size > 1
  106. !                 && top >= uep->ue_top
  107. !                 && top + 2 <= uep->ue_top + uep->ue_size + 1))
  108.               break;
  109.   
  110.           /* If it's the same line we can skip saving it again. */
  111.           if (uep->ue_size == 1 && uep->ue_top == top)
  112.           {
  113.               if (i > 0)
  114.               {
  115. +             /* It's not the last entry: get ue_bot for the last
  116. +              * entry now.  Following deleted/inserted lines go to
  117. +              * the re-used entry. */
  118.               u_getbot();
  119.               curbuf->b_u_synced = FALSE;
  120. + 
  121. +             /* Move the found entry to become the last entry.  The
  122. +              * order of undo/redo doesn't matter for the entries
  123. +              * we move it over, since they don't change the line
  124. +              * count and don't include this line.  It does matter
  125. +              * for the found entry if the line count is changed by
  126. +              * the executed command. */
  127. +             prev_uep->ue_next = uep->ue_next;
  128. +             uep->ue_next = curbuf->b_u_newhead->uh_entry;
  129. +             curbuf->b_u_newhead->uh_entry = uep;
  130.               }
  131.   
  132. !             /* The executed command may change the line count. */
  133.               if (newbot != 0)
  134.               uep->ue_bot = newbot;
  135.               else if (bot > curbuf->b_ml.ml_line_count)
  136.               uep->ue_bot = 0;
  137.               else
  138. ***************
  139. *** 314,319 ****
  140. --- 312,318 ----
  141.               }
  142.               return OK;
  143.           }
  144. +         prev_uep = uep;
  145.           uep = uep->ue_next;
  146.           }
  147.       }
  148. ***************
  149. *** 775,782 ****
  150.       {
  151.       /*
  152.        * the new ue_bot is computed from the number of lines that has been
  153. !      * inserted (0 - deleted) since calling u_save. This is equal to the old
  154. !      * line count subtracted from the current line count.
  155.        */
  156.       extra = curbuf->b_ml.ml_line_count - uep->ue_lcount;
  157.       uep->ue_bot = uep->ue_top + uep->ue_size + 1 + extra;
  158. --- 774,781 ----
  159.       {
  160.       /*
  161.        * the new ue_bot is computed from the number of lines that has been
  162. !      * inserted (0 - deleted) since calling u_save. This is equal to the
  163. !      * old line count subtracted from the current line count.
  164.        */
  165.       extra = curbuf->b_ml.ml_line_count - uep->ue_lcount;
  166.       uep->ue_bot = uep->ue_top + uep->ue_size + 1 + extra;
  167. ***************
  168. *** 788,804 ****
  169.                            * without deleting the current
  170.                            * ones */
  171.       }
  172. - 
  173. -     /* If not the newest entry and lines have been inserted or deleted,
  174. -      * newer entries must have their line numbers adjusted. */
  175. -     if (extra != 0)
  176. -         for (uep = curbuf->b_u_newhead->uh_entry;
  177. -             uep != curbuf->b_u_newhead->uh_getbot_entry;
  178. -             uep = uep->ue_next)
  179. -         {
  180. -         uep->ue_bot -= extra;
  181. -         uep->ue_top -= extra;
  182. -         }
  183.   
  184.       curbuf->b_u_newhead->uh_getbot_entry = NULL;
  185.       }
  186. --- 787,792 ----
  187. *** ../vim61.164/src/version.c    Sat Aug 24 23:19:56 2002
  188. --- src/version.c    Sat Aug 24 23:21:09 2002
  189. ***************
  190. *** 608,609 ****
  191. --- 608,611 ----
  192.   {   /* Add new patch number below this line */
  193. + /**/
  194. +     165,
  195.   /**/
  196.  
  197. -- 
  198.        His head smashed in,  and his heart cut out,
  199.        And his liver removed, and his bowels unplugged,
  200.        And his nostrils raped, and his bottom burned off,
  201.        And his penis split ... and his ...
  202.                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
  203.  
  204.  ///  Bram Moolenaar -- Bram@moolenaar.net -- http://www.moolenaar.net  \\\
  205. ///   Creator of Vim -- http://vim.sf.net -- ftp://ftp.vim.org/pub/vim   \\\
  206. \\\           Project leader for A-A-P -- http://www.a-a-p.org           ///
  207.  \\\ Lord Of The Rings helps Uganda - http://iccf-holland.org/lotr.html ///
  208.