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 / old / 5.5.017 < prev    next >
Encoding:
Internet Message Format  |  1999-10-03  |  4.4 KB
  1. To: vim-dev@vim.org
  2. Subject: patch 5.5.017
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. ------------
  6.  
  7. Patch 5.5.017
  8. Problem:    If an error occurs when closing the current window, Vim could get
  9.         stuck in the error handling.
  10. Solution:   Don't set curwin to NULL when closing the current window.
  11. Files:        src/window.c
  12.  
  13.  
  14. *** ../vim-5.5.16/src/window.c    Wed Sep 22 10:06:19 1999
  15. --- src/window.c    Mon Oct  4 11:29:09 1999
  16. ***************
  17. *** 17,22 ****
  18. --- 17,23 ----
  19.   static void win_exchange __ARGS((long));
  20.   static void win_rotate __ARGS((int, int));
  21.   static void win_goto __ARGS((WIN *wp));
  22. + static void win_enter_ext __ARGS((WIN *wp, int undo_sync, int no_curwin));
  23.   static void win_append __ARGS((WIN *, WIN *));
  24.   static void win_remove __ARGS((WIN *));
  25.   static void win_new_height __ARGS((WIN *, int));
  26. ***************
  27. *** 871,876 ****
  28. --- 872,878 ----
  29.   #ifdef AUTOCMD
  30.       int        other_buffer = FALSE;
  31.   #endif
  32. +     int        close_curwin = FALSE;
  33.   
  34.       if (lastwin == firstwin)
  35.       {
  36. ***************
  37. *** 932,944 ****
  38.   
  39.       win_new_height(wp, wp->w_height + win->w_height + win->w_status_height);
  40.       win_free(win);
  41.       if (win == curwin)
  42. !     curwin = NULL;
  43.       if (p_ea)
  44.       win_equal(wp, FALSE);
  45. !     if (curwin == NULL)
  46.       {
  47. !     win_enter(wp, FALSE);
  48.   #ifdef AUTOCMD
  49.       if (other_buffer)
  50.           /* careful: after this wp and win may be invalid! */
  51. --- 934,952 ----
  52.   
  53.       win_new_height(wp, wp->w_height + win->w_height + win->w_status_height);
  54.       win_free(win);
  55. + 
  56. +     /* Make sure curwin isn't invalid.  It can cause severe trouble when
  57. +      * printing an error message. */
  58.       if (win == curwin)
  59. !     {
  60. !     curwin = wp;
  61. !     close_curwin = TRUE;
  62. !     }
  63.       if (p_ea)
  64.       win_equal(wp, FALSE);
  65. !     if (close_curwin)
  66.       {
  67. !     win_enter_ext(wp, FALSE, TRUE);
  68.   #ifdef AUTOCMD
  69.       if (other_buffer)
  70.           /* careful: after this wp and win may be invalid! */
  71. ***************
  72. *** 1106,1127 ****
  73.   
  74.   /*
  75.    * Make window wp the current window.
  76. -  * Can be called when curwin == NULL, if curwin already has been closed.
  77.    */
  78.       void
  79.   win_enter(wp, undo_sync)
  80. !     WIN        *wp;
  81. !     int        undo_sync;
  82.   {
  83.   #ifdef AUTOCMD
  84.       int        other_buffer = FALSE;
  85.   #endif
  86.   
  87. !     if (wp == curwin)        /* nothing to do */
  88.       return;
  89.   
  90.   #ifdef AUTOCMD
  91. !     if (curwin != NULL)
  92.       {
  93.       /*
  94.        * Be careful: If autocommands delete the window, return now.
  95. --- 1114,1148 ----
  96.   
  97.   /*
  98.    * Make window wp the current window.
  99.    */
  100.       void
  101.   win_enter(wp, undo_sync)
  102. !     WIN        *wp;
  103. !     int        undo_sync;
  104. ! {
  105. !     win_enter_ext(wp, undo_sync, FALSE);
  106. ! }
  107. ! 
  108. ! /*
  109. !  * Make window wp the current window.
  110. !  * Can be called with "curwin_invalid" TRUE, which means that curwin has just
  111. !  * been closed and isn't valid.
  112. !  */
  113. !     static void
  114. ! win_enter_ext(wp, undo_sync, curwin_invalid)
  115. !     WIN        *wp;
  116. !     int        undo_sync;
  117. !     int        curwin_invalid;
  118.   {
  119.   #ifdef AUTOCMD
  120.       int        other_buffer = FALSE;
  121.   #endif
  122.   
  123. !     if (wp == curwin && !curwin_invalid)    /* nothing to do */
  124.       return;
  125.   
  126.   #ifdef AUTOCMD
  127. !     if (!curwin_invalid)
  128.       {
  129.       /*
  130.        * Be careful: If autocommands delete the window, return now.
  131. ***************
  132. *** 1145,1151 ****
  133.       /* may have to copy the buffer options when 'cpo' contains 'S' */
  134.       if (wp->w_buffer != curbuf)
  135.       buf_copy_options(curbuf, wp->w_buffer, BCO_ENTER | BCO_NOHELP);
  136. !     if (curwin != NULL)
  137.       {
  138.       prevwin = curwin;    /* remember for CTRL-W p */
  139.       curwin->w_redr_status = TRUE;
  140. --- 1166,1172 ----
  141.       /* may have to copy the buffer options when 'cpo' contains 'S' */
  142.       if (wp->w_buffer != curbuf)
  143.       buf_copy_options(curbuf, wp->w_buffer, BCO_ENTER | BCO_NOHELP);
  144. !     if (!curwin_invalid)
  145.       {
  146.       prevwin = curwin;    /* remember for CTRL-W p */
  147.       curwin->w_redr_status = TRUE;
  148. *** ../vim-5.5.16/src/version.c    Mon Oct  4 11:25:03 1999
  149. --- src/version.c    Mon Oct  4 11:24:44 1999
  150. ***************
  151. *** 420,420 ****
  152. --- 420,421 ----
  153.   {   /* Add new patch number below this line */
  154. +     17,
  155.  
  156. --
  157. ARTHUR: (as the MAN next to him is squashed by a sheep) Knights!  Run away!
  158.    Midst echoing shouts of "run away" the KNIGHTS retreat to cover with the odd
  159.    cow or goose hitting them still.  The KNIGHTS crouch down under cover.
  160.                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
  161.  
  162. --/-/---- Bram Moolenaar ---- Bram@moolenaar.net ---- Bram@vim.org ---\-\--
  163.   \ \    www.vim.org/iccf      www.moolenaar.net       www.vim.org    / /
  164.