MSCMS: new dll

[Hans Leidekker]

On Sunday 19 September 2004 06:06, Dmitry Timoshkov wrote:

> > LittleCMS library. I had to resort to some rather ugly preprocessor
> > trickery to be able to include lcms.h, since it happens to define basic
> > Windows types when not compiled on Windows ;-(.
> 
> Then perhaps it's better to make lcms.h think that it's used in a Windows
> environment (define WIN32 or whatever it wants to see before inclusion).
> That will help to completely avoid all the ugliness with redefined stuff.

Yes, I tried that option but it has even bigger problems. First, the header
doesn't have such a define! It defaults to building on non-windows platforms
and requires you to edit the file if you want a Windows build.

Furthermore, when you do so it immediately includes windows.h, which is not
allowed within Wine.

> You have to use FIXMEs not TRACEs above to clearly indicate the status
> of the APIs.

Yes, thanks for pointing that out. I'll send an updated patch.

 -Hans