RFR (M): JDK-6394757: AbstractSet.removeAll semantics are surprisingly dependent on relative sizes

Pavel Rappo pavel.rappo at oracle.com
Tue May 7 15:21:54 UTC 2019


Stuart,

That looks good.

A tiny note on your FIXME comment [1]:

  68     // FIXME: the first paragraph below is copied from Set.equals, because
  69     // {@inheritDoc} will include text from Object.equals.

You might want to check if it is the same as [2] or [3]. If it is the same,
consider adding a reference to the appropriate issue to that FIXME comment for
future maintainers. It looks like there are some problems when inheriting
documentation from methods defined in java.lang.Object.

Thanks,
-Pavel

-------------------------------------------------------------------------------
[1] http://cr.openjdk.java.net/~smarks/reviews/6394757/webrev.0/src/java.base/share/classes/java/util/AbstractSet.java.sdiff.html
[2] https://bugs.openjdk.java.net/browse/JDK-8144034
[3] https://bugs.openjdk.java.net/browse/JDK-6376959



More information about the core-libs-dev mailing list