bug#23864: 25.1.50; Poor network responsivenes using open-network-stream

[Christer Ekholm <che@chrekh.se>]

Eli Zaretskii writes:
 > 
 > Isn't that strange?  The commit made exactly the change you proposed,
 > didn't it?

Apparently not. But it was not easy to spot the difference. In my
try the test was added before the "if (WAIT == timeout)".

 > 
 > > I'll examine further and report back here if I figure something out.

I have tested some more now, and a simple change from "else if" to
just "if" does help.

diff --git a/src/process.c b/src/process.c
index 376e872..031967e 100644
--- a/src/process.c
+++ b/src/process.c
@@ -5275,7 +5275,7 @@ wait_reading_process_output (intmax_t time_limit, int nsecs, int read_kbd,
            break;
          else if (wait == TIMEOUT)
            cmp_time = end_time, have_cmp_time = true;
-         else if (!process_skipped && got_some_output > 0
+         if (!process_skipped && got_some_output > 0
                   && (timeout.tv_sec > 0 || timeout.tv_nsec > 0))
            {
              if (!timespec_valid_p (got_output_end_time))

-- 
 Christer