Factual / geo

Clojure library for working with geohashes, polygons, and other world geometry
Eclipse Public License 1.0
303 stars 17 forks source link

Remove spatial4j clone workaround, and change JTS clone to copy #23

Closed willcohen closed 6 years ago

willcohen commented 6 years ago

JTS 1.15 deprecated clone for copy, so this library should switch accordingly. Additionally, spatial4j 0.7 includes a fix for the reported dateline workaround, so this removes that workaround and streamlines to-shape for Geometry.

willcohen commented 6 years ago

@worace do you have any questions about this PR? I can’t think of any other side effects that could come from this one and it seems like there’s already a test to check for the dateline workaround issue that still passes after this.

worace commented 6 years ago

Yeah I think this is good. Thanks for checking on the existing test case.