<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 29, 2018 at 1:55 PM Martin Davis <<a href="mailto:mtnclimb@gmail.com">mtnclimb@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hmmm... where are you seeing this "large retrofit"?  </div></blockquote><div><br></div><div>Admittedly, this was pretty non-scientific, I just did a global search for it and found it in a number of classes more than would fit on one page... :) so I knew it wasn't just one new method in Geometry.java that delegated to equals() ... :) </div><div><br></div><div>Maybe it would all be fine'n'dandy. I'll look more closely tomorrow, if it's only a few dozen locations, I'll just do it in the interests of convergence.</div><div><br>P</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>The confusion over equals VS equalsTopo is indeed a thing in the Java world, and equalsTopo was created to encourage being unambiguous about the intended meaning.  But in JTS equals just delegates to equalsTopo, and equals is still used all over the place in JTS code (equalsTopo - not so much).  </div><div><br></div><div>Actually equals is not used at all in the core code (since it's highly inefficient for low-level use).  It's used in the unit tests, where performance doesn't matter as much.  But even there it's mostly only used either where it is being tested, or in a somewhat casual way that should probably be replaced.</div><div><br></div><div>So before opining about which way GEOS should go need to know more about where it's being used.</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 29, 2018 at 1:42 PM Paul Ramsey <<a href="mailto:pramsey@cleverelephant.ca" target="_blank">pramsey@cleverelephant.ca</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hrm... so, it seems at some point there was a large retrofit to flip equals() to equalsTopo(). I assume that's because (if I recall right) equals() has some special meaning/uses in the Java world, and having it with a different meaning made some things (maps, I think?) not work.<div><br></div><div>GEOS never made that flip, not having the same problem. On the one hand, making the flip would potentially make future porting simpler (method names would line up). On the other hand, it could be an error-prone and labour intensive process... what's your feeling?<br><br>P</div></div>
_______________________________________________<br>
geos-devel mailing list<br>
<a href="mailto:geos-devel@lists.osgeo.org" target="_blank">geos-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/geos-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/geos-devel</a></blockquote></div>
_______________________________________________<br>
geos-devel mailing list<br>
<a href="mailto:geos-devel@lists.osgeo.org" target="_blank">geos-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/geos-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/geos-devel</a></blockquote></div></div>