GNU bug report logs - #20976
25.0.50; [PATCH] Fix too high return in some cases of status_notify

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 20976 in the body.
You can then email your comments to 20976 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ian Kelling <ian <at> iankelling.org>
To: bug-gnu-emacs <at> gnu.org
Subject: 25.0.50; [PATCH] Fix too high return in some cases of status_notify
Date: Sat, 04 Jul 2015 02:13:32 -0700

* src/process.c (status_notify): Fix too high return in some
cases.
---
This was clearly intended to be part of the patch for debbugs:17647 (git
05d2821). The same pattern used elsewhere and wait_proc is an an unused
parameter added in that patch.

diff --git a/src/process.c b/src/process.c
index 3132f19..5272792 100644
--- a/src/process.c
+++ b/src/process.c
@@ -6714,7 +6714,8 @@ status_notify (struct Lisp_Process *deleting_process,
                 && p != deleting_process)
            {
              int nread = read_process_output (proc, p->infd);
-	      if (got_some_input < nread)
+	      if ((!wait_proc || wait_proc == XPROCESS (proc))
+                  && got_some_input < nread)
                got_some_input = nread;
              if (nread <= 0)
                break;
--
1.9.1

Message #10 received at 20976-done <at> debbugs.gnu.org (full text, mbox):

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Ian Kelling <ian <at> iankelling.org>
Cc: 20976-done <at> debbugs.gnu.org
Subject: Re:  25.0.50;
 [PATCH] Fix too high return in some cases of status_notify
Date: Sun, 05 Jul 2015 15:16:22 -0700

Thanks, I installed that.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.