[PATCH 2/5] wait until all data from earlier test has been written in test_waittxempty

Wolfgang Walter wine at stwm.de
Thu Sep 5 04:32:52 CDT 2013


Am Donnerstag, 5. September 2013, 17:54:58 schrieb Dmitry Timoshkov:
> Wolfgang Walter <wine at stwm.de> wrote:
> > > > +    if (res || (!res && GetLastError() == ERROR_IO_PENDING))
> > > > +        /* if data has been sent: wait for termination */
> > > > +        Sleep(timeout);
> > > 
> > > I don't see such a problem with real COM-port and serial-USB cable
> > > under Windows or Linux here and under testbot VMs.
> > 
> > Wine does that here (vanilla). I added this so that the NEXT test does not
> > depend what wine exactly does.
> 
> When Wine behaviour differs from Windows one the test results need to be
> marked as todo_wine, and such places already have it.

I don't unterstand you. I didn't change the test case. It still above and is 
still marked with todo.

This change simply ensures that the next test is not distorted. If wine does 
not pass the test

"WriteFile on an overlapped handle without ovl structure should fail"

but instead sends the bytes we should wait long enough before starting the 
next test so that there are no remaining bytes in the tx buffer. Otherwise the 
next test will not test sending 17 bytes but much more.

Regards,
-- 
Wolfgang Walter
Studentenwerk München
Anstalt des öffentlichen Rechts
Abteilungsleiter IT
Leopoldstraße 15
80802 München



More information about the wine-devel mailing list