[PATCH 5/5] kernel32/tests: remove several todos in test_waittxempty

Dmitry Timoshkov dmitry at baikal.ru
Wed Sep 4 20:42:40 CDT 2013


Wolfgang Walter <wine at stwm.de> wrote:

> -        ok(!res && GetLastError() == ERROR_IO_PENDING, "%d: WaitCommEvent error %d\n", i, GetLastError());
> +        ok(res || (!res && GetLastError() == ERROR_IO_PENDING), "%d: WaitCommEvent error %d\n", i, GetLastError());

This change looks spurious and unrelated. Also todos must be removed
in the patch that fixes appropriate behaviour.

-- 
Dmitry.



More information about the wine-devel mailing list