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.472 < prev    next >
Encoding:
Internet Message Format  |  2003-04-17  |  6.3 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.1.472
  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.472
  11. Problem:    When there is an authentication error when connecting to the X
  12.         server Vim exits.
  13. Solution:   Use XSetIOErrorHandler() to catch the error and longjmp() to avoid
  14.         the exit.  Also do this in the main loop, so that when the X
  15.         server exits a Vim running in a console isn't killed.
  16. Files:        src/globals.h, src/main.c, src/os_unix.c
  17.  
  18.  
  19. *** ../vim61.471/src/globals.h    Thu Apr 10 22:38:13 2003
  20. --- src/globals.h    Wed Apr 16 22:19:13 2003
  21. ***************
  22. *** 542,547 ****
  23. --- 542,552 ----
  24.   EXTERN int    vr_lines_changed INIT(= 0); /* #Lines changed by "gR" so far */
  25.   #endif
  26.   
  27. + #if defined(FEAT_X11) && defined(FEAT_XCLIPBOARD)
  28. + /* argument to SETJMP() for handling X IO errors */
  29. + EXTERN JMP_BUF x_jump_env;
  30. + #endif
  31. + 
  32.   #if defined(HAVE_SETJMP_H)
  33.   /*
  34.    * Stuff for setjmp() and longjmp().
  35. *** ../vim61.471/src/main.c    Wed Apr 16 20:43:41 2003
  36. --- src/main.c    Fri Apr 18 14:23:46 2003
  37. ***************
  38. *** 1940,1945 ****
  39. --- 1940,1977 ----
  40.   {
  41.       oparg_T    oa;    /* operator arguments */
  42.   
  43. + #if defined(FEAT_X11) && defined(FEAT_XCLIPBOARD)
  44. +     /* Setup to catch a terminating error from the X server.  Just ignore
  45. +      * it, restore the state and continue.  This might not always work
  46. +      * properly, but at least we don't exit unexpectedly when the X server
  47. +      * exists while Vim is running in a console. */
  48. +     if (!cmdwin && SETJMP(x_jump_env))
  49. +     {
  50. +     State = NORMAL;
  51. + # ifdef FEAT_VISUAL
  52. +     VIsual_active = FALSE;
  53. + # endif
  54. +     got_int = TRUE;
  55. +     need_wait_return = FALSE;
  56. +     global_busy = FALSE;
  57. +     exmode_active = 0;
  58. +     skip_redraw = FALSE;
  59. +     RedrawingDisabled = 0;
  60. +     no_wait_return = 0;
  61. + # ifdef FEAT_EVAL
  62. +     emsg_skip = 0;
  63. + # endif
  64. +     emsg_off = 0;
  65. + # ifdef FEAT_MOUSE
  66. +     setmouse();
  67. + # endif
  68. +     settmode(TMODE_RAW);
  69. +     starttermcap();
  70. +     scroll_start();
  71. +     redraw_later_clear();
  72. +     }
  73. + #endif
  74. + 
  75.       clear_oparg(&oa);
  76.       while (!cmdwin
  77.   #ifdef FEAT_CMDWIN
  78. *** ../vim61.471/src/os_unix.c    Wed Apr 16 20:56:52 2003
  79. --- src/os_unix.c    Wed Apr 16 22:18:16 2003
  80. ***************
  81. *** 751,757 ****
  82.    * A simplistic version of setjmp() that only allows one level of using.
  83.    * Don't call twice before calling mch_endjmp()!.
  84.    * Usage:
  85. !  *    mch_startjmp()
  86.    *    if (SETJMP(lc_jump_env) != 0)
  87.    *    {
  88.    *        mch_didjmp();
  89. --- 751,757 ----
  90.    * A simplistic version of setjmp() that only allows one level of using.
  91.    * Don't call twice before calling mch_endjmp()!.
  92.    * Usage:
  93. !  *    mch_startjmp();
  94.    *    if (SETJMP(lc_jump_env) != 0)
  95.    *    {
  96.    *        mch_didjmp();
  97. ***************
  98. *** 1190,1195 ****
  99. --- 1190,1234 ----
  100.       return 0;
  101.   }
  102.   
  103. + #if defined(FEAT_X11) && defined(FEAT_XCLIPBOARD)
  104. + # if defined(HAVE_SETJMP_H)
  105. + /*
  106. +  * An X IO Error handler, used to catch error while opening the display.
  107. +  */
  108. + static int x_IOerror_check __ARGS((Display *dpy));
  109. + 
  110. + /* ARGSUSED */
  111. +     static int
  112. + x_IOerror_check(dpy)
  113. +     Display *dpy;
  114. + {
  115. +     /* This function should not return, it causes exit().  Longjump instead. */
  116. +     LONGJMP(lc_jump_env, 1);
  117. +     /*NOTREACHED*/
  118. + }
  119. + # endif
  120. + 
  121. + /*
  122. +  * An X IO Error handler, used to catch terminal errors.
  123. +  */
  124. + static int x_IOerror_handler __ARGS((Display *dpy));
  125. + 
  126. + /* ARGSUSED */
  127. +     static int
  128. + x_IOerror_handler(dpy)
  129. +     Display *dpy;
  130. + {
  131. +     xterm_dpy = NULL;
  132. +     x11_window = 0;
  133. +     x11_display = NULL;
  134. +     xterm_Shell = (Widget)0;
  135. + 
  136. +     /* This function should not return, it causes exit().  Longjump instead. */
  137. +     LONGJMP(x_jump_env, 1);
  138. +     /*NOTREACHED*/
  139. + }
  140. + #endif
  141. + 
  142.   /*
  143.    * Return TRUE when connection to the X server is desired.
  144.    */
  145. ***************
  146. *** 5203,5209 ****
  147. --- 5242,5251 ----
  148.        */
  149.       mch_startjmp();
  150.       if (SETJMP(lc_jump_env) != 0)
  151. +     {
  152.           success = FALSE;
  153. +         mch_didjmp();
  154. +     }
  155.       else
  156.   # endif
  157.       {
  158. ***************
  159. *** 5310,5315 ****
  160. --- 5352,5360 ----
  161.       if (app_context != NULL && xterm_Shell == (Widget)0)
  162.       {
  163.       int (*oldhandler)();
  164. + #if defined(HAVE_SETJMP_H)
  165. +     int (*oldIOhandler)();
  166. + #endif
  167.   # if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
  168.       struct timeval  start_tv;
  169.   
  170. ***************
  171. *** 5320,5328 ****
  172.       /* Ignore X errors while opening the display */
  173.       oldhandler = XSetErrorHandler(x_error_check);
  174.   
  175. !     xterm_dpy = XtOpenDisplay(app_context, xterm_display,
  176. !         "vim_xterm", "Vim_xterm", NULL, 0, &z, &strp);
  177.   
  178.       /* Now handle X errors normally. */
  179.       (void)XSetErrorHandler(oldhandler);
  180.   
  181. --- 5365,5393 ----
  182.       /* Ignore X errors while opening the display */
  183.       oldhandler = XSetErrorHandler(x_error_check);
  184.   
  185. ! #if defined(HAVE_SETJMP_H)
  186. !     /* Ignore X IO errors while opening the display */
  187. !     oldIOhandler = XSetIOErrorHandler(x_IOerror_check);
  188. !     mch_startjmp();
  189. !     if (SETJMP(lc_jump_env) != 0)
  190. !     {
  191. !         mch_didjmp();
  192. !         xterm_dpy = NULL;
  193. !     }
  194. !     else
  195. ! #endif
  196. !     {
  197. !         xterm_dpy = XtOpenDisplay(app_context, xterm_display,
  198. !             "vim_xterm", "Vim_xterm", NULL, 0, &z, &strp);
  199. ! #if defined(HAVE_SETJMP_H)
  200. !         mch_endjmp();
  201. ! #endif
  202. !     }
  203.   
  204. + #if defined(HAVE_SETJMP_H)
  205. +     /* Now handle X IO errors normally. */
  206. +     (void)XSetIOErrorHandler(oldIOhandler);
  207. + #endif
  208.       /* Now handle X errors normally. */
  209.       (void)XSetErrorHandler(oldhandler);
  210.   
  211. ***************
  212. *** 5332,5337 ****
  213. --- 5397,5405 ----
  214.           MSG(_("Opening the X display failed"));
  215.           return;
  216.       }
  217. + 
  218. +     /* Catch terminating error of the X server connection. */
  219. +     (void)XSetIOErrorHandler(x_IOerror_handler);
  220.   
  221.   # if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
  222.       if (p_verbose > 0)
  223. *** ../vim61.471/src/version.c    Fri Apr 18 12:32:44 2003
  224. --- src/version.c    Fri Apr 18 14:30:24 2003
  225. ***************
  226. *** 613,614 ****
  227. --- 613,616 ----
  228.   {   /* Add new patch number below this line */
  229. + /**/
  230. +     472,
  231.   /**/
  232.  
  233. -- 
  234. hundred-and-one symptoms of being an internet addict:
  235. 20. When looking at a pageful of someone else's links, you notice all of them
  236.     are already highlighted in purple.
  237.  
  238.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  239. ///          Creator of Vim - Vi IMproved -- http://www.Vim.org          \\\
  240. \\\              Project leader for A-A-P -- http://www.A-A-P.org        ///
  241.  \\\     Help AIDS victims, buy at Amazon -- http://ICCF.nl/click1.html ///
  242.