RFR (XS) 8210347 : Combine subsequent calls to Set.contains() and Set.add()
Martin Buchholz
martinrb at google.com
Wed Sep 12 00:13:27 UTC 2018
A meeting of the minds?
http://hg.openjdk.java.net/jdk/jdk/rev/745ce8f5efc8
Looks good!
On Tue, Sep 11, 2018 at 1:09 PM, Ivan Gerasimov
<ivan.gerasimov at oracle.com> wrote:
> Hello!
>
> It's a pico-optimization.
>
> In a few places a call to Set.contains() can be merged with the following
> Set.add().
>
> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8210347
>
> WEBREV: http://cr.openjdk.java.net/~igerasim/8210347/00/webrev/
>
> Would you please help review the trivial fix?
>
> --
> With kind regards,
> Ivan Gerasimov
>
More information about the core-libs-dev
mailing list