search

paleolimbot / s2geography

Simple features (ish) for s2geometry
Other
29 stars 3 forks source link

WKTReader parses illegal WKT without error #18

Open pramsey opened 4 months ago

pramsey commented 4 months ago

Parsing POINT(0 ) yields POINT(0 0).

Parsing `LINESTRING(0 0)' works. Though this is perhaps allowed as SFS says nothing about line validity. As long as other line algorithms don't choke on single-point inputs.

Parsing LINESTRING(0 ) yields LINESTRING(0 0).

paleolimbot commented 4 months ago

Thanks! I think the geoarrow-c parser handles this one (#16) (with apologies for not getting that PR cleaned up quickly!)