# HG changeset patch
# User igerasim
# Date 1536702705 25200
# Node ID 4ffb0a33f265a7ce28cea8443730d68396ed5517
# Parent  ebd5b1ad971a08cba6f56b02211b3f17047afa6e
8210347: Combine subsequent calls to Set.contains() and Set.add()
Reviewed-by: smarks, bpb

diff -r ebd5b1ad971a -r 4ffb0a33f265 src/java.base/share/classes/java/lang/ModuleLayer.java
--- a/src/java.base/share/classes/java/lang/ModuleLayer.java	Tue Sep 11 13:54:34 2018 -0700
+++ b/src/java.base/share/classes/java/lang/ModuleLayer.java	Tue Sep 11 14:51:45 2018 -0700
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014, 2017, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2014, 2018, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -790,8 +790,7 @@
             // push in reverse order
             for (int i = layer.parents.size() - 1; i >= 0; i--) {
                 ModuleLayer parent = layer.parents.get(i);
-                if (!visited.contains(parent)) {
-                    visited.add(parent);
+                if (visited.add(parent)) {
                     stack.push(parent);
                 }
             }
diff -r ebd5b1ad971a -r 4ffb0a33f265 src/java.base/share/classes/java/lang/module/Configuration.java
--- a/src/java.base/share/classes/java/lang/module/Configuration.java	Tue Sep 11 13:54:34 2018 -0700
+++ b/src/java.base/share/classes/java/lang/module/Configuration.java	Tue Sep 11 14:51:45 2018 -0700
@@ -601,8 +601,7 @@
                 // push in reverse order
                 for (int i = layer.parents.size() - 1; i >= 0; i--) {
                     Configuration parent = layer.parents.get(i);
-                    if (!visited.contains(parent)) {
-                        visited.add(parent);
+                    if (visited.add(parent)) {
                         stack.push(parent);
                     }
                 }
diff -r ebd5b1ad971a -r 4ffb0a33f265 src/java.base/share/classes/java/util/ServiceLoader.java
--- a/src/java.base/share/classes/java/util/ServiceLoader.java	Tue Sep 11 13:54:34 2018 -0700
+++ b/src/java.base/share/classes/java/util/ServiceLoader.java	Tue Sep 11 14:51:45 2018 -0700
@@ -938,8 +938,7 @@
                     List<ModuleLayer> parents = layer.parents();
                     for (int i = parents.size() - 1; i >= 0; i--) {
                         ModuleLayer parent = parents.get(i);
-                        if (!visited.contains(parent)) {
-                            visited.add(parent);
+                        if (visited.add(parent)) {
                             stack.push(parent);
                         }
                     }
diff -r ebd5b1ad971a -r 4ffb0a33f265 src/java.base/share/classes/java/util/stream/DistinctOps.java
--- a/src/java.base/share/classes/java/util/stream/DistinctOps.java	Tue Sep 11 13:54:34 2018 -0700
+++ b/src/java.base/share/classes/java/util/stream/DistinctOps.java	Tue Sep 11 14:51:45 2018 -0700
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012, 2013, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2012, 2018, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -170,8 +170,7 @@
 
                         @Override
                         public void accept(T t) {
-                            if (!seen.contains(t)) {
-                                seen.add(t);
+                            if (seen.add(t)) {
                                 downstream.accept(t);
                             }
                         }