os: WaitForSomething(): also try to flush out older buffer content

Always try to flush out anything we have, even for clients that had been stuck for a short while because we've tried to send out too much at once. If we'd only do it when someting *new* is in the queue, then those clients might get stuck for long time, until something else triggers the flush by mere accident. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
2026-10-26 01:59:57 +00:00 · 2025-08-07 17:09:16 +02:00 · 2025-08-07 17:09:16 +02:00 · 58db712376
commit 58db712376
parent 0f7e5a132a
1 changed files with 9 additions and 2 deletions
--- a/os/WaitFor.c
+++ b/os/WaitFor.c
@ -196,8 +196,15 @@ WaitForSomething(Bool are_ready)
            timeout = 0;

        BlockHandler(&timeout);
-        if (NewOutputPending)
-            FlushAllOutput();
+
+        /* Always try to flush out anything we have, even for clients that had
+           been stuck for a short while because we've tried to send out too
+           much at once. If we'd only do it when someting *new* is in the queue,
+           then those clients might get stuck for long time, until something else
+           triggers the flush by mere accident.
+        */
+        FlushAllOutput();
+
        /* keep this check close to select() call to minimize race */
        if (dispatchException)
            i = -1;