7.2.154

   1 To: vim-dev@vim.org
   2 Subject: Patch 7.2.154
   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 7.2.154 (after 7.2.132)
  11 Problem:    ":cd" is still possible in a SwapExists autocmd.
  12 Solution:   Set allbuf_lock in do_swapexists().
  13 Files:      src/memline.c
  14
  15
  16 *** ../vim-7.2.153/src/memline.c        Wed Apr 22 15:37:12 2009
  17 --- src/memline.c       Wed Apr 22 15:54:48 2009
  18 ***************
  19 *** 3771,3778 ****
  20       set_vim_var_string(VV_SWAPCHOICE, NULL, -1);
  21
  22       /* Trigger SwapExists autocommands with <afile> set to the file being
  23 !      * edited. */
  24       apply_autocmds(EVENT_SWAPEXISTS, buf->b_fname, NULL, FALSE, NULL);
  25
  26       set_vim_var_string(VV_SWAPNAME, NULL, -1);
  27
  28 --- 3771,3780 ----
  29       set_vim_var_string(VV_SWAPCHOICE, NULL, -1);
  30
  31       /* Trigger SwapExists autocommands with <afile> set to the file being
  32 !      * edited.  Disallow changing directory here. */
  33 !     ++allbuf_lock;
  34       apply_autocmds(EVENT_SWAPEXISTS, buf->b_fname, NULL, FALSE, NULL);
  35 +     --allbuf_lock;
  36
  37       set_vim_var_string(VV_SWAPNAME, NULL, -1);
  38
  39 ***************
  40 *** 3798,3803 ****
  41 --- 3800,3806 ----
  42    *
  43    * Note: If BASENAMELEN is not correct, you will get error messages for
  44    *     not being able to open the swapfile
  45 +  * Note: May trigger SwapExists autocmd, pointers may change!
  46    */
  47       static char_u *
  48   findswapname(buf, dirp, old_fname)
  49 *** ../vim-7.2.153/src/version.c        Wed Apr 22 15:37:12 2009
  50 --- src/version.c       Wed Apr 22 15:55:48 2009
  51 ***************
  52 *** 678,679 ****
  53 --- 678,681 ----
  54   {   /* Add new patch number below this line */
  55 + /**/
  56 +     154,
  57   /**/
  58
  59 --
  60 ARTHUR:  Be quiet!
  61 DENNIS:  Well you can't expect to wield supreme executive power just 'cause
  62          some watery tart threw a sword at you!
  63 ARTHUR:  Shut up!
  64 DENNIS:  I mean, if I went around sayin' I was an empereror just because some
  65          moistened bint had lobbed a scimitar at me they'd put me away!
  66                                   The Quest for the Holy Grail (Monty Python)
  67
  68  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  69 ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  70 \\\        download, build and distribute -- http://www.A-A-P.org        ///
  71  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///