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.324 < prev    next >
Encoding:
Internet Message Format  |  2003-02-15  |  10.1 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.1.324
  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.324
  11. Problem:    Crash when dragging a vertical separator when <LeftMouse> is
  12.         remapped to jump to another window.
  13. Solution:   Pass the window pointer to the function doing the dragging instead
  14.         of always using the current window. (Daniel Elstner)
  15.         Also fix that starting a drag changes window focus.
  16. Files:        src/normal.c, src/proto/window.pro, src/ui.c, src/vim.h,
  17.         src/window.c
  18.  
  19.  
  20. *** ../vim61.323/src/normal.c    Sun Feb 16 19:41:35 2003
  21. --- src/normal.c    Sun Feb 16 17:10:13 2003
  22. ***************
  23. *** 2413,2418 ****
  24. --- 2419,2428 ----
  25.       oap->motion_type = MCHAR;
  26.       }
  27.   
  28. +     /* When releasing the button let jump_to_mouse() know. */
  29. +     if (!is_click && !is_drag)
  30. +     jump_flags |= MOUSE_RELEASED;
  31. + 
  32.       /*
  33.        * JUMP!
  34.        */
  35. *** ../vim61.323/src/proto/window.pro    Sun Jul 21 21:47:39 2002
  36. --- src/proto/window.pro    Thu Feb 13 20:30:06 2003
  37. ***************
  38. *** 24,31 ****
  39.   void win_setheight __ARGS((int height));
  40.   void win_setwidth __ARGS((int width));
  41.   void win_setminheight __ARGS((void));
  42. ! void win_drag_status_line __ARGS((int offset));
  43. ! void win_drag_vsep_line __ARGS((int offset));
  44.   void win_comp_scroll __ARGS((win_T *wp));
  45.   void command_height __ARGS((long old_p_ch));
  46.   void last_status __ARGS((int morewin));
  47. --- 24,31 ----
  48.   void win_setheight __ARGS((int height));
  49.   void win_setwidth __ARGS((int width));
  50.   void win_setminheight __ARGS((void));
  51. ! void win_drag_status_line __ARGS((win_T *dragwin, int offset));
  52. ! void win_drag_vsep_line __ARGS((win_T *dragwin, int offset));
  53.   void win_comp_scroll __ARGS((win_T *wp));
  54.   void command_height __ARGS((long old_p_ch));
  55.   void last_status __ARGS((int morewin));
  56. *** ../vim61.323/src/ui.c    Mon Jan  6 10:16:31 2003
  57. --- src/ui.c    Thu Feb 13 20:19:00 2003
  58. ***************
  59. *** 2207,2216 ****
  60.   #endif
  61.       static int    prev_row = -1;
  62.       static int    prev_col = -1;
  63. ! #ifdef FEAT_CMDWIN
  64. !     static int  drag_prev_win = FALSE;    /* dragging status line above
  65. !                        command-line window */
  66. ! #endif
  67.   
  68.       win_T    *wp, *old_curwin;
  69.       pos_T    old_cursor;
  70. --- 2208,2215 ----
  71.   #endif
  72.       static int    prev_row = -1;
  73.       static int    prev_col = -1;
  74. !     static win_T *dragwin = NULL;    /* window being dragged */
  75. !     static int    did_drag = FALSE;    /* drag was noticed */
  76.   
  77.       win_T    *wp, *old_curwin;
  78.       pos_T    old_cursor;
  79. ***************
  80. *** 2225,2230 ****
  81. --- 2224,2239 ----
  82.       mouse_past_bottom = FALSE;
  83.       mouse_past_eol = FALSE;
  84.   
  85. +     if (flags & MOUSE_RELEASED)
  86. +     {
  87. +     /* On button release we may change window focus if positioned on a
  88. +      * status line and no dragging happened. */
  89. +     if (dragwin != NULL && !did_drag)
  90. +         flags &= ~(MOUSE_FOCUS | MOUSE_DID_MOVE);
  91. +     dragwin = NULL;
  92. +     did_drag = FALSE;
  93. +     }
  94. + 
  95.       if ((flags & MOUSE_DID_MOVE)
  96.           && prev_row == mouse_row
  97.           && prev_col == mouse_col)
  98. ***************
  99. *** 2282,2297 ****
  100. --- 2291,2313 ----
  101.   #else
  102.       wp = firstwin;
  103.   #endif
  104. +     dragwin = NULL;
  105.       /*
  106.        * winpos and height may change in win_enter()!
  107.        */
  108.       if (row >= wp->w_height)        /* In (or below) status line */
  109. +     {
  110.           on_status_line = row - wp->w_height + 1;
  111. +         dragwin = wp;
  112. +     }
  113.       else
  114.           on_status_line = 0;
  115.   #ifdef FEAT_VERTSPLIT
  116.       if (col >= wp->w_width)        /* In separator line */
  117. +     {
  118.           on_sep_line = col - wp->w_width + 1;
  119. +         dragwin = wp;
  120. +     }
  121.       else
  122.           on_sep_line = 0;
  123.   
  124. ***************
  125. *** 2333,2351 ****
  126.       }
  127.   #endif
  128.   #ifdef FEAT_CMDWIN
  129. -     drag_prev_win = FALSE;
  130.       if (cmdwin_type != 0 && wp != curwin)
  131.       {
  132.           /* A click outside the command-line window: Use modeless
  133.            * selection if possible.  Allow dragging the status line of the
  134.            * window just above the command-line window. */
  135. !         if (wp == curwin->w_prev)
  136. !         drag_prev_win = TRUE;
  137. !         else
  138.           on_status_line = 0;
  139.           on_sep_line = 0;
  140.   # ifdef FEAT_CLIPBOARD
  141. !         if (drag_prev_win)
  142.           return IN_STATUS_LINE;
  143.           return IN_OTHER_WIN;
  144.   # else
  145. --- 2349,2369 ----
  146.       }
  147.   #endif
  148.   #ifdef FEAT_CMDWIN
  149.       if (cmdwin_type != 0 && wp != curwin)
  150.       {
  151.           /* A click outside the command-line window: Use modeless
  152.            * selection if possible.  Allow dragging the status line of the
  153.            * window just above the command-line window. */
  154. !         if (wp != curwin->w_prev)
  155. !         {
  156.           on_status_line = 0;
  157. +         dragwin = NULL;
  158. +         }
  159. + # ifdef FEAT_VERTSPLIT
  160.           on_sep_line = 0;
  161. + # endif
  162.   # ifdef FEAT_CLIPBOARD
  163. !         if (on_status_line)
  164.           return IN_STATUS_LINE;
  165.           return IN_OTHER_WIN;
  166.   # else
  167. ***************
  168. *** 2356,2362 ****
  169.       }
  170.   #endif
  171.   #ifdef FEAT_WINDOWS
  172. !     win_enter(wp, TRUE);            /* can make wp invalid! */
  173.   # ifdef CHECK_DOUBLE_CLICK
  174.       /* set topline, to be able to check for double click ourselves */
  175.       if (curwin != old_curwin)
  176. --- 2374,2384 ----
  177.       }
  178.   #endif
  179.   #ifdef FEAT_WINDOWS
  180. !     /* Only change window focus when not clicking on or dragging the
  181. !      * status line.  Do change focus when releasing the mouse button
  182. !      * (MOUSE_FOCUS was set above if we dragged first). */
  183. !     if (dragwin == NULL || (flags & MOUSE_RELEASED))
  184. !         win_enter(wp, TRUE);        /* can make wp invalid! */
  185.   # ifdef CHECK_DOUBLE_CLICK
  186.       /* set topline, to be able to check for double click ourselves */
  187.       if (curwin != old_curwin)
  188. ***************
  189. *** 2394,2422 ****
  190.       else if (on_status_line && which_button == MOUSE_LEFT)
  191.       {
  192.   #ifdef FEAT_WINDOWS
  193. !     wp = curwin;
  194. ! # ifdef FEAT_CMDWIN
  195. !     if (cmdwin_type != 0 && drag_prev_win && curwin->w_prev != NULL)
  196. !         /* Drag the status line of the window above the command-line
  197. !          * window. */
  198. !         curwin = curwin->w_prev;
  199. ! # endif
  200. !     /* Drag the status line */
  201. !     count = row - curwin->w_winrow - curwin->w_height + 1 - on_status_line;
  202. !     win_drag_status_line(count);
  203. ! 
  204. ! # ifdef FEAT_CMDWIN
  205. !     curwin = wp;
  206. ! # endif
  207.   #endif
  208.       return IN_STATUS_LINE;            /* Cursor didn't move */
  209.       }
  210.   #ifdef FEAT_VERTSPLIT
  211.       else if (on_sep_line && which_button == MOUSE_LEFT)
  212.       {
  213. !     /* Drag the separator column */
  214. !     count = col - curwin->w_wincol - curwin->w_width + 1 - on_sep_line;
  215. !     win_drag_vsep_line(count);
  216.       return IN_SEP_LINE;            /* Cursor didn't move */
  217.       }
  218.   #endif
  219. --- 2416,2443 ----
  220.       else if (on_status_line && which_button == MOUSE_LEFT)
  221.       {
  222.   #ifdef FEAT_WINDOWS
  223. !     if (dragwin != NULL)
  224. !         {
  225. !         /* Drag the status line */
  226. !         count = row - dragwin->w_winrow - dragwin->w_height + 1
  227. !                                  - on_status_line;
  228. !         win_drag_status_line(dragwin, count);
  229. !         did_drag |= count;
  230. !     }
  231.   #endif
  232.       return IN_STATUS_LINE;            /* Cursor didn't move */
  233.       }
  234.   #ifdef FEAT_VERTSPLIT
  235.       else if (on_sep_line && which_button == MOUSE_LEFT)
  236.       {
  237. !     if (dragwin != NULL)
  238. !     {
  239. !         /* Drag the separator column */
  240. !         count = col - dragwin->w_wincol - dragwin->w_width + 1
  241. !                                 - on_sep_line;
  242. !         win_drag_vsep_line(dragwin, count);
  243. !         did_drag |= count;
  244. !     }
  245.       return IN_SEP_LINE;            /* Cursor didn't move */
  246.       }
  247.   #endif
  248. *** ../vim61.323/src/vim.h    Sun Jan  5 22:14:46 2003
  249. --- src/vim.h    Sun Feb 16 15:57:09 2003
  250. ***************
  251. *** 1388,1393 ****
  252. --- 1392,1398 ----
  253.   # define MOUSE_DID_MOVE        0x04    /* only act when mouse has moved */
  254.   # define MOUSE_SETPOS        0x08    /* only set current mouse position */
  255.   # define MOUSE_MAY_STOP_VIS    0x10    /* may stop Visual mode */
  256. + # define MOUSE_RELEASED        0x20    /* button was released */
  257.   
  258.   # if defined(UNIX) && defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
  259.   #  define CHECK_DOUBLE_CLICK 1    /* Checking for double clicks ourselves. */
  260. *** ../vim61.323/src/window.c    Sun Feb  2 21:43:36 2003
  261. --- src/window.c    Thu Feb 13 20:22:57 2003
  262. ***************
  263. *** 3820,3829 ****
  264.   #ifdef FEAT_MOUSE
  265.   
  266.   /*
  267. !  * Status line of curwin is dragged "offset" lines down (negative is up).
  268.    */
  269.       void
  270. ! win_drag_status_line(offset)
  271.       int        offset;
  272.   {
  273.       frame_T    *curfr;
  274. --- 3827,3837 ----
  275.   #ifdef FEAT_MOUSE
  276.   
  277.   /*
  278. !  * Status line of dragwin is dragged "offset" lines down (negative is up).
  279.    */
  280.       void
  281. ! win_drag_status_line(dragwin, offset)
  282. !     win_T    *dragwin;
  283.       int        offset;
  284.   {
  285.       frame_T    *curfr;
  286. ***************
  287. *** 3833,3839 ****
  288.       int        up;    /* if TRUE, drag status line up, otherwise down */
  289.       int        n;
  290.   
  291. !     fr = curwin->w_frame;
  292.       curfr = fr;
  293.       if (fr != topframe)        /* more than one window */
  294.       {
  295. --- 3841,3847 ----
  296.       int        up;    /* if TRUE, drag status line up, otherwise down */
  297.       int        n;
  298.   
  299. !     fr = dragwin->w_frame;
  300.       curfr = fr;
  301.       if (fr != topframe)        /* more than one window */
  302.       {
  303. ***************
  304. *** 3950,3959 ****
  305.   
  306.   #ifdef FEAT_VERTSPLIT
  307.   /*
  308. !  * Separator line of curwin is dragged "offset" lines right (negative is left).
  309.    */
  310.       void
  311. ! win_drag_vsep_line(offset)
  312.       int        offset;
  313.   {
  314.       frame_T    *curfr;
  315. --- 3958,3968 ----
  316.   
  317.   #ifdef FEAT_VERTSPLIT
  318.   /*
  319. !  * Separator line of dragwin is dragged "offset" lines right (negative is left).
  320.    */
  321.       void
  322. ! win_drag_vsep_line(dragwin, offset)
  323. !     win_T    *dragwin;
  324.       int        offset;
  325.   {
  326.       frame_T    *curfr;
  327. ***************
  328. *** 3962,3968 ****
  329.       int        left;    /* if TRUE, drag separator line left, otherwise right */
  330.       int        n;
  331.   
  332. !     fr = curwin->w_frame;
  333.       if (fr == topframe)        /* only one window (cannot happe?) */
  334.       return;
  335.       curfr = fr;
  336. --- 3971,3977 ----
  337.       int        left;    /* if TRUE, drag separator line left, otherwise right */
  338.       int        n;
  339.   
  340. !     fr = dragwin->w_frame;
  341.       if (fr == topframe)        /* only one window (cannot happe?) */
  342.       return;
  343.       curfr = fr;
  344. *** ../vim61.323/src/version.c    Sun Feb 16 20:14:02 2003
  345. --- src/version.c    Sun Feb 16 20:26:22 2003
  346. ***************
  347. *** 608,609 ****
  348. --- 612,615 ----
  349.   {   /* Add new patch number below this line */
  350. + /**/
  351. +     324,
  352.   /**/
  353.  
  354. -- 
  355. hundred-and-one symptoms of being an internet addict:
  356. 263. You have more e-mail addresses than shorts.
  357.  
  358.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  359. ///          Creator of Vim - Vi IMproved -- http://www.Vim.org          \\\
  360. \\\              Project leader for A-A-P -- http://www.A-A-P.org        ///
  361.  \\\     Help AIDS victims, buy at Amazon -- http://ICCF.nl/click1.html ///
  362.