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 2013/01/30 18:25:38 UTC

svn commit: r1440536 - in /commons/proper/cli/trunk/src/test/java/org/apache/commons/cli: BugsTest.java OptionBuilderTest.java bug/BugCLI71Test.java

Author: sebb
Date: Wed Jan 30 17:25:38 2013
New Revision: 1440536

URL: http://svn.apache.org/viewvc?rev=1440536&view=rev
Log:
Unused variables

Modified:
    commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/BugsTest.java
    commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/OptionBuilderTest.java
    commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java

Modified: commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/BugsTest.java
URL: http://svn.apache.org/viewvc/commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/BugsTest.java?rev=1440536&r1=1440535&r2=1440536&view=diff
==============================================================================
--- commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/BugsTest.java (original)
+++ commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/BugsTest.java Wed Jan 30 17:25:38 2013
@@ -268,7 +268,7 @@ public class BugsTest extends TestCase
         String[] args = new String[] {  };
         try
         {
-            CommandLine line = parser.parse(opts, args);
+            parser.parse(opts, args);
         }
         catch (ParseException exp)
         {
@@ -285,7 +285,7 @@ public class BugsTest extends TestCase
         args = new String[] { "-s" };
         try
         {
-            CommandLine line = parser.parse(opts, args);
+            parser.parse(opts, args);
         }
         catch (ParseException exp)
         {

Modified: commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/OptionBuilderTest.java
URL: http://svn.apache.org/viewvc/commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/OptionBuilderTest.java?rev=1440536&r1=1440535&r2=1440536&view=diff
==============================================================================
--- commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/OptionBuilderTest.java (original)
+++ commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/OptionBuilderTest.java Wed Jan 30 17:25:38 2013
@@ -127,7 +127,7 @@ public class OptionBuilderTest extends T
 
         // bad character in option string
         try {
-            Option opt = OptionBuilder.create( "opt`" );
+            OptionBuilder.create( "opt`" );
             fail( "IllegalArgumentException not caught" );
         }
         catch( IllegalArgumentException exp ) {
@@ -136,7 +136,7 @@ public class OptionBuilderTest extends T
 
         // valid option 
         try {
-            Option opt = OptionBuilder.create( "opt" );
+            OptionBuilder.create( "opt" );
             // success
         }
         catch( IllegalArgumentException exp ) {

Modified: commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java
URL: http://svn.apache.org/viewvc/commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java?rev=1440536&r1=1440535&r2=1440536&view=diff
==============================================================================
--- commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java (original)
+++ commons/proper/cli/trunk/src/test/java/org/apache/commons/cli/bug/BugCLI71Test.java Wed Jan 30 17:25:38 2013
@@ -67,7 +67,7 @@ public class BugCLI71Test extends TestCa
     public void testLackOfError() throws Exception {
         String[] args = new String[] { "-k", "-a",  "Caesar" };
         try {
-            CommandLine line = parser.parse( options, args);
+            parser.parse( options, args);
             fail("MissingArgumentException expected");
         } catch(MissingArgumentException e) {
             assertEquals("option missing an argument", "k", e.getOption().getOpt());