You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by tn...@apache.org on 2013/11/13 22:14:15 UTC

svn commit: r1541722 - in /commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators: FilterIteratorTest.java IteratorIterableTest.java

Author: tn
Date: Wed Nov 13 21:14:15 2013
New Revision: 1541722

URL: http://svn.apache.org/r1541722
Log:
Move SuppressWarnings tag from for loop to method level to prevent compilation problems with IBM JDK 1.5

Modified:
    commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/FilterIteratorTest.java
    commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/IteratorIterableTest.java

Modified: commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/FilterIteratorTest.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/FilterIteratorTest.java?rev=1541722&r1=1541721&r2=1541722&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/FilterIteratorTest.java (original)
+++ commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/FilterIteratorTest.java Wed Nov 13 21:14:15 2013
@@ -91,8 +91,9 @@ public class FilterIteratorTest<E> exten
         }
     }
 
+    @SuppressWarnings("unused")
     public void testRepeatedNext() {
-        for (@SuppressWarnings("unused") final String element : array) {
+        for (final String element : array) {
             iterator.next();
         }
         verifyNoMoreElements();

Modified: commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/IteratorIterableTest.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/IteratorIterableTest.java?rev=1541722&r1=1541721&r2=1541722&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/IteratorIterableTest.java (original)
+++ commons/proper/collections/trunk/src/test/java/org/apache/commons/collections4/iterators/IteratorIterableTest.java Wed Nov 13 21:14:15 2013
@@ -45,6 +45,7 @@ public class IteratorIterableTest extend
         return iter;
     }
 
+    @SuppressWarnings("unused")
     public void testIterator() {
         final Iterator<Integer> iter = createIterator();
         final Iterable<Number> iterable = new IteratorIterable<Number>(iter);
@@ -53,7 +54,7 @@ public class IteratorIterableTest extend
         verifyIteration(iterable);
 
         // second use
-        for (@SuppressWarnings("unused") final Number actual : iterable) {
+        for (final Number actual : iterable) {
             fail("should not be able to iterate twice");
         }
     }