Skip to content

Commit 9d05157

Browse files
committed
Fix some obvious raw types
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/trunk@1440544 13f79535-47bb-0310-9956-ffa450edef68
1 parent 1fcf87d commit 9d05157

3 files changed

Lines changed: 13 additions & 13 deletions

File tree

src/main/java/org/apache/commons/cli/PatternOptionBuilder.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -55,16 +55,16 @@
5555
public class PatternOptionBuilder
5656
{
5757
/** String class */
58-
public static final Class STRING_VALUE = String.class;
58+
public static final Class<String> STRING_VALUE = String.class;
5959

6060
/** Object class */
61-
public static final Class OBJECT_VALUE = Object.class;
61+
public static final Class<Object> OBJECT_VALUE = Object.class;
6262

6363
/** Number class */
64-
public static final Class NUMBER_VALUE = Number.class;
64+
public static final Class<Number> NUMBER_VALUE = Number.class;
6565

6666
/** Date class */
67-
public static final Class DATE_VALUE = Date.class;
67+
public static final Class<Date> DATE_VALUE = Date.class;
6868

6969
/** Class class */
7070
public static final Class CLASS_VALUE = Class.class;
@@ -74,16 +74,16 @@ public class PatternOptionBuilder
7474
// ie) it's for reading not writing.
7575

7676
/** FileInputStream class */
77-
public static final Class EXISTING_FILE_VALUE = FileInputStream.class;
77+
public static final Class<FileInputStream> EXISTING_FILE_VALUE = FileInputStream.class;
7878

7979
/** File class */
80-
public static final Class FILE_VALUE = File.class;
80+
public static final Class<File> FILE_VALUE = File.class;
8181

8282
/** File array class */
83-
public static final Class FILES_VALUE = File[].class;
83+
public static final Class<File[]> FILES_VALUE = File[].class;
8484

8585
/** URL class */
86-
public static final Class URL_VALUE = URL.class;
86+
public static final Class<URL> URL_VALUE = URL.class;
8787

8888
/**
8989
* Retrieve the class that <code>ch</code> represents.

src/test/java/org/apache/commons/cli/ParserTestCase.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -421,15 +421,15 @@ public void testPropertiesOption1() throws Exception
421421

422422
CommandLine cl = parser.parse(options, args);
423423

424-
List values = Arrays.asList(cl.getOptionValues("J"));
424+
List<String> values = Arrays.asList(cl.getOptionValues("J"));
425425
assertNotNull("null values", values);
426426
assertEquals("number of values", 4, values.size());
427427
assertEquals("value 1", "source", values.get(0));
428428
assertEquals("value 2", "1.5", values.get(1));
429429
assertEquals("value 3", "target", values.get(2));
430430
assertEquals("value 4", "1.5", values.get(3));
431431

432-
List argsleft = cl.getArgList();
432+
List<?> argsleft = cl.getArgList();
433433
assertEquals("Should be 1 arg left", 1, argsleft.size());
434434
assertEquals("Expecting foo", "foo", argsleft.get(0));
435435
}
@@ -449,7 +449,7 @@ public void testPropertiesOption2() throws Exception
449449
assertEquals("property 1", "true", props.getProperty("param1"));
450450
assertEquals("property 2", "value2", props.getProperty("param2"));
451451

452-
List argsleft = cl.getArgList();
452+
List<?> argsleft = cl.getArgList();
453453
assertEquals("Should be no arg left", 0, argsleft.size());
454454
}
455455

src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public void testSimplePattern() throws Exception
4343

4444
assertEquals("flag a", "foo", line.getOptionValue("a"));
4545
assertEquals("string flag a", "foo", line.getOptionObject("a"));
46-
assertEquals("object flag b", new Vector(), line.getOptionObject("b"));
46+
assertEquals("object flag b", new Vector<Object>(), line.getOptionObject("b"));
4747
assertTrue("boolean true flag c", line.hasOption("c"));
4848
assertFalse("boolean false flag d", line.hasOption("d"));
4949
assertEquals("file flag e", new File("build.xml"), line.getOptionObject("e"));
@@ -54,7 +54,7 @@ public void testSimplePattern() throws Exception
5454
// tests the char methods of CommandLine that delegate to the String methods
5555
assertEquals("flag a", "foo", line.getOptionValue('a'));
5656
assertEquals("string flag a", "foo", line.getOptionObject('a'));
57-
assertEquals("object flag b", new Vector(), line.getOptionObject('b'));
57+
assertEquals("object flag b", new Vector<Object>(), line.getOptionObject('b'));
5858
assertTrue("boolean true flag c", line.hasOption('c'));
5959
assertFalse("boolean false flag d", line.hasOption('d'));
6060
assertEquals("file flag e", new File("build.xml"), line.getOptionObject('e'));

0 commit comments

Comments
 (0)