Am Donnerstag, 5. September 2013, 17:52:42 schrieb Dmitry Timoshkov: > Wolfgang Walter <w...@stwm.de> wrote: > > On Thursday 05 September 2013 10:42:40 you wrote: > > > Wolfgang Walter <w...@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. > > > > It is allowed that WaitCommEvent returns immediately with TRUE if there is > > an event pending, as far as I can see. > > WaitCommEvent always returns FALSE when device is opened in overlapped mode.
Not according to MSN documentation of WaitCommEvent(): "If the overlapped operation cannot be completed immediately, the function returns FALSE and the GetLastError function returns ERROR_IO_PENDING, indicating that the operation is executing in the background." Regards. -- Wolfgang Walter Studentenwerk München Anstalt des öffentlichen Rechts Abteilungsleiter IT Leopoldstraße 15 80802 München
