Patch : fixme:msvcrt:MSVCRT__sopen : pmode 0x01a0 ignored

[Erik de Castro Lopo]

Hi all,

My interest in Wine is mainly because it allows me run the test suite
for my projects when I use the Linux -> windows MinGW compiler.

Currently, the programs in my test suite spit out a lot of fixme
warnings like:

    fixme:msvcrt:MSVCRT__sopen : pmode 0x01a0 ignored

I've tracked this down in the code to the function in
dlls/msvcrt/file.c.

The patch below removes this fixme warning. I beleive that simply removing
the warning is valid in this case because the variable being warned about,
pmode, is not used after the warning. In fact, the only use of this variable
is the FIXME or WARN messages.

Cheers,
Erik

--- dlls/msvcrt/file.c~ 2008-03-25 23:42:47.000000000 +1100
+++ dlls/msvcrt/file.c  2008-03-25 23:44:14.000000000 +1100
@@ -1442,11 +1442,6 @@
       pmode = va_arg(ap, int);
     va_end(ap);
 
-    if(pmode & ~(MSVCRT__S_IREAD | MSVCRT__S_IWRITE))
-      FIXME(": pmode 0x%04x ignored\n", pmode);
-    else
-      WARN(": pmode 0x%04x ignored\n", pmode);
-
     if (oflags & MSVCRT__O_EXCL)
       creation = CREATE_NEW;
     else if (oflags & MSVCRT__O_TRUNC)


-- 
-----------------------------------------------------------------
Erik de Castro Lopo
-----------------------------------------------------------------
Only people who suck have a problem with elitism.