src/share/classes/java/util/Collections.java

rev 7994 : 8024291: Rename Collection.removeIf(Predicate) back to removeAll(Predicate)
Reviewed-by: duke

@@ -1138,11 +1138,11 @@
         @Override
         public void forEach(Consumer<? super E> action) {
             c.forEach(action);
         }
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
+        public boolean removeAll(Predicate<? super E> filter) {
             throw new UnsupportedOperationException();
         }
         @SuppressWarnings("unchecked")
         @Override
         public Spliterator<E> spliterator() {

@@ -2120,12 +2120,12 @@
         @Override
         public void forEach(Consumer<? super E> consumer) {
             synchronized (mutex) {c.forEach(consumer);}
         }
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
-            synchronized (mutex) {return c.removeIf(filter);}
+        public boolean removeAll(Predicate<? super E> filter) {
+            synchronized (mutex) {return c.removeAll(filter);}
         }
         @Override
         public Spliterator<E> spliterator() {
             return c.spliterator(); // Must be manually synched by user!
         }

@@ -3182,12 +3182,12 @@
 
         // Override default methods in Collection
         @Override
         public void forEach(Consumer<? super E> action) {c.forEach(action);}
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
-            return c.removeIf(filter);
+        public boolean removeAll(Predicate<? super E> filter) {
+            return c.removeAll(filter);
         }
         @Override
         public Spliterator<E> spliterator() {return c.spliterator();}
         @Override
         public Stream<E> stream()           {return c.stream();}

@@ -4391,11 +4391,11 @@
         @Override
         public void forEach(Consumer<? super E> action) {
             Objects.requireNonNull(action);
         }
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
+        public boolean removeAll(Predicate<? super E> filter) {
             Objects.requireNonNull(filter);
             return false;
         }
         @Override
         public Spliterator<E> spliterator() { return Spliterators.emptySpliterator(); }

@@ -4517,11 +4517,11 @@
         }
 
         public int hashCode() { return 1; }
 
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
+        public boolean removeAll(Predicate<? super E> filter) {
             Objects.requireNonNull(filter);
             return false;
         }
         @Override
         public void replaceAll(UnaryOperator<E> operator) {

@@ -4827,11 +4827,11 @@
         @Override
         public Spliterator<E> spliterator() {
             return singletonSpliterator(element);
         }
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
+        public boolean removeAll(Predicate<? super E> filter) {
             throw new UnsupportedOperationException();
         }
     }
 
     /**

@@ -4878,11 +4878,11 @@
         @Override
         public void forEach(Consumer<? super E> action) {
             action.accept(element);
         }
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
+        public boolean removeAll(Predicate<? super E> filter) {
             throw new UnsupportedOperationException();
         }
         @Override
         public void replaceAll(UnaryOperator<E> operator) {
             throw new UnsupportedOperationException();

@@ -5532,12 +5532,12 @@
         @Override
         public void forEach(Consumer<? super E> action) {
             s.forEach(action);
         }
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
-            return s.removeIf(filter);
+        public boolean removeAll(Predicate<? super E> filter) {
+            return s.removeAll(filter);
         }
 
         @Override
         public Spliterator<E> spliterator() {return s.spliterator();}
         @Override

@@ -5607,12 +5607,12 @@
 
         // Override default methods in Collection
         @Override
         public void forEach(Consumer<? super E> action) {q.forEach(action);}
         @Override
-        public boolean removeIf(Predicate<? super E> filter) {
-            return q.removeIf(filter);
+        public boolean removeAll(Predicate<? super E> filter) {
+            return q.removeAll(filter);
         }
         @Override
         public Spliterator<E> spliterator() {return q.spliterator();}
         @Override
         public Stream<E> stream()           {return q.stream();}