Re: [win32gui] Re: [perl-win32-gui-hackers] Re: Roadmap to a v1.02 release

[Robert M. <rm...@po...>]

Glenn Linderman wrote:

>   Hmmmm.  Having problems with compiling math.h, I tried this, but got
>
>     cl -c    -nologo -Gf -W3 -MD -Zi -DNDEBUG -O1 -DWIN32 -D_CONSOLE 
> -DNO_STRICT -DHAVE_DES_FCRYPT -DNO_HASH_SEED -DPERL_IMPLICIT_CONTEXT 
> -DPERL_IMPLICIT_SYS -DUSE_PERLIO -DPERL_MSVCRT_READFIX -MD -Zi 
> -DNDEBUG -O1    -DVERSION=\"1.01_03\"  -DXS_VERSION=\"1.01_03\" 
> "-IC:\Perl\lib\CORE"   GUI.cpp
> GUI.cpp
>
> *** Using Preserved Perl context.
>
> GUI.xs(200) : error C2065: '__TEMP_WORD' : undeclared identifier
> GUI.xs(288) : error C2065: 'IDI_DEFAULTICON' : undeclared identifier
> GUI.xs(328) : error C2065: 'IDC_HSPLIT' : undeclared identifier
> GUI.xs(331) : error C2065: 'IDC_VSPLIT' : undeclared identifier
> GUI.xs(4465) : error C2146: syntax error : missing ';' before 
> identifier 'cClrBits'
> GUI.xs(4465) : error C2065: 'cClrBits' : undeclared identifier
> GUI.xs(4517) : error C2146: syntax error : missing ')' before 
> identifier 'pbih'
> GUI.xs(4517) : error C2059: syntax error : ')'
> GUI.xs(4517) : error C2143: syntax error : missing ';' before '{'
> *** PACKAGE Win32::GUI::Menu...
> *** PACKAGE Win32::GUI::MenuButton...
> *** PACKAGE Win32::GUI::MenuItem...
> NMAKE : fatal error U1077: 'cl' : return code '0x2'
> Stop.
>
>
> I think that is an improvement, in the sense that the compilation 
> proceeded further than before.

I'm working on it.  I think I understand all the changes that Reini gave 
me as parts of the cygwi and his MSVC6 patches.  I'm reverting GUI.h to 
something much closer to what went out with the V1.0 release.

The problems all stem from the 'extern "C"' declarations around the perl 
includes - Reini changed the conditions under which they were/were not 
defined, resulting in them being added for builds usin 
-DPERL_IMPLICIT_CONTEXT, when they were not before;  he then added 
math.h outside this block, so that when math.h was included again by 
perl.h in had already been seen, and was not parsed in "C" context.

I think I've got it going - give me another 15 minutes and I'll check in 
an updated version.

Rob.