<div dir="ltr">Hmmm... where are you seeing this "large retrofit"?  <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">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>