fhandler_tty_slave::tcflush() in fhandler_tty.cc
Dave Korn
dave.korn@artimi.com
Fri Aug 5 14:17:00 GMT 2005
----Original Message----
>From: Vaclav Haisman
>Sent: 05 August 2005 14:01
> fhandler_tty_slave::tcflush() is IMHO still wrong. The result of
> comparison is bool and bool converted to int is either 1 or 0. The
> following patch should cure it.
Just to enlarge upon that, the problem is not just that it's returning a
zero-or-one when it should be returning a zero-or-minus-one result, but that
the logical sense is inverted too: when flushing input, the test means that
it used to return zero for failure and non-zero for success. Your patch
looks good to me.
cheers,
DaveK
--
Can't think of a witty .sigline today....
More information about the Cygwin-patches
mailing list