[ 
https://issues.apache.org/jira/browse/LUCENE-8587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16717020#comment-16717020
 ] 

Karl Wright commented on LUCENE-8587:
-------------------------------------

Ok, you're right, this is more complex.  We cannot do without the testpoint and 
the in/out of set boolean, even though moving these around might produce 
exactly the same polygon.

On the other hand, blaming the serialization of the testpoint also seems odd 
since it's basically preserved from the constructor in whatever form was there. 
 Perhaps serialization/deserialization of the geopoint needs to change.  Let me 
examine that next.

> Self comparison bug in GeoComplexPolygon.equals method
> ------------------------------------------------------
>
>                 Key: LUCENE-8587
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8587
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: modules/spatial3d
>    Affects Versions: 7.1
>            Reporter: Zsolt Gyulavari
>            Assignee: Karl Wright
>            Priority: Major
>         Attachments: LUCENE-8587.patch
>
>
> GeoComplexPolygon.equals method checks equality with own testPoint1 field 
> instead of the other.testPoint1.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to