[geos-devel] Re: Thread Safe CAPI 2.0 patch online

Paul Ramsey pramsey at cleverelephant.ca
Tue Jan 6 15:46:10 EST 2009


I can review again, but a second pair of eyes would be preferred.

P

On Jan 6, 2009, at 11:01 AM, Chuck Thibert wrote:

> Hi again,
>
> With the holidays over, just want to push this up the stack again.  I
> want to be sure it's commit ready when I hand it off.  Any further
> comments?
>
> Thanks,
> Chuck
>
>
> On Fri, 2008-12-19 at 15:42 -0500, Chuck Thibert wrote:
>> Hi all,
>>
>> I've changed the patch based on Paul's comments.
>>
>> -- combined the two header files for easy maintenance. (This  
>> 'fixed' the
>> configure error by getting rid of the thread safe header.)
>> -- fixed the char * error.
>> -- changed the .c from geosTS_c.cpp to geos_ts_c.cpp
>> -- moved the test executables to tests/unit/capi directory.  This  
>> seemed
>> to cause patch a bit of problems so I created a little script that  
>> does
>> the svn move before applying the patch.
>>
>> All this is in a new 2.0 tar file attached to the ticket.
>>
>> Thanks again for the testing and comments Paul.
>>
>> Chuck
> _______________________________________________
> geos-devel mailing list
> geos-devel at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/geos-devel


--
Paul Ramsey
pramsey at cleverelephant.ca
+1 250 885 0632



More information about the geos-devel mailing list