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 / unreleased / patches / old / 5.4p.5 < prev    next >
Encoding:
Internet Message Format  |  1999-07-19  |  4.7 KB
  1. To: vim-dev@vim.org
  2. Subject: patch 5.4p.5
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. ------------
  6.  
  7. Another nasty autocommand problem.  It appears more people are using
  8. autocommands these days.  It requires a radical change to how autocommands are
  9. executed.  But not right now.
  10.  
  11.  
  12. Patch 5.4p.5
  13. Problem:    When changing buffers in a BufDelete autocommand, there could be
  14.             ml_line errors and/or a crash. (Schandl)  Was caused by deleting
  15.             the current buffer.
  16. Solution:   When the buffer to be deleted unexpectedly becomes the current
  17.             buffer, don't delete it.
  18.             Also added a check for this in test13.
  19. Files:      src/buffer.c, src/testdir/test13.in, src/testdir/test13.ok
  20.  
  21.  
  22. *** ../vim-5.4p/src/buffer.c    Mon Jul 19 11:08:56 1999
  23. --- src/buffer.c    Tue Jul 20 13:17:13 1999
  24. ***************
  25. *** 185,190 ****
  26. --- 185,194 ----
  27.       int        free_buf;
  28.       int        del_buf;
  29.   {
  30. + #ifdef AUTOCMD
  31. +     int        is_curbuf;
  32. + #endif
  33. + 
  34.       if (buf->b_nwindows > 0)
  35.       --buf->b_nwindows;
  36.       if (buf->b_nwindows == 0 && win != NULL)
  37. ***************
  38. *** 218,226 ****
  39.        * Free all things allocated for this buffer.
  40.        * Also calls the "BufDelete" autocommands when del_buf is TRUE.
  41.        */
  42.       buf_freeall(buf, del_buf);
  43. !     if (!buf_valid(buf))        /* autocommands may delete the buffer */
  44.       return;
  45.   
  46.       /*
  47.        * Remove the buffer from the list.
  48. --- 222,242 ----
  49.        * Free all things allocated for this buffer.
  50.        * Also calls the "BufDelete" autocommands when del_buf is TRUE.
  51.        */
  52. + #ifdef AUTOCMD
  53. +     is_curbuf = (buf == curbuf);
  54. + #endif
  55.       buf_freeall(buf, del_buf);
  56. ! #ifdef AUTOCMD
  57. !     /*
  58. !      * Autocommands may have deleted the buffer.
  59. !      * It's possible that autocommands change curbuf to the one being deleted.
  60. !      * This might cause curbuf to be deleted unexpectedly.  But in some cases
  61. !      * it's OK to delete the curbuf, because a new one is obtained anyway.
  62. !      * Therefore only return if curbuf changed to the deleted buffer.
  63. !      */
  64. !     if (!buf_valid(buf) || (buf == curbuf && !is_curbuf))
  65.       return;
  66. + #endif
  67.   
  68.       /*
  69.        * Remove the buffer from the list.
  70. ***************
  71. *** 267,276 ****
  72.   /*ARGSUSED*/
  73.       void
  74.   buf_freeall(buf, del_buf)
  75. !     BUF        *buf;
  76. !     int        del_buf;    /* buffer is going to be deleted */
  77.   {
  78.   #ifdef AUTOCMD
  79.       apply_autocmds(EVENT_BUFUNLOAD, buf->b_fname, buf->b_fname, FALSE, buf);
  80.       if (!buf_valid(buf))        /* autocommands may delete the buffer */
  81.       return;
  82. --- 283,294 ----
  83.   /*ARGSUSED*/
  84.       void
  85.   buf_freeall(buf, del_buf)
  86. !     BUF        *buf;
  87. !     int        del_buf;    /* buffer is going to be deleted */
  88.   {
  89.   #ifdef AUTOCMD
  90. +     int        is_curbuf = (buf == curbuf);
  91. + 
  92.       apply_autocmds(EVENT_BUFUNLOAD, buf->b_fname, buf->b_fname, FALSE, buf);
  93.       if (!buf_valid(buf))        /* autocommands may delete the buffer */
  94.       return;
  95. ***************
  96. *** 280,285 ****
  97. --- 298,311 ----
  98.       if (!buf_valid(buf))        /* autocommands may delete the buffer */
  99.           return;
  100.       }
  101. +     /*
  102. +      * It's possible that autocommands change curbuf to the one being deleted.
  103. +      * This might cause curbuf to be deleted unexpectedly.  But in some cases
  104. +      * it's OK to delete the curbuf, because a new one is obtained anyway.
  105. +      * Therefore only return if curbuf changed to the deleted buffer.
  106. +      */
  107. +     if (buf == curbuf && !is_curbuf)
  108. +     return;
  109.   #endif
  110.   #ifdef HAVE_TCL
  111.       tcl_buffer_free(buf);
  112. *** ../vim-5.4p/src/testdir/test13.in    Mon Jul 19 11:08:50 1999
  113. --- src/testdir/test13.in    Tue Jul 20 13:31:27 1999
  114. ***************
  115. *** 7,12 ****
  116. --- 7,15 ----
  117.   Also test deleting the buffer on a Unload event.  If this goes wrong there
  118.   will be the ATTENTION prompt.
  119.   
  120. + Also test changing buffers in a BufDel autocommand.  If this goes wrong there
  121. + are ml_line errors and/or a Crash.
  122. + 
  123.   STARTTEST
  124.   :/^start of testfile/,/^end of testfile/w! Xtestje1
  125.   :/^start of testfile/,/^end of testfile/w! Xtestje2
  126. ***************
  127. *** 36,41 ****
  128. --- 39,51 ----
  129.   :e Xtestje2
  130.   :sp Xtestje1
  131.   :e
  132. + :w >>test.out
  133. + :au!
  134. + :only
  135. + :e Xtestje1
  136. + :bdel Xtestje2 Xtestje3 test.out test13.in
  137. + :au BufDelete Xtestje1 buf Xtestje1
  138. + :bd
  139.   :w >>test.out
  140.   :!rm -rf Xtestje*
  141.   :qa!
  142. *** ../vim-5.4p/src/testdir/test13.ok    Mon Jul 19 11:08:50 1999
  143. --- src/testdir/test13.ok    Tue Jul 20 13:35:40 1999
  144. ***************
  145. *** 22,24 ****
  146. --- 22,30 ----
  147.       contents
  148.       contents
  149.   end of testfile
  150. + start of testfile
  151. + testje1
  152. +     contents
  153. +     contents
  154. +     contents
  155. + end of testfile
  156.  
  157. --
  158. hundred-and-one symptoms of being an internet addict:
  159. 174. You know what a listserv is.
  160.  
  161. --/-/---- Bram Moolenaar ---- Bram@moolenaar.net ---- Bram@vim.org ---\-\--
  162.   \ \    www.vim.org/iccf      www.moolenaar.net       www.vim.org    / /
  163.