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.0.242 < prev    next >
Encoding:
Internet Message Format  |  2002-02-20  |  2.9 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.0.242
  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.0.242
  11. Problem:    GUI: On a system with an Exceed X server sometimes get a "Bad
  12.         Window" error. (Tommi Maekitalo)
  13. Solution:   When forking, use a pipe to wait in the parent for the child to
  14.         have done the setsid() call.
  15. Files:        src/gui.c
  16.  
  17.  
  18. *** ../vim60.241/src/gui.c    Mon Feb 11 15:45:47 2002
  19. --- src/gui.c    Thu Feb 21 12:11:52 2002
  20. ***************
  21. *** 107,119 ****
  22.        */
  23.       if (gui.in_use && dofork)
  24.       {
  25.       pid = fork();
  26.       if (pid > 0)        /* Parent */
  27.       {
  28.           /* Give the child some time to do the setsid(), otherwise the
  29.            * exit() may kill the child too (when starting gvim from inside a
  30.            * gvim). */
  31. !         ui_delay(100L, TRUE);
  32.   
  33.           /*
  34.            * The parent must skip the normal exit() processing, the child
  35. --- 107,136 ----
  36.        */
  37.       if (gui.in_use && dofork)
  38.       {
  39. +     int    pipefd[2];    /* pipe between parent and child */
  40. +     int    pipe_error;
  41. +     char    dummy;
  42. + 
  43. +     /* Setup a pipe between the child and the parent, so that the parent
  44. +      * knows when the child has done the setsid() call and is allowed to
  45. +      * exit. */
  46. +     pipe_error = (pipe(pipefd) < 0);
  47.       pid = fork();
  48.       if (pid > 0)        /* Parent */
  49.       {
  50.           /* Give the child some time to do the setsid(), otherwise the
  51.            * exit() may kill the child too (when starting gvim from inside a
  52.            * gvim). */
  53. !         if (pipe_error)
  54. !         ui_delay(300L, TRUE);
  55. !         else
  56. !         {
  57. !         /* The read returns when the child closes the pipe (or when
  58. !          * the child dies for some reason). */
  59. !         close(pipefd[1]);
  60. !         (void)read(pipefd[0], &dummy, (size_t)1);
  61. !         close(pipefd[0]);
  62. !         }
  63.   
  64.           /*
  65.            * The parent must skip the normal exit() processing, the child
  66. ***************
  67. *** 134,139 ****
  68. --- 151,161 ----
  69.           (void)setpgid(0, 0);
  70.   #  endif
  71.   # endif
  72. +     if (!pipe_error)
  73. +     {
  74. +         close(pipefd[0]);
  75. +         close(pipefd[1]);
  76. +     }
  77.       }
  78.   #else
  79.   # if defined(__QNXNTO__)
  80. *** ../vim60.241/src/version.c    Wed Feb 20 22:07:15 2002
  81. --- src/version.c    Thu Feb 21 12:29:54 2002
  82. ***************
  83. *** 608,609 ****
  84. --- 608,611 ----
  85.   {   /* Add new patch number below this line */
  86. + /**/
  87. +     242,
  88.   /**/
  89.  
  90. -- 
  91. I once paid $12 to peer at the box that held King Tutankhamen's little
  92. bandage-covered midget corpse at the De Young Museum in San Francisco.  I
  93. remember thinking how pleased he'd be about the way things turned out in his
  94. afterlife.
  95.                 (Scott Adams - The Dilbert principle)
  96.  
  97.  ///  Bram Moolenaar -- Bram@moolenaar.net -- http://www.moolenaar.net  \\\
  98. ///   Creator of Vim -- http://vim.sf.net -- ftp://ftp.vim.org/pub/vim   \\\
  99. \\\           Project leader for A-A-P -- http://www.a-a-p.org           ///
  100.  \\\  Help me helping AIDS orphans in Uganda - http://iccf-holland.org  ///
  101.