[BRLTTY] Virtual driver is not reading the socket

Dave Mielke dave at mielke.cc
Thu May 12 08:43:30 EDT 2016


[quoted lines by Jaroslav Skarvada on 2016/05/12 at 05:52 -0400]

>Thanks, it works for me.

Good, I'll commit it.

>But isn't more clean to just partially revert the commit (change while/do to
>do/while), i.e. the previous patch?

No, there was a reason for the change. The order of the three actions is 
important, and the current order is correct.

1) Test the condition. This must occur before the first wait.
2) Test the timeout. This must occur before the next wait.
3) Wiat for something to do.

The bug was that the timeout was being tested even before the first wait.

-- 
Dave Mielke           | 2213 Fox Crescent | The Bible is the very Word of God.
Phone: 1-613-726-0014 | Ottawa, Ontario   | http://Mielke.cc/bible/
EMail: Dave at Mielke.cc | Canada  K2A 1H7   | http://FamilyRadio.org/


More information about the BRLTTY mailing list