You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2021/09/03 13:44:36 UTC
[commons-io] branch master updated: IO-746: dropping unnecessary
casts and conversions (#267)
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-io.git
The following commit(s) were added to refs/heads/master by this push:
new 086e34b IO-746: dropping unnecessary casts and conversions (#267)
086e34b is described below
commit 086e34b9516beaffe82aed5d93da074ad7a3b1e3
Author: Davide Angelocola <df...@users.noreply.github.com>
AuthorDate: Fri Sep 3 15:44:29 2021 +0200
IO-746: dropping unnecessary casts and conversions (#267)
Signed-off-by: Davide Angelocola <da...@gmail.com>
---
src/main/java/org/apache/commons/io/IOUtils.java | 2 +-
.../java/org/apache/commons/io/FilenameUtilsTestCase.java | 10 +++++-----
.../java/org/apache/commons/io/IOUtilsCopyTestCase.java | 2 +-
src/test/java/org/apache/commons/io/IOUtilsTestCase.java | 4 ++--
.../apache/commons/io/filefilter/FileFilterTestCase.java | 12 ++++++------
.../commons/io/input/CharacterSetFilterReaderTest.java | 2 +-
src/test/java/org/apache/commons/io/input/TailerTest.java | 2 +-
.../org/apache/commons/io/input/TeeInputStreamTest.java | 14 +++++++-------
8 files changed, 24 insertions(+), 24 deletions(-)
diff --git a/src/main/java/org/apache/commons/io/IOUtils.java b/src/main/java/org/apache/commons/io/IOUtils.java
index aecbc46..720b82f 100644
--- a/src/main/java/org/apache/commons/io/IOUtils.java
+++ b/src/main/java/org/apache/commons/io/IOUtils.java
@@ -463,7 +463,7 @@ public class IOUtils {
* @see Throwable#addSuppressed(java.lang.Throwable)
*/
public static void closeQuietly(final Closeable closeable) {
- closeQuietly(closeable, (Consumer<IOException>) null);
+ closeQuietly(closeable, null);
}
/**
diff --git a/src/test/java/org/apache/commons/io/FilenameUtilsTestCase.java b/src/test/java/org/apache/commons/io/FilenameUtilsTestCase.java
index ca3bd07..c0790b2 100644
--- a/src/test/java/org/apache/commons/io/FilenameUtilsTestCase.java
+++ b/src/test/java/org/apache/commons/io/FilenameUtilsTestCase.java
@@ -1151,32 +1151,32 @@ public class FilenameUtilsTestCase {
assertFalse(FilenameUtils.isExtension(null, (Collection<String>) null));
assertFalse(FilenameUtils.isExtension("file.txt", (Collection<String>) null));
assertTrue(FilenameUtils.isExtension("file", (Collection<String>) null));
- assertFalse(FilenameUtils.isExtension("file.txt", new ArrayList<String>()));
+ assertFalse(FilenameUtils.isExtension("file.txt", new ArrayList<>()));
assertTrue(FilenameUtils.isExtension("file.txt", new ArrayList<>(Arrays.asList("txt"))));
assertFalse(FilenameUtils.isExtension("file.txt", new ArrayList<>(Arrays.asList("rtf"))));
assertTrue(FilenameUtils.isExtension("file", new ArrayList<>(Arrays.asList("rtf", ""))));
assertTrue(FilenameUtils.isExtension("file.txt", new ArrayList<>(Arrays.asList("rtf", "txt"))));
assertFalse(FilenameUtils.isExtension("a/b/file.txt", (Collection<String>) null));
- assertFalse(FilenameUtils.isExtension("a/b/file.txt", new ArrayList<String>()));
+ assertFalse(FilenameUtils.isExtension("a/b/file.txt", new ArrayList<>()));
assertTrue(FilenameUtils.isExtension("a/b/file.txt", new ArrayList<>(Arrays.asList("txt"))));
assertFalse(FilenameUtils.isExtension("a/b/file.txt", new ArrayList<>(Arrays.asList("rtf"))));
assertTrue(FilenameUtils.isExtension("a/b/file.txt", new ArrayList<>(Arrays.asList("rtf", "txt"))));
assertFalse(FilenameUtils.isExtension("a.b/file.txt", (Collection<String>) null));
- assertFalse(FilenameUtils.isExtension("a.b/file.txt", new ArrayList<String>()));
+ assertFalse(FilenameUtils.isExtension("a.b/file.txt", new ArrayList<>()));
assertTrue(FilenameUtils.isExtension("a.b/file.txt", new ArrayList<>(Arrays.asList("txt"))));
assertFalse(FilenameUtils.isExtension("a.b/file.txt", new ArrayList<>(Arrays.asList("rtf"))));
assertTrue(FilenameUtils.isExtension("a.b/file.txt", new ArrayList<>(Arrays.asList("rtf", "txt"))));
assertFalse(FilenameUtils.isExtension("a\\b\\file.txt", (Collection<String>) null));
- assertFalse(FilenameUtils.isExtension("a\\b\\file.txt", new ArrayList<String>()));
+ assertFalse(FilenameUtils.isExtension("a\\b\\file.txt", new ArrayList<>()));
assertTrue(FilenameUtils.isExtension("a\\b\\file.txt", new ArrayList<>(Arrays.asList("txt"))));
assertFalse(FilenameUtils.isExtension("a\\b\\file.txt", new ArrayList<>(Arrays.asList("rtf"))));
assertTrue(FilenameUtils.isExtension("a\\b\\file.txt", new ArrayList<>(Arrays.asList("rtf", "txt"))));
assertFalse(FilenameUtils.isExtension("a.b\\file.txt", (Collection<String>) null));
- assertFalse(FilenameUtils.isExtension("a.b\\file.txt", new ArrayList<String>()));
+ assertFalse(FilenameUtils.isExtension("a.b\\file.txt", new ArrayList<>()));
assertTrue(FilenameUtils.isExtension("a.b\\file.txt", new ArrayList<>(Arrays.asList("txt"))));
assertFalse(FilenameUtils.isExtension("a.b\\file.txt", new ArrayList<>(Arrays.asList("rtf"))));
assertTrue(FilenameUtils.isExtension("a.b\\file.txt", new ArrayList<>(Arrays.asList("rtf", "txt"))));
diff --git a/src/test/java/org/apache/commons/io/IOUtilsCopyTestCase.java b/src/test/java/org/apache/commons/io/IOUtilsCopyTestCase.java
index 2be2e29..88d2e80 100644
--- a/src/test/java/org/apache/commons/io/IOUtilsCopyTestCase.java
+++ b/src/test/java/org/apache/commons/io/IOUtilsCopyTestCase.java
@@ -275,7 +275,7 @@ public class IOUtilsCopyTestCase {
final ByteArrayOutputStream baout = new ByteArrayOutputStream();
final OutputStream out = new ThrowOnFlushAndCloseOutputStream(baout, true, true);
final Appendable writer = new OutputStreamWriter(out, StandardCharsets.US_ASCII);
- assertThrows(NullPointerException.class, () -> IOUtils.copy((Reader) null, writer));
+ assertThrows(NullPointerException.class, () -> IOUtils.copy(null, writer));
}
@SuppressWarnings("resource") // 'in' is deliberately not closed
diff --git a/src/test/java/org/apache/commons/io/IOUtilsTestCase.java b/src/test/java/org/apache/commons/io/IOUtilsTestCase.java
index 9e61e0d..30fa881 100644
--- a/src/test/java/org/apache/commons/io/IOUtilsTestCase.java
+++ b/src/test/java/org/apache/commons/io/IOUtilsTestCase.java
@@ -392,7 +392,7 @@ public class IOUtilsTestCase {
@Test
public void testCloseQuietly_CloseableIOException() {
assertDoesNotThrow(() -> {
- IOUtils.closeQuietly((Closeable) () -> {
+ IOUtils.closeQuietly(() -> {
throw new IOException();
});
});
@@ -598,7 +598,7 @@ public class IOUtilsTestCase {
@Test
public void testContentEqualsIgnoreEOL() throws Exception {
{
- assertTrue(IOUtils.contentEqualsIgnoreEOL((Reader) null, null));
+ assertTrue(IOUtils.contentEqualsIgnoreEOL(null, null));
}
{
final Reader input1 = new CharArrayReader("".toCharArray());
diff --git a/src/test/java/org/apache/commons/io/filefilter/FileFilterTestCase.java b/src/test/java/org/apache/commons/io/filefilter/FileFilterTestCase.java
index 13fe707..ed536d7 100644
--- a/src/test/java/org/apache/commons/io/filefilter/FileFilterTestCase.java
+++ b/src/test/java/org/apache/commons/io/filefilter/FileFilterTestCase.java
@@ -312,7 +312,7 @@ public class FileFilterTestCase {
assertFiltering(new AndFileFilter(filters), new File("test"), false);
assertFiltering(new AndFileFilter(), new File("test"), false);
- assertThrows(NullPointerException.class, () -> new AndFileFilter(falseFilter, (IOFileFilter) null));
+ assertThrows(NullPointerException.class, () -> new AndFileFilter(falseFilter, null));
assertThrows(NullPointerException.class, () -> new AndFileFilter(null, falseFilter));
assertThrows(NullPointerException.class, () -> new AndFileFilter((List<IOFileFilter>) null));
}
@@ -790,7 +790,7 @@ public class FileFilterTestCase {
@Test
public void testFilterListNullParameters() {
try {
- FileFilterUtils.filterList(null, Collections.<File>emptyList());
+ FileFilterUtils.filterList(null, Collections.emptyList());
fail();
} catch (final IllegalArgumentException iae) {
// Test passes, exception thrown for null filter
@@ -798,7 +798,7 @@ public class FileFilterTestCase {
final IOFileFilter filter = FileFilterUtils.trueFileFilter();
try {
- FileFilterUtils.filterList(filter, Collections.singletonList((File) null));
+ FileFilterUtils.filterList(filter, Collections.singletonList(null));
} catch (final IllegalArgumentException iae) {
// Test passes, exception thrown for list containing null
}
@@ -870,7 +870,7 @@ public class FileFilterTestCase {
@Test
public void testFilterSetNullParameters() {
try {
- FileFilterUtils.filterSet(null, Collections.<File>emptySet());
+ FileFilterUtils.filterSet(null, Collections.emptySet());
fail();
} catch (final IllegalArgumentException iae) {
// Test passes, exception thrown for null filter
@@ -878,7 +878,7 @@ public class FileFilterTestCase {
final IOFileFilter filter = FileFilterUtils.trueFileFilter();
try {
- FileFilterUtils.filterSet(filter, new HashSet<>(Collections.singletonList((File) null)));
+ FileFilterUtils.filterSet(filter, new HashSet<>(Collections.singletonList(null)));
} catch (final IllegalArgumentException iae) {
// Test passes, exception thrown for set containing null
}
@@ -1355,7 +1355,7 @@ public class FileFilterTestCase {
assertFalse(orFilter.accept(testFile.getParentFile(), testFile.getName()));
assertEquals(FileVisitResult.TERMINATE, orFilter.accept(testPath, null));
- assertThrows(NullPointerException.class, () -> new OrFileFilter(falseFilter, (IOFileFilter) null));
+ assertThrows(NullPointerException.class, () -> new OrFileFilter(falseFilter, null));
}
@Test
diff --git a/src/test/java/org/apache/commons/io/input/CharacterSetFilterReaderTest.java b/src/test/java/org/apache/commons/io/input/CharacterSetFilterReaderTest.java
index 54c2da4..670a98b 100644
--- a/src/test/java/org/apache/commons/io/input/CharacterSetFilterReaderTest.java
+++ b/src/test/java/org/apache/commons/io/input/CharacterSetFilterReaderTest.java
@@ -36,7 +36,7 @@ public class CharacterSetFilterReaderTest {
@Test
public void testInputSize0FilterSize0() throws IOException {
final StringReader input = new StringReader("");
- try (CharacterSetFilterReader reader = new CharacterSetFilterReader(input, new HashSet<Integer>(0))) {
+ try (CharacterSetFilterReader reader = new CharacterSetFilterReader(input, new HashSet<>(0))) {
assertEquals(-1, reader.read());
}
}
diff --git a/src/test/java/org/apache/commons/io/input/TailerTest.java b/src/test/java/org/apache/commons/io/input/TailerTest.java
index 713ff1b..a900c8a 100644
--- a/src/test/java/org/apache/commons/io/input/TailerTest.java
+++ b/src/test/java/org/apache/commons/io/input/TailerTest.java
@@ -435,7 +435,7 @@ public class TailerTest {
private static class TestTailerListener extends TailerListenerAdapter {
// Must be synchronized because it is written by one thread and read by another
- private final List<String> lines = Collections.synchronizedList(new ArrayList<String>());
+ private final List<String> lines = Collections.synchronizedList(new ArrayList<>());
volatile Exception exception;
diff --git a/src/test/java/org/apache/commons/io/input/TeeInputStreamTest.java b/src/test/java/org/apache/commons/io/input/TeeInputStreamTest.java
index 82e5047..4eea3f9 100644
--- a/src/test/java/org/apache/commons/io/input/TeeInputStreamTest.java
+++ b/src/test/java/org/apache/commons/io/input/TeeInputStreamTest.java
@@ -54,13 +54,13 @@ public class TeeInputStreamTest {
@Test
public void testReadNothing() throws Exception {
- assertEquals("", new String(output.toString(ASCII)));
+ assertEquals("", output.toString(ASCII));
}
@Test
public void testReadOneByte() throws Exception {
assertEquals('a', tee.read());
- assertEquals("a", new String(output.toString(ASCII)));
+ assertEquals("a", output.toString(ASCII));
}
@Test
@@ -69,7 +69,7 @@ public class TeeInputStreamTest {
assertEquals('b', tee.read());
assertEquals('c', tee.read());
assertEquals(-1, tee.read());
- assertEquals("abc", new String(output.toString(ASCII)));
+ assertEquals("abc", output.toString(ASCII));
}
@Test
@@ -80,7 +80,7 @@ public class TeeInputStreamTest {
assertEquals('b', buffer[1]);
assertEquals('c', buffer[2]);
assertEquals(-1, tee.read(buffer));
- assertEquals("abc", new String(output.toString(ASCII)));
+ assertEquals("abc", output.toString(ASCII));
}
@Test
@@ -91,7 +91,7 @@ public class TeeInputStreamTest {
assertEquals('b', buffer[5]);
assertEquals('c', buffer[6]);
assertEquals(-1, tee.read(buffer, 4, 4));
- assertEquals("abc", new String(output.toString(ASCII)));
+ assertEquals("abc", output.toString(ASCII));
}
@Test
@@ -100,7 +100,7 @@ public class TeeInputStreamTest {
assertEquals(1, tee.skip(1));
assertEquals('c', tee.read());
assertEquals(-1, tee.read());
- assertEquals("ac", new String(output.toString(ASCII)));
+ assertEquals("ac", output.toString(ASCII));
}
@Test
@@ -112,7 +112,7 @@ public class TeeInputStreamTest {
assertEquals('b', tee.read());
assertEquals('c', tee.read());
assertEquals(-1, tee.read());
- assertEquals("abbc", new String(output.toString(ASCII)));
+ assertEquals("abbc", output.toString(ASCII));
}
/**