Re: file.c:FILE_ReadFileEx - fd is not overlapped

[Martin Wilck]

On Fri, 26 Apr 2002, Marcelo Welter wrote:
> I upgrade my wine sources today with CVS and more advantages are in this new code (i'm very happy) but
> with one printer i receive the messages:
> trace:file:fd is not overlapped

Allright, another bug of mine :(

I wasn't aware that FILE_ReadFileEx can be called on non-overlapped file
descriptors. The lines you deleted were basically a double-check, removing
them should do no harm.

So here we are:

--- files/file.c.orig	Fri Apr 26 18:48:40 2002
+++ files/file.c	Fri Apr 26 18:50:41 2002
@@ -1562,11 +1562,6 @@
         WARN ( "Couldn't get FD\n" );
         return FALSE;
     }
-    if ( ! (flags & FD_FLAG_OVERLAPPED) ) {
-        WARN ( "fd is not overlapped\n" );
-        SetLastError ( ERROR_INVALID_PARAMETER );
-        goto error;
-    }

ovp = (async_fileio*) HeapAlloc(GetProcessHeap(), 0, sizeof (async_fileio));
     if(!ovp)
@@ -1788,11 +1783,6 @@
     {
         TRACE( "Couldn't get FD\n" );
         return FALSE;
-    }
-    if ( ! (flags & FD_FLAG_OVERLAPPED) ) {
-        WARN ( "fd is not overlapped\n" );
-        SetLastError ( ERROR_INVALID_PARAMETER );
-        goto error;
     }

ovp = (async_fileio*) HeapAlloc(GetProcessHeap(), 0, sizeof (async_fileio));