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.093 < prev    next >
Encoding:
Internet Message Format  |  2002-06-08  |  7.4 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.1.093 (extra)
  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.093 (extra)
  11. Problem:    Mac and MS-Windows GUI: when scrolling while ":s" is working the
  12.         results can be messed up, because the cursor is moved.
  13. Solution:   Disallow direct scrolling when not waiting for a character.
  14. Files:        src/gui_mac.c, src/gui_w16.c, src/gui_w32.c, src/gui_w48.c
  15.  
  16.  
  17. *** ../vim61.092/src/gui_mac.c    Fri May 17 19:29:08 2002
  18. --- src/gui_mac.c    Sun Jun  9 19:56:56 2002
  19. ***************
  20. *** 174,179 ****
  21. --- 174,186 ----
  22.   static short dragRectEnbl;
  23.   static short dragRectControl;
  24.   
  25. + /* This variable is set when waiting for an event, which is the only moment
  26. +  * scrollbar dragging can be done directly.  It's not allowed while commands
  27. +  * are executed, because it may move the cursor and that may cause unexpected
  28. +  * problems (e.g., while ":s" is working).
  29. +  */
  30. + static int allow_scrollbar = FALSE;
  31. + 
  32.   /* Last mouse click caused contextual menu, (to provide proper release) */
  33.   #ifdef USE_CTRLCLICKMENU
  34.   static short clickIsPopup;
  35. ***************
  36. *** 1499,1504 ****
  37. --- 1506,1512 ----
  38.       scrollbar_T        *sb;
  39.       int            value, dragging;
  40.       ControlHandle    theControlToUse;
  41. +     int            dont_scroll_save = dont_scroll;
  42.   
  43.       theControlToUse = dragged_sb;
  44.   
  45. ***************
  46. *** 1511,1517 ****
  47. --- 1519,1529 ----
  48.       value = GetControl32BitValue (theControlToUse);
  49.       dragging = TRUE;
  50.   
  51. +     /* When "allow_scrollbar" is FALSE still need to remember the new
  52. +      * position, but don't actually scroll by setting "dont_scroll". */
  53. +     dont_scroll = !allow_scrollbar;
  54.       gui_drag_scrollbar(sb, value, dragging);
  55. +     dont_scroll = dont_scroll_save;
  56.   }
  57.   
  58.       pascal
  59. ***************
  60. *** 1524,1529 ****
  61. --- 1536,1542 ----
  62.       long    value;
  63.       int        page;
  64.       int        dragging = FALSE;
  65. +     int        dont_scroll_save = dont_scroll;
  66.   
  67.       sb = gui_find_scrollbar((long) GetControlReference (theControl));
  68.   
  69. ***************
  70. *** 1565,1571 ****
  71. --- 1578,1588 ----
  72.       else if (value < 0)
  73.       value = 0;*/
  74.   
  75. +     /* When "allow_scrollbar" is FALSE still need to remember the new
  76. +      * position, but don't actually scroll by setting "dont_scroll". */
  77. +     dont_scroll = !allow_scrollbar;
  78.       gui_drag_scrollbar(sb, value, dragging);
  79. +     dont_scroll = dont_scroll_save;
  80.   
  81.       out_flush();
  82.       gui_mch_set_scrollbar_thumb(sb, value, sb_info->size, sb_info->max);
  83. ***************
  84. *** 3468,3473 ****
  85. --- 3485,3492 ----
  86.   
  87.       entryTick = TickCount();
  88.   
  89. +     allow_scrollbar = TRUE;
  90. + 
  91.       do
  92.       {
  93.   /*    if (dragRectControl == kCreateEmpty)
  94. ***************
  95. *** 3483,3493 ****
  96.       }
  97.       /*
  98.        * Don't use gui_mch_update() because then we will spin-lock until a
  99. !      * char arrives, instead we use XtAppProcessEvent() to hang until an
  100.        * event arrives.  No need to check for input_buf_full because we are
  101. !      * returning as soon as it contains a single char.  Note that
  102. !      * XtAppNextEvent() may not be used because it will not return after a
  103. !      * timer event has arrived -- webb
  104.        */
  105.       /* TODO: reduce wtime accordinly???  */
  106.       if (wtime > -1)
  107. --- 3502,3510 ----
  108.       }
  109.       /*
  110.        * Don't use gui_mch_update() because then we will spin-lock until a
  111. !      * char arrives, instead we use WaitNextEventWrp() to hang until an
  112.        * event arrives.  No need to check for input_buf_full because we are
  113. !      * returning as soon as it contains a single char.
  114.        */
  115.       /* TODO: reduce wtime accordinly???  */
  116.       if (wtime > -1)
  117. ***************
  118. *** 3501,3512 ****
  119. --- 3518,3533 ----
  120.   #endif
  121.           gui_mac_handle_event (&event);
  122.           if (!vim_is_input_buf_empty())
  123. +         {
  124. +         allow_scrollbar = FALSE;
  125.           return OK;
  126. +         }
  127.       }
  128.       currentTick = TickCount();
  129.       }
  130.       while ((wtime == -1) || ((currentTick - entryTick) < 60*wtime/1000));
  131.   
  132. +     allow_scrollbar = FALSE;
  133.       return FAIL;
  134.   }
  135.   
  136. *** ../vim61.092/src/gui_w16.c    Sat May  4 22:23:07 2002
  137. --- src/gui_w16.c    Sun Jun  9 20:02:03 2002
  138. ***************
  139. *** 81,87 ****
  140.   #endif
  141.   
  142.   
  143. - 
  144.   #ifdef FEAT_MENU
  145.   /*
  146.    * Figure out how high the menu bar is at the moment.
  147. --- 81,86 ----
  148. ***************
  149. *** 149,154 ****
  150. --- 148,154 ----
  151.       int        nPos;
  152.   #endif
  153.       static UINT    prev_code = 0;   /* code of previous call */
  154. +     int        dont_scroll_save = dont_scroll;
  155.   
  156.       sb = gui_mswin_find_scrollbar(hwndCtl);
  157.       if (sb == NULL)
  158. ***************
  159. *** 239,246 ****
  160. --- 239,252 ----
  161.   
  162.       /* Don't let us be interrupted here by another message. */
  163.       s_busy_processing = TRUE;
  164. + 
  165. +     /* When "allow_scrollbar" is FALSE still need to remember the new
  166. +      * position, but don't actually scroll by setting "dont_scroll". */
  167. +     dont_scroll = !allow_scrollbar;
  168. + 
  169.       gui_drag_scrollbar(sb, val, dragging);
  170.       s_busy_processing = FALSE;
  171. +     dont_scroll = dont_scroll_save;
  172.   
  173.       return 0;
  174.   }
  175. *** ../vim61.092/src/gui_w32.c    Sat May  4 22:23:07 2002
  176. --- src/gui_w32.c    Sun Jun  9 20:04:41 2002
  177. ***************
  178. *** 365,370 ****
  179. --- 365,371 ----
  180.       int        nPos;
  181.   #endif
  182.       static UINT    prev_code = 0;   /* code of previous call */
  183. +     int        dont_scroll_save = dont_scroll;
  184.   
  185.       sb = gui_mswin_find_scrollbar(hwndCtl);
  186.       if (sb == NULL)
  187. ***************
  188. *** 457,464 ****
  189. --- 458,472 ----
  190.   
  191.       /* Don't let us be interrupted here by another message. */
  192.       s_busy_processing = TRUE;
  193. + 
  194. +     /* When "allow_scrollbar" is FALSE still need to remember the new
  195. +      * position, but don't actually scroll by setting "dont_scroll". */
  196. +     dont_scroll = !allow_scrollbar;
  197. + 
  198.       gui_drag_scrollbar(sb, val, dragging);
  199. + 
  200.       s_busy_processing = FALSE;
  201. +     dont_scroll = dont_scroll_save;
  202.   
  203.       return 0;
  204.   }
  205. *** ../vim61.092/src/gui_w48.c    Sat May  4 22:23:07 2002
  206. --- src/gui_w48.c    Sun Jun  9 20:03:16 2002
  207. ***************
  208. *** 166,171 ****
  209. --- 166,179 ----
  210.   
  211.   static int        s_need_activate = FALSE;
  212.   
  213. + /* This variable is set when waiting for an event, which is the only moment
  214. +  * scrollbar dragging can be done directly.  It's not allowed while commands
  215. +  * are executed, because it may move the cursor and that may cause unexpected
  216. +  * problems (e.g., while ":s" is working).
  217. +  */
  218. + static int allow_scrollbar = FALSE;
  219. + 
  220. + 
  221.   #ifdef GLOBAL_IME
  222.   # undef DefWindowProc
  223.   # define DefWindowProc(a, b, c, d) global_ime_DefWindowProc(a, b, c, d)
  224. ***************
  225. *** 1655,1660 ****
  226. --- 1663,1670 ----
  227.                                (TIMERPROC)_OnTimer);
  228.       }
  229.   
  230. +     allow_scrollbar = TRUE;
  231. + 
  232.       focus = gui.in_focus;
  233.       while (!s_timed_out)
  234.       {
  235. ***************
  236. *** 1697,1705 ****
  237. --- 1707,1717 ----
  238.               ;
  239.           s_wait_timer = 0;
  240.           }
  241. +         allow_scrollbar = FALSE;
  242.           return OK;
  243.       }
  244.       }
  245. +     allow_scrollbar = FALSE;
  246.       return FAIL;
  247.   }
  248.   
  249. *** ../vim61.092/src/version.c    Sun Jun  9 18:28:25 2002
  250. --- src/version.c    Sun Jun  9 20:31:49 2002
  251. ***************
  252. *** 608,609 ****
  253. --- 608,611 ----
  254.   {   /* Add new patch number below this line */
  255. + /**/
  256. +     93,
  257.   /**/
  258.  
  259. -- 
  260. hundred-and-one symptoms of being an internet addict:
  261. 146. You experience ACTUAL physical withdrawal symptoms when away
  262.      from your 'puter and the net.
  263.  
  264.  ///  Bram Moolenaar -- Bram@moolenaar.net -- http://www.moolenaar.net  \\\
  265. ///   Creator of Vim -- http://vim.sf.net -- ftp://ftp.vim.org/pub/vim   \\\
  266. \\\           Project leader for A-A-P -- http://www.a-a-p.org           ///
  267.  \\\  Help me helping AIDS orphans in Uganda - http://iccf-holland.org  ///
  268.