diff -r 60e247b8d9a4 -r f9c6f543c4db jdk/src/java.base/share/classes/java/util/concurrent/CopyOnWriteArrayList.java --- a/jdk/src/java.base/share/classes/java/util/concurrent/CopyOnWriteArrayList.java Fri Feb 03 13:24:59 2017 -0800 +++ b/jdk/src/java.base/share/classes/java/util/concurrent/CopyOnWriteArrayList.java Fri Feb 03 13:24:59 2017 -0800 @@ -797,7 +797,7 @@ * @throws NullPointerException {@inheritDoc} */ public void forEach(Consumer action) { - if (action == null) throw new NullPointerException(); + Objects.requireNonNull(action); for (Object x : getArray()) { @SuppressWarnings("unchecked") E e = (E) x; action.accept(e); @@ -808,7 +808,7 @@ * @throws NullPointerException {@inheritDoc} */ public boolean removeIf(Predicate filter) { - if (filter == null) throw new NullPointerException(); + Objects.requireNonNull(filter); return bulkRemove(filter); } @@ -865,7 +865,7 @@ } public void replaceAll(UnaryOperator operator) { - if (operator == null) throw new NullPointerException(); + Objects.requireNonNull(operator); synchronized (lock) { replaceAll(operator, 0, getArray().length); } @@ -1329,7 +1329,7 @@ } public void forEach(Consumer action) { - if (action == null) throw new NullPointerException(); + Objects.requireNonNull(action); int i, end; final Object[] es; synchronized (l.lock) { es = getArrayChecked(); @@ -1341,7 +1341,7 @@ } public void replaceAll(UnaryOperator operator) { - if (operator == null) throw new NullPointerException(); + Objects.requireNonNull(operator); synchronized (l.lock) { checkForComodification(); l.replaceAll(operator, offset, offset + size);