JTS is specifically case insensitive, wonder if they added that after our
fork.
I will report the bug for now: https://jira.codehaus.org/browse/GEOT-5021
--
Jody Garnett
On 12 February 2015 at 15:56, Andrea Aime <andrea.a...@geo-solutions.it>
wrote:
> Never seen lowercase or mixed case WKT, so I believe that's a fix, bit you
> can check the spec and/or JTS
>
> Cheers
> Andrea
> Il 12/feb/2015 19:38 "Jody Garnett" <jody.garn...@gmail.com> ha scritto:
>
>> I am hanging out at the code sprint with the GeoMesa project - and trying
>> to update to 13-SNAPSHOT as a sanity check before next weeks RC1.
>>
>> I have one solid fix in:
>>
>> * GEOT-5016 <https://jira.codehaus.org/browse/GEOT-5016>
>> https://github.com/geotools/geotools/pull/731 (Ensure JAI logging does
>> not prevent startup when JAI is not available)
>>
>> And one question:
>>
>> We noticed a change (between 11.2 --> 13-SNAPSHOT) to the parsing of WKT.
>> Previously test cases that used "Point(1 2)" and now it has to be "POINT(1
>> 2)".
>>
>> Was this case sensitivity expected? Or was it introduced accidentally
>> during the addition of Curve support...
>>
>> Let me check with a test case:
>>
>> WKTReader reader = new WKTReader(); // JTS Reader
>> assertNotNull( reader.read("POINT(1 2)") );
>> assertNotNull( reader.read("Point(1 2)") );
>>
>> reader = new WKTReader2(); // GeoTools Fork
>> assertNotNull( reader.read("POINT(1 2)") );
>> assertNotNull( reader.read("Point(1 2)") ); // <-- failure
>>
>>
>> --
>> Jody Garnett
>>
>>
>> ------------------------------------------------------------------------------
>> Dive into the World of Parallel Programming. The Go Parallel Website,
>> sponsored by Intel and developed in partnership with Slashdot Media, is
>> your
>> hub for all things parallel software development, from weekly thought
>> leadership blogs to news, videos, case studies, tutorials and more. Take a
>> look and join the conversation now. http://goparallel.sourceforge.net/
>> _______________________________________________
>> GeoTools-Devel mailing list
>> GeoTools-Devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>
>>
------------------------------------------------------------------------------
Dive into the World of Parallel Programming. The Go Parallel Website,
sponsored by Intel and developed in partnership with Slashdot Media, is your
hub for all things parallel software development, from weekly thought
leadership blogs to news, videos, case studies, tutorials and more. Take a
look and join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
