diff options
author | Bram Moolenaar <Bram@vim.org> | 2008-04-01 12:31:14 +0000 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2008-04-01 12:31:14 +0000 |
commit | e9b2884c088558d6b00b36f78c4e86a8ac2893a1 (patch) | |
tree | deeaeb98d78e8362e0782e3afeb2f32c146e01f3 | |
parent | 525145642bd362b93e1dc02d44f33b0bbf70c010 (diff) | |
download | vim-git-e9b2884c088558d6b00b36f78c4e86a8ac2893a1.tar.gz |
updated for version 7.1-288v7.1.288
-rw-r--r-- | src/if_cscope.c | 39 | ||||
-rw-r--r-- | src/version.c | 2 |
2 files changed, 39 insertions, 2 deletions
diff --git a/src/if_cscope.c b/src/if_cscope.c index b2d39cf42..376e2a69c 100644 --- a/src/if_cscope.c +++ b/src/if_cscope.c @@ -2130,6 +2130,7 @@ cs_release_csp(i, freefnpp) } #if defined(UNIX) { + int waitpid_errno; int pstat; pid_t pid; @@ -2145,6 +2146,7 @@ cs_release_csp(i, freefnpp) /* Block until cscope exits or until timer expires */ pid = waitpid(csinfo[i].pid, &pstat, 0); + waitpid_errno = errno; /* cancel pending alarm if still there and restore signal */ alarm(0); @@ -2158,6 +2160,7 @@ cs_release_csp(i, freefnpp) for (waited = 0; waited < 40; ++waited) { pid = waitpid(csinfo[i].pid, &pstat, WNOHANG); + waitpid_errno = errno; if (pid != 0) break; /* break unless the process is still running */ mch_delay(50, FALSE); /* sleep 50 ms */ @@ -2170,8 +2173,40 @@ cs_release_csp(i, freefnpp) */ if (pid < 0 && csinfo[i].pid > 1) { - kill(csinfo[i].pid, SIGKILL); - (void)waitpid(csinfo[i].pid, &pstat, 0); +# ifdef ECHILD + int alive = TRUE; + + if (waitpid_errno == ECHILD) + { + /* + * When using 'vim -g', vim is forked and cscope process is + * no longer a child process but a sibling. So waitpid() + * fails with errno being ECHILD (No child processes). + * Don't send SIGKILL to cscope immediately but wait + * (polling) for it to exit normally as result of sending + * the "q" command, hence giving it a chance to clean up + * its temporary files. + */ + int waited; + + sleep(0); + for (waited = 0; waited < 40; ++waited) + { + /* Check whether cscope process is still alive */ + if (kill(csinfo[i].pid, 0) != 0) + { + alive = FALSE; /* cscope process no longer exists */ + break; + } + mch_delay(50, FALSE); /* sleep 50ms */ + } + } + if (alive) +# endif + { + kill(csinfo[i].pid, SIGKILL); + (void)waitpid(csinfo[i].pid, &pstat, 0); + } } } #else /* !UNIX */ diff --git a/src/version.c b/src/version.c index e105f140e..c2f7e97d4 100644 --- a/src/version.c +++ b/src/version.c @@ -667,6 +667,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 288, +/**/ 287, /**/ 286, |