[wine-test-devel] Patch - cabinet.dll FDI Conformance Test

Dan Kegel dank at kegel.com
Fri Mar 11 09:21:39 CST 2005

Rizwan Kassim wrote:
> Signed-off-by: Rizwan Kassim <rizwank at geekymedia.com>
> Signed-off-by: Dan Kegel <dank at kegel.com>
> Signed-off-by: Aleksandr Liber <aliber at ucla.edu>

I didn't sign off on this version, for what it's worth.
Rizwan, please only put the Signed-off-by header
with my name on it if I've actually reviewed the patch
being submitted.

A few notes:

#define _S_IREAD  0x0100
#define _S_IWRITE 0x0080

I told you earlier that's not a good idea; you should get those
from a standard windows header.

#ifndef WIN_ENV
#include <msvcrt/fcntl.h>
#include <fcntl.h>

I told you earlier that plain old <fcntl.h> should do,
and it works fine here with msvc and winegcc.
You shouldn't ever need to include <msvcrt/fcntl.h>.

You decompress complex.cab but don't verify the results;
that seems bad.

Can you address those issues, please?
- Dan

Trying to get a job as a c++ developer?  See http://kegel.com/academy/getting-hired.html

More information about the wine-devel mailing list