georust / geo

Geospatial primitives and algorithms for Rust
https://crates.io/crates/geo
Other
1.49k stars 194 forks source link

Add more tests for `line_segmentize` #1075

Closed Robinlovelace closed 10 months ago

Robinlovelace commented 10 months ago

As shown in https://github.com/JosiahParry/rsgeo/issues/28 there's an example where n is set to 4 but only 3 geometries are returned.

Not sure what the underlying cause is but thought it worth adding more tests to ensure it's not on the Rust side.

urschrei commented 10 months ago

Just checking: are these tests in addition to what's in #1078?

JosiahParry commented 10 months ago

I think the tests I've added in #1078 should cover the cases