On 03/11/13 21:23, Alexandre Julliard wrote:
Mathis Beer <default_357-line(a)yahoo.de> writes:
diff --git a/dlls/ntdll/virtual.c b/dlls/ntdll/virtual.c index a99bca4..e4f2bb6 100644 --- a/dlls/ntdll/virtual.c +++ b/dlls/ntdll/virtual.c @@ -62,6 +62,10 @@ WINE_DECLARE_DEBUG_CHANNEL(module); #define MS_SYNC 0 #endif
+#ifndef MS_ASYNC +#define MS_ASYNC 1 +#endif That's questionable, there's no reason the value would be 1. We defined MS_SYNC to 0 on the theory that it's better to pass 0 than making up values, but probably not calling the function at all would be preferable.
#define MS_ASYNC 1 was taken from my /usr/include/bits/mman.h . The msync manpage does not define the sync behavior of the function when neither MS_SYNC nor MS_ASYNC are set; it probably defaults to safe. In any case, since MS_SYNC is an already-defined constant (four), defining it to 0 is highly misleading. I thought it was bad style to import system headers in Wine code. :) In any case thanks for the response, I'll submit an improved patch.