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

[Paul Ramsey]

OK cool kids, I'm just going to review it again myself and then apply  
it if it looks OK to *me*... (and we all know where *that* leads...:)

P.

On Jan 6, 2009, at 12:46 PM, Paul Ramsey wrote:

> 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
>


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