RFR (XS) 8210347 : Combine subsequent calls to Set.contains() and Set.add()

Brian Burkhalter brian.burkhalter at oracle.com
Tue Sep 11 21:33:11 UTC 2018


Hello Ivan,

This looks all right to me.

Thanks,

Brian

On Sep 11, 2018, at 1:09 PM, Ivan Gerasimov <ivan.gerasimov at oracle.com> wrote:

> 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?



More information about the core-libs-dev mailing list