Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

JtsShapeFactory.makeShapeFromGeometry IllegalStateException on empty Point #163

Closed
abrokenjester opened this issue Aug 7, 2018 · 0 comments
Closed

JtsShapeFactory.makeShapeFromGeometry IllegalStateException on empty Point #163

abrokenjester opened this issue Aug 7, 2018 · 0 comments
Labels
bug enhancement

Comments

@abrokenjester
Copy link
Contributor

When using JtsShapeFactory.makeShapeFromGeometry on a Geometry that represents an empty Point, an IllegalStateException is thrown:

java.lang.IllegalStateException: getX called on empty Point
	at org.locationtech.jts.geom.Point.getX(Point.java:102)
	at org.locationtech.spatial4j.shape.jts.JtsShapeFactory.makeShapeFromGeometry(JtsShapeFactory.java:478)

Perhaps it could properly convert an empty JTS Geometry to a corresponding empty Shape.
@abrokenjester abrokenjester mentioned this issue Aug 7, 2018
Merged
dsmiley added a commit to dsmiley/spatial4j that referenced this issue Mar 8, 2020
@dsmiley
EMPTY points in JTS are now convertible to a Shape instead of throwin…
98a32d1 
…g an exception.

Fixes locationtech#163
@dsmiley dsmiley mentioned this issue Mar 8, 2020
Closed
abrokenjester pushed a commit to abrokenjester/spatial4j that referenced this issue Mar 12, 2020
@dsmiley @abrokenjester
EMPTY points in JTS are now convertible to a Shape instead of throwin…
5df4234 
…g an exception.

Fixes locationtech#163
@barthanssens barthanssens mentioned this issue Dec 16, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

EMPTY points in JTS are now convertible to a Shape instead of throwin… dsmiley/spatial4j
2 participants
@dsmiley @abrokenjester