in the inner loop of values(), there's an assignment to a boolean that is 
never used.

Index: ./src/java/org/apache/commons/collections/MultiHashMap.java
===================================================================
RCS file: /home/cvspublic/jakarta-commons/collections/src/java/org/apache/commons/collections/MultiHashMap.java,v
retrieving revision 1.4
diff -u -w -r1.4 MultiHashMap.java
--- ./src/java/org/apache/commons/collections/MultiHashMap.java	12 Jun 2002 03:59:15 -0000	1.4
+++ ./src/java/org/apache/commons/collections/MultiHashMap.java	6 Oct 2002 17:50:26 -0000
@@ -187,7 +191,7 @@
             
             Object[] values = list.toArray();
             for( int ii=0; ii < values.length; ii++ ) {
-                boolean successfulAdd = returnList.add( values[ii] );
+                returnList.add( values[ii] );
             }
         }
         return returnList;

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to