changeset 9548:146f75962536

Review comment from Mike
author henryjen
date Mon, 19 Aug 2013 16:50:07 -0700
parents 238f6fdac6ec
children 67e00b862126
files src/share/classes/java/util/Collections.java test/java/util/Collections/Wrappers.java
diffstat 2 files changed, 4 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/classes/java/util/Collections.java	Thu Aug 08 12:03:04 2013 -0700
+++ b/src/share/classes/java/util/Collections.java	Mon Aug 19 16:50:07 2013 -0700
@@ -1148,10 +1148,12 @@
         public Spliterator<E> spliterator() {
             return (Spliterator<E>)c.spliterator();
         }
+        @SuppressWarnings("unchecked")
         @Override
         public Stream<E> stream() {
             return (Stream<E>)c.stream();
         }
+        @SuppressWarnings("unchecked")
         @Override
         public Stream<E> parallelStream() {
             return (Stream<E>)c.parallelStream();
--- a/test/java/util/Collections/Wrappers.java	Thu Aug 08 12:03:04 2013 -0700
+++ b/test/java/util/Collections/Wrappers.java	Mon Aug 19 16:50:07 2013 -0700
@@ -24,7 +24,7 @@
 /**
  * @test
  * @run testng Wrappers
- * @summary Unit tests for wrapping classes should delegate to default methods
+ * @summary Ensure Collections wrapping classes provide non-default implementations
  */
 
 import java.lang.reflect.Method;
@@ -109,7 +109,7 @@
     static Method[] defaultMethods;
 
     static {
-        ArrayList<Method> list = new ArrayList<>();
+        List<Method> list = new ArrayList<>();
         Method[] methods = Collection.class.getMethods();
         for (Method m: methods) {
             if (m.isDefault()) {