Hello Ivan, This looks all right to me.
Thanks, Brian On Sep 11, 2018, at 1:09 PM, Ivan Gerasimov <ivan.gerasi...@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?