Re: gnuplot 4.6.6 (Prelease, cvs) for Windows, and Cygwin (32 and 64 bit)

[Bastian M. <bma...@we...>]

Am 04.07.2014 11:26, schrieb Tatsuro MATSUOKA:
> ----- Original Message -----
>
>> T atsuro, the backport is (or at least should be) complete. The massive
>> number of ifdefs can be avoided since Win32 and Win64 do share the same
>> API. If it does not compile using Mingw64 in the current state (without
>> additional patches), please let me know.
>>
>>     Bastian
>
>
> I deleted all check out source and re-check out again.
> And I tried again without Allin's patch.
>
>
> x86_64-w64-mingw32-gcc -c -I/c/Programs/gplibs64/include -fno-keep-inline-dllexport -D__USE_MINGW_ANSI_STDIO -O2 -pipe -I. -I../../src -D_Windows -DHAVE_CONFIG_H -DPIPES -DWGP_CONSOLE -DCONSOLE_SWITCH_CP -DGNUPLOT_SHARE_DIR=\"share\" -DDEVELOPMENT_VERSION -DUSE_MOUSE=1 -DWIN_IPC -DWITH_HTML_HELP -I/c/PROGRA~2/HELPWO~1/include -DHAVE_LIBGD -DHAVE_LIBPNG -DHAVE_LIBGD -DHAVE_GD_H -DHAVE_GD_GIF -DGIF_ANIMATION -DHAVE_GD_PNG -DHAVE_GD_JPEG -DHAVE_GD_TTF -DHAVE_CAIROPDF -DWXWIDGETS -DHAVE_LUA -DHAVE_ICONV  -MMD -MT 'wgraph.$(O)' -MF wgraph.d -o wgraph.co ../../src/win/wgraph.c
> ../../src/win/wgraph.c: In function 'GraphInit':
> ../../src/win/wgraph.c:440:36: error: 'GCL_HICON' undeclared (first use in this function)
>     SetClassLongPtr(lpgw->hWndGraph, GCL_HICON,
>                                      ^
> ../../src/win/wgraph.c:440:36: note: each undeclared identifier is reported only once for each function it appears in
> ../../src/win/wgraph.c:441:4: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
>      (LONG) LoadIcon(lpgw->hInstance, "GRPICON"));
>      ^
> make[1]: *** [wgraph.co] Error 1
> make[1]: Leaving directory `/e/usr/Tatsu/mingw64work/gnuplot/branch-4-6-stable/gnuplot46/config/mingw'
> make: *** [console] Error 2
>
> It seems that the current source cannot be built without Allin's patch using MinGW_64 at least for me.
> (MinGW build, thread model win32, exception seh, Ver. 4.8.3-release 0 )
>
>
> Tatsuro
>

Thanks for the report. Both errors should now be fixed in CVS.

   Bastian