Current CVS psengine.cpp compilation fail under mingw32

[Gellule]

Compilation output:

C++ ./out/msysx86/debug/src/client/psengine.o
src/client/psengine.cpp: In member function `iMaterialWrapper*
   psEngine::LoadMaterial(const char*, const char*)\':
src/client/psengine.cpp:988: no matching function for call to `iLoader::
   LoadImageA(const char*&, int)\'

It seems that the #undef LoadImage work around in the first few lines of psengine.cpp (OS specific defines) does not work with me. It looks like there is a LoadImage->LoadImageA defined in one of the following included headers.

I traced down this definition as coming from both:
#include \"globals.h\"
#include \"psengine.h\"

Would it be a good solution to move the OS specific defines after the headers inclusions?

-Gellule

[dfryer]

Someone on IRC had this problem, I believe, and it was fixed by either forcing the block involving the #undef, or moving it above (or maybe below) some headers...  To summarize, I don\'t know what I\'m talking about but I seen it before!

[Gellule]

Thanks! That\'s exactly what I did. Move the #undef below the #include.

Beside that, when you find this kind of problem is posting here the best thing to do? Can you do more and propose a modification of code?

Other question: how well is the mingw32 build suported? Is it possible to help with it?

-Gellule

[acraig]

I believe that Jorrit ( the head of Crystal Space project ) uses mingw32 to build.  He is the only person that I know that uses it so it is supported but just not very well.  I will place any fixes that you have that can make the compile better for mingw32.

Posting here is good ( why this forum was created ) or contacting us on irc ( irc.freenode.net in #planeshift )  or contact me directly ( acraig@planeshift.it ) directly with any patches.

[Gellule]

Then, what\'s the most convenient way to submit a patch?
Any tool I should use?

[Gellule]

I think I found: I sent you an E-mail with the diff log.

I\'d like to send another one on compiling-mingw.txt. Should I go ahead ?

-Gellule