Wolfgang Walter wine@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.