You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2012/05/04 02:52:23 UTC

svn commit: r1333710 - /commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/language/DoubleMetaphoneTest.java

Author: sebb
Date: Fri May  4 00:52:23 2012
New Revision: 1333710

URL: http://svn.apache.org/viewvc?rev=1333710&view=rev
Log:
Eliminate unused variable warning by slight code rearrangement

Modified:
    commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/language/DoubleMetaphoneTest.java

Modified: commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/language/DoubleMetaphoneTest.java
URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/language/DoubleMetaphoneTest.java?rev=1333710&r1=1333709&r2=1333710&view=diff
==============================================================================
--- commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/language/DoubleMetaphoneTest.java (original)
+++ commons/proper/codec/trunk/src/test/java/org/apache/commons/codec/language/DoubleMetaphoneTest.java Fri May  4 00:52:23 2012
@@ -1190,12 +1190,12 @@ public class DoubleMetaphoneTest extends
                 matches.append("{\"" + name0 + "\", \"" + name1 + "\"}," + cr);
             }
         }
-        String msg = failures.toString();
         matches.append("};");
         // Turn on to print a new MATCH array
         //System.out.println(matches.toString());
-        if (msg.length() > 0) {
+        if (failCount > 0) {
             // Turn on to see which pairs do NOT match.
+            // String msg = failures.toString();
             //fail(failCount + " failures out of " + FIXTURE.length + ". The
             // following could be made to match: " + cr + msg);
         }