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.220 < prev    next >
Encoding:
Internet Message Format  |  2002-10-12  |  10.2 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.1.220
  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.220
  11. Problem:    When using a BufReadPost autocommand that changes the line count,
  12.         e.g., "$-1join", reloading a file that was changed outside Vim
  13.         does not work properly. (Alan G Isaac)
  14. Solution:   Make the buffer empty before reading the new version of the file.
  15.         Save the lines in a dummy buffer, so that they can be put back
  16.         when reading the file fails.
  17. Files:        src/buffer.c, src/ex_cmds.c, src/fileio.c, src/globals.h,
  18.         src/proto/buffer.pro
  19.  
  20.  
  21. *** ../vim61.219/src/buffer.c    Mon Oct  7 20:45:41 2002
  22. --- src/buffer.c    Sun Oct 13 18:22:22 2002
  23. ***************
  24. *** 4545,4551 ****
  25.       /* Force the 'fileencoding' and 'fileformat' to be equal. */
  26.       if (prep_exarg(&ea, buf) == FAIL)
  27.       {
  28. !     close_buffer(NULL, newbuf, DOBUF_WIPE);
  29.       return TRUE;
  30.       }
  31.   
  32. --- 4545,4551 ----
  33.       /* Force the 'fileencoding' and 'fileformat' to be equal. */
  34.       if (prep_exarg(&ea, buf) == FAIL)
  35.       {
  36. !     wipe_buffer(newbuf, FALSE);
  37.       return TRUE;
  38.       }
  39.   
  40. ***************
  41. *** 4585,4591 ****
  42.   #endif
  43.   
  44.       if (curbuf != newbuf)    /* safety check */
  45. !     wipe_buffer(newbuf);
  46.   
  47.       return differ;
  48.   }
  49. --- 4585,4591 ----
  50.   #endif
  51.   
  52.       if (curbuf != newbuf)    /* safety check */
  53. !     wipe_buffer(newbuf, FALSE);
  54.   
  55.       return differ;
  56.   }
  57. ***************
  58. *** 4595,4605 ****
  59.    * this buffer.  Call this to wipe out a temp buffer that does not contain any
  60.    * marks.
  61.    */
  62.       void
  63. ! wipe_buffer(buf)
  64.       buf_T    *buf;
  65.   {
  66.       if (buf->b_fnum == top_file_num - 1)
  67.       --top_file_num;
  68.       close_buffer(NULL, buf, DOBUF_WIPE);
  69.   }
  70. --- 4595,4616 ----
  71.    * this buffer.  Call this to wipe out a temp buffer that does not contain any
  72.    * marks.
  73.    */
  74. + /*ARGSUSED*/
  75.       void
  76. ! wipe_buffer(buf, aucmd)
  77.       buf_T    *buf;
  78. +     int        aucmd;        /* When TRUE trigger autocommands. */
  79.   {
  80.       if (buf->b_fnum == top_file_num - 1)
  81.       --top_file_num;
  82. + 
  83. + #ifdef FEAT_AUTOCMD
  84. +     if (!aucmd)            /* Don't trigger BufDelete autocommands here. */
  85. +     ++autocmd_block;
  86. + #endif
  87.       close_buffer(NULL, buf, DOBUF_WIPE);
  88. + #ifdef FEAT_AUTOCMD
  89. +     if (!aucmd)
  90. +     --autocmd_block;
  91. + #endif
  92.   }
  93. *** ../vim61.219/src/ex_cmds.c    Wed Sep 25 22:21:44 2002
  94. --- src/ex_cmds.c    Sun Oct 13 18:22:31 2002
  95. ***************
  96. *** 4518,4524 ****
  97.       {
  98.       buf = buflist_findnr(empty_fnum);
  99.       if (buf != NULL)
  100. !         wipe_buffer(buf);
  101.       }
  102.   
  103.       /* keep the previous alternate file */
  104. --- 4518,4524 ----
  105.       {
  106.       buf = buflist_findnr(empty_fnum);
  107.       if (buf != NULL)
  108. !         wipe_buffer(buf, TRUE);
  109.       }
  110.   
  111.       /* keep the previous alternate file */
  112. *** ../vim61.219/src/fileio.c    Sat Oct 12 15:48:03 2002
  113. --- src/fileio.c    Sun Oct 13 18:42:11 2002
  114. ***************
  115. *** 5043,5048 ****
  116. --- 5218,5268 ----
  117.   }
  118.   
  119.   /*
  120. +  * Move all the lines from buffer "frombuf" to buffer "tobuf".
  121. +  * Return OK or FAIL.  When FAIL "tobuf" is incomplete and/or "frombuf" is not
  122. +  * empty.
  123. +  */
  124. +     static int
  125. + move_lines(frombuf, tobuf)
  126. +     buf_T    *frombuf;
  127. +     buf_T    *tobuf;
  128. + {
  129. +     buf_T    *tbuf = curbuf;
  130. +     int        retval = OK;
  131. +     linenr_T    lnum;
  132. +     char_u    *p;
  133. + 
  134. +     /* Copy the lines in "frombuf" to "tobuf". */
  135. +     curbuf = tobuf;
  136. +     for (lnum = 1; lnum <= frombuf->b_ml.ml_line_count; ++lnum)
  137. +     {
  138. +     p = vim_strsave(ml_get_buf(frombuf, lnum, FALSE));
  139. +     if (p == NULL || ml_append(lnum - 1, p, 0, FALSE) == FAIL)
  140. +     {
  141. +         retval = FAIL;
  142. +         break;
  143. +     }
  144. +     }
  145. + 
  146. +     /* Delete all the lines in "frombuf". */
  147. +     if (retval != FAIL)
  148. +     {
  149. +     curbuf = frombuf;
  150. +     while (!bufempty())
  151. +         if (ml_delete(curbuf->b_ml.ml_line_count, FALSE) == FAIL)
  152. +         {
  153. +         /* Oops!  We could try putting back the saved lines, but that
  154. +          * might fail again... */
  155. +         retval = FAIL;
  156. +         break;
  157. +         }
  158. +     }
  159. + 
  160. +     curbuf = tbuf;
  161. +     return retval;
  162. + }
  163. + 
  164. + /*
  165.    * Check if buffer "buf" has been changed.
  166.    * Also check if the file for a new buffer unexpectedly appeared.
  167.    * return 1 if a changed buffer was found.
  168. ***************
  169. *** 5230,5240 ****
  170.   
  171.       if (reload)
  172.       {
  173. -     linenr_T    old_line_count = buf->b_ml.ml_line_count;
  174.       exarg_T        ea;
  175.       pos_T        old_cursor;
  176.       linenr_T    old_topline;
  177.       int        old_ro = curbuf->b_p_ro;
  178.   #ifdef FEAT_AUTOCMD
  179.       aco_save_T    aco;
  180.   
  181. --- 5450,5461 ----
  182.   
  183.       if (reload)
  184.       {
  185.       exarg_T        ea;
  186.       pos_T        old_cursor;
  187.       linenr_T    old_topline;
  188.       int        old_ro = curbuf->b_p_ro;
  189. +     buf_T        *savebuf;
  190. +     int        saved = OK;
  191.   #ifdef FEAT_AUTOCMD
  192.       aco_save_T    aco;
  193.   
  194. ***************
  195. *** 5254,5280 ****
  196.       {
  197.           old_cursor = curwin->w_cursor;
  198.           old_topline = curwin->w_topline;
  199.           if (bufempty())
  200. !         old_line_count = 0;
  201. !         curbuf->b_flags |= BF_CHECK_RO;    /* check for RO again */
  202. ! #ifdef FEAT_AUTOCMD
  203. !         keep_filetype = TRUE;        /* don't detect 'filetype' */
  204. ! #endif
  205. !         if (readfile(buf->b_ffname, buf->b_fname, (linenr_T)0, (linenr_T)0,
  206. !             (linenr_T)MAXLNUM, &ea, READ_NEW) == FAIL)
  207. !         EMSG2(_("E321: Could not reload \"%s\""), buf->b_fname);
  208.           else
  209.           {
  210. !         /* Delete the old lines. */
  211. !         while (old_line_count-- > 0)
  212. !             ml_delete(buf->b_ml.ml_line_count, FALSE);
  213. !         /* Mark the buffer as unmodified and free undo info. */
  214. !         unchanged(buf, TRUE);
  215. !         u_blockfree(buf);
  216. !         u_clearall(buf);
  217.           }
  218.           vim_free(ea.cmd);
  219.   
  220.   #ifdef FEAT_DIFF
  221.           /* Invalidate diff info if necessary. */
  222.           diff_invalidate();
  223. --- 5475,5547 ----
  224.       {
  225.           old_cursor = curwin->w_cursor;
  226.           old_topline = curwin->w_topline;
  227. + 
  228. +         /*
  229. +          * To behave like when a new file is edited (matters for
  230. +          * BufReadPost autocommands) we first need to delete the current
  231. +          * buffer contents.  But if reading the file fails we should keep
  232. +          * the old contents.  Can't use memory only, the file might be
  233. +          * too big.  Use a hidden buffer to move the buffer contents to.
  234. +          */
  235.           if (bufempty())
  236. !         savebuf = NULL;
  237.           else
  238.           {
  239. !         /* Allocate a buffer without putting it in the buffer list. */
  240. !         savebuf = buflist_new(NULL, NULL, (linenr_T)1, BLN_DUMMY);
  241. !         if (savebuf != NULL)
  242. !         {
  243. !             /* Open the memline. */
  244. !             curbuf = savebuf;
  245. !             curwin->w_buffer = savebuf;
  246. !             saved = ml_open();
  247. !             curbuf = buf;
  248. !             curwin->w_buffer = buf;
  249. !         }
  250. !         if (savebuf == NULL || saved == FAIL
  251. !                       || move_lines(buf, savebuf) == FAIL)
  252. !         {
  253. !             EMSG2(_("E461: Could not prepare for reloading \"%s\""),
  254. !                                 buf->b_fname);
  255. !             saved = FAIL;
  256. !         }
  257. !         }
  258. ! 
  259. !         if (saved == OK)
  260. !         {
  261. !         curbuf->b_flags |= BF_CHECK_RO;    /* check for RO again */
  262. ! #ifdef FEAT_AUTOCMD
  263. !         keep_filetype = TRUE;        /* don't detect 'filetype' */
  264. ! #endif
  265. !         if (readfile(buf->b_ffname, buf->b_fname, (linenr_T)0,
  266. !                 (linenr_T)0,
  267. !                 (linenr_T)MAXLNUM, &ea, READ_NEW) == FAIL)
  268. !         {
  269. !             EMSG2(_("E321: Could not reload \"%s\""), buf->b_fname);
  270. !             if (savebuf != NULL)
  271. !             {
  272. !             /* Put the text back from the save buffer.  First
  273. !              * delete any lines that readfile() added. */
  274. !             while (!bufempty())
  275. !                 if (ml_delete(curbuf->b_ml.ml_line_count, FALSE)
  276. !                                       == FAIL)
  277. !                 break;
  278. !             (void)move_lines(savebuf, buf);
  279. !             }
  280. !         }
  281. !         else
  282. !         {
  283. !             /* Mark the buffer as unmodified and free undo info. */
  284. !             unchanged(buf, TRUE);
  285. !             u_blockfree(buf);
  286. !             u_clearall(buf);
  287. !         }
  288.           }
  289.           vim_free(ea.cmd);
  290.   
  291. +         if (savebuf != NULL)
  292. +         wipe_buffer(savebuf, FALSE);
  293. + 
  294.   #ifdef FEAT_DIFF
  295.           /* Invalidate diff info if necessary. */
  296.           diff_invalidate();
  297. ***************
  298. *** 5313,5319 ****
  299.   #ifdef FEAT_AUTOCMD
  300.       /* restore curwin/curbuf and a few other things */
  301.       aucmd_restbuf(&aco);
  302. - 
  303.       /* Careful: autocommands may have made "buf" invalid! */
  304.   #else
  305.       curwin->w_buffer = save_curbuf;
  306. --- 5580,5585 ----
  307. ***************
  308. *** 6820,6828 ****
  309.   #endif
  310.   
  311.       /*
  312. !      * Quickly return if there are no autocommands for this event.
  313.        */
  314. !     if (first_autopat[(int)event] == NULL)
  315.       return retval;
  316.   
  317.       /*
  318. --- 7086,7095 ----
  319.   #endif
  320.   
  321.       /*
  322. !      * Quickly return if there are no autocommands for this event or
  323. !      * autocommands are blocked.
  324.        */
  325. !     if (first_autopat[(int)event] == NULL || autocmd_block > 0)
  326.       return retval;
  327.   
  328.       /*
  329. *** ../vim61.219/src/globals.h    Mon Oct  7 20:45:41 2002
  330. --- src/globals.h    Sun Oct 13 18:18:07 2002
  331. ***************
  332. *** 238,243 ****
  333. --- 238,244 ----
  334.   EXTERN int    autocmd_busy INIT(= FALSE);    /* Is apply_autocmds() busy? */
  335.   EXTERN int    autocmd_no_enter INIT(= FALSE); /* *Enter autocmds disabled */
  336.   EXTERN int    autocmd_no_leave INIT(= FALSE); /* *Leave autocmds disabled */
  337. + EXTERN int    autocmd_block INIT(= 0);    /* block all autocmds */
  338.   EXTERN int    modified_was_set;        /* did ":set modified" */
  339.   EXTERN int    did_filetype INIT(= FALSE);    /* FileType event found */
  340.   EXTERN int    keep_filetype INIT(= FALSE);    /* value for did_filetype when
  341. *** ../vim61.219/src/proto/buffer.pro    Fri Mar 22 21:41:05 2002
  342. --- src/proto/buffer.pro    Sun Oct 13 18:23:45 2002
  343. ***************
  344. *** 60,64 ****
  345.   void sign_mark_adjust __ARGS((linenr_T line1, linenr_T line2, long amount, long amount_after));
  346.   void set_buflisted __ARGS((int on));
  347.   int buf_contents_changed __ARGS((buf_T *buf));
  348. ! void wipe_buffer __ARGS((buf_T *buf));
  349.   /* vim: set ft=c : */
  350. --- 60,64 ----
  351.   void sign_mark_adjust __ARGS((linenr_T line1, linenr_T line2, long amount, long amount_after));
  352.   void set_buflisted __ARGS((int on));
  353.   int buf_contents_changed __ARGS((buf_T *buf));
  354. ! void wipe_buffer __ARGS((buf_T *buf, int aucmd));
  355.   /* vim: set ft=c : */
  356. *** ../vim61.219/src/version.c    Sun Oct 13 16:05:33 2002
  357. --- src/version.c    Sun Oct 13 18:45:52 2002
  358. ***************
  359. *** 608,609 ****
  360. --- 608,611 ----
  361.   {   /* Add new patch number below this line */
  362. + /**/
  363. +     220,
  364.   /**/
  365.  
  366. -- 
  367. hundred-and-one symptoms of being an internet addict:
  368. 211. Your husband leaves you...taking the computer with him and you
  369.      call him crying, and beg him to bring the computer back.
  370.  
  371.  ///  Bram Moolenaar -- Bram@moolenaar.net -- http://www.moolenaar.net  \\\
  372. ///          Creator of Vim - Vi IMproved -- http://www.vim.org          \\\
  373. \\\           Project leader for A-A-P -- http://www.a-a-p.org           ///
  374.  \\\ Lord Of The Rings helps Uganda - http://iccf-holland.org/lotr.html ///
  375.