A Buffer(0) that could switch to using GeometryFixer? - Githubissues

NetTopologySuite / NetTopologySuite

A .NET GIS solution that is fast and reliable for the .NET platform.

Other

1.44k stars 317 forks source link

A Buffer(0) that could switch to using GeometryFixer? #612

Open airbreather opened 2 years ago

airbreather commented 2 years ago

A coworker pointed me to this block:

https://github.com/NetTopologySuite/NetTopologySuite/blob/e4e6f3f4e1f19e57d0d2ea71183b04a8332a97e2/src/NetTopologySuite/Simplify/DouglasPeuckerSimplifier.cs#L192-L193

Seems like we could use GeometryFixer here, no? Probably after confirming with upstream...

DGuidi commented 2 years ago

I see the same code in jts codebase so probably it should be fixed in JTS code too

peetw commented 2 years ago

I guess this would also apply here?

https://github.com/NetTopologySuite/NetTopologySuite/blob/73679f4940d194bb2e074498798e6d2adb33b40e/src/NetTopologySuite/Densify/Densifier.cs#L194-L199