apr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wr...@apache.org
Subject svn commit: r388614 - /apr/apr/branches/0.9.x/file_io/win32/seek.c
Date Fri, 24 Mar 2006 19:28:01 GMT
Author: wrowe
Date: Fri Mar 24 11:27:59 2006
New Revision: 388614

URL: http://svn.apache.org/viewcvs?rev=388614&view=rev
Log:

  Fix with additional notes for older PSDK winbase.h headers.

Backport 388287

Modified:
    apr/apr/branches/0.9.x/file_io/win32/seek.c

Modified: apr/apr/branches/0.9.x/file_io/win32/seek.c
URL: http://svn.apache.org/viewcvs/apr/apr/branches/0.9.x/file_io/win32/seek.c?rev=388614&r1=388613&r2=388614&view=diff
==============================================================================
--- apr/apr/branches/0.9.x/file_io/win32/seek.c (original)
+++ apr/apr/branches/0.9.x/file_io/win32/seek.c Fri Mar 24 11:27:59 2006
@@ -43,10 +43,11 @@
         DWORD offhi = (DWORD)(pos >> 32);
         rc = SetFilePointer(thefile->filehand, offlo, &offhi, FILE_BEGIN);
 
-        if (rc == INVALID_SET_FILE_POINTER)
-            /* A legal value, perhaps?  MSDN implies prior SetLastError isn't 
+        if (rc == (DWORD)-1)
+            /* A legal value, perhaps?  MSDN implies prior SetLastError isn't
              * needed, googling for SetLastError SetFilePointer seems
-             * to confirm this.
+             * to confirm this.  INVALID_SET_FILE_POINTER is too recently
+             * added for us to rely on it as a constant.
              */
             rv = apr_get_os_error();
         else



Mime
View raw message