<html><head><style type="text/css"><!-- DIV {margin:0px;} --></style></head><body><div style="font-family:arial, helvetica, sans-serif;font-size:12pt"><DIV>Yes,</DIV>
<DIV><BR>That new version of inlines.cpp works absolutely fine for me.<BR><BR>Cheers<BR><BR>Steve<BR></DIV>
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: arial, helvetica, sans-serif">
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: times new roman, new york, times, serif">
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: arial, helvetica, sans-serif">
<DIV style="FONT-SIZE: 12pt; FONT-FAMILY: arial, helvetica, sans-serif"><BR>
<DIV style="FONT-SIZE: 10pt; FONT-FAMILY: arial, helvetica, sans-serif"><FONT face=Tahoma size=2>
<HR SIZE=1>
<B><SPAN style="FONT-WEIGHT: bold">From:</SPAN></B> Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk><BR><B><SPAN style="FONT-WEIGHT: bold">To:</SPAN></B> GEOS Development List <geos-devel@lists.osgeo.org><BR><B><SPAN style="FONT-WEIGHT: bold">Cc:</SPAN></B> S Knox <roxyknox@yahoo.co.uk><BR><B><SPAN style="FONT-WEIGHT: bold">Sent:</SPAN></B> Saturday, 15 November, 2008 14:24:44<BR><B><SPAN style="FONT-WEIGHT: bold">Subject:</SPAN></B> Re: [geos-devel] Fw: 3.0<BR></FONT><BR>On Sun, 2008-11-09 at 23:04 +0000, S Knox wrote:<BR>> Mark,<BR>> <BR>> after some deliberation, I think I've found the issue. Currently the<BR>> line reads:<BR>> <BR>> #if !defined(__MINGW32__) || defined(__MINGW32__) && !<BR>> defined(DLL_EXPORT) || <BR>> !defined(__CYGWIN__) || defined(__CYGWIN__) && !defined(DLL_EXPORT)<BR>> <BR>> However if it is changed to read:<BR>> <BR>> #if defined(__MINGW32__) &&
!defined(DLL_EXPORT) || <BR>> !defined(__CYGWIN__) || defined(__CYGWIN__) && !defined(DLL_EXPORT)<BR>> <BR>> It compiles correctly.<BR>> <BR>> Therefore I think this is a logic issue. If __CYGWIN__ is defined, then you would expect __MINGW32__ not to be defined, or vice versa.. So this condition will always be met even if you have either Cygwin or MinGW (at least that seems to be what's happening on my system).<BR><BR><BR>Hi Steve,<BR><BR>I had a look at this on a cygwin gcc-4.3 install and the issue is that<BR>libtool doesn't define DLL_EXPORT during the build process. The<BR>interesting aspect coming from this is that I think upon review that the<BR>MingW/Cygwin workarounds are effectively just silently removing the<BR>inlines from the build. However, I don't have enough experience with the<BR>linker to know if this is even possible in the Win32 world, or if it's a<BR>gcc thing :(<BR><BR>Please find attached 2 files: the first is
a new source/inlines.cpp<BR>which should replace the one in GEOS 3.0.3rc1. This works for me under<BR>Linux, MingW32 and cygwin, so please test and report back.<BR><BR>The second file is a patch to apply the above changes to SVN 3.0 branch<BR>and SVN trunk, assuming that everything works as it should. If the above<BR>testing goes to plan and these are applied to SVN just before release,<BR>please can someone verify that this doesn't affect the MSVC GEOS build?<BR>I'm reasonably confident they shouldn't, but it would be nice to make<BR>sure...<BR><BR><BR>ATB,<BR><BR>Mark.<BR><BR></DIV></DIV></DIV><BR></DIV></DIV></div><br>
</body></html>