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 2020/11/21 02:36:23 UTC

[commons-io] 02/02: No need to initialize to default value.

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

commit 393a8042eaa186ac6d65f4898967aed1ce3668d0
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Fri Nov 20 21:36:16 2020 -0500

    No need to initialize to default value.
---
 .../java/org/apache/commons/io/input/CharSequenceInputStreamTest.java  | 3 +--
 .../org/apache/commons/io/monitor/FileAlterationMonitorTestCase.java   | 3 +--
 .../org/apache/commons/io/monitor/FileAlterationObserverTestCase.java  | 3 +--
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/test/java/org/apache/commons/io/input/CharSequenceInputStreamTest.java b/src/test/java/org/apache/commons/io/input/CharSequenceInputStreamTest.java
index 04fc61f..c728344 100644
--- a/src/test/java/org/apache/commons/io/input/CharSequenceInputStreamTest.java
+++ b/src/test/java/org/apache/commons/io/input/CharSequenceInputStreamTest.java
@@ -374,10 +374,9 @@ public class CharSequenceInputStreamTest {
         final String input = "test";
         try (InputStream r = new CharSequenceInputStream(input, csName)) {
             int available = checkAvail(r, input.length());
-            byte buff[] = new byte[available];
             assertEquals(available - 1, r.skip(available - 1)); // skip all but one
             available = checkAvail(r, 1);
-            buff = new byte[available];
+            byte[] buff = new byte[available];
             assertEquals(available, r.read(buff, 0, available));
         }
     }
diff --git a/src/test/java/org/apache/commons/io/monitor/FileAlterationMonitorTestCase.java b/src/test/java/org/apache/commons/io/monitor/FileAlterationMonitorTestCase.java
index 0fc9efc..15bdf6e 100644
--- a/src/test/java/org/apache/commons/io/monitor/FileAlterationMonitorTestCase.java
+++ b/src/test/java/org/apache/commons/io/monitor/FileAlterationMonitorTestCase.java
@@ -59,10 +59,9 @@ public class FileAlterationMonitorTestCase extends AbstractMonitorTestCase {
     @Test
     public void testAddRemoveObservers() {
         FileAlterationObserver[] observers = null;
-        FileAlterationMonitor monitor = null;
 
         // Null Observers
-        monitor = new FileAlterationMonitor(123, observers);
+        FileAlterationMonitor monitor = new FileAlterationMonitor(123, observers);
         assertEquals(123, monitor.getInterval(), "Interval");
         assertFalse(monitor.getObservers().iterator().hasNext(), "Observers[1]");
 
diff --git a/src/test/java/org/apache/commons/io/monitor/FileAlterationObserverTestCase.java b/src/test/java/org/apache/commons/io/monitor/FileAlterationObserverTestCase.java
index f114657..327b723 100644
--- a/src/test/java/org/apache/commons/io/monitor/FileAlterationObserverTestCase.java
+++ b/src/test/java/org/apache/commons/io/monitor/FileAlterationObserverTestCase.java
@@ -73,9 +73,8 @@ public class FileAlterationObserverTestCase extends AbstractMonitorTestCase {
     @Test
     public void testToString() {
         final File file = new File("/foo");
-        FileAlterationObserver observer = null;
 
-        observer = new FileAlterationObserver(file);
+        FileAlterationObserver observer = new FileAlterationObserver(file);
         assertEquals("FileAlterationObserver[file='" + file.getPath() +  "', listeners=0]",
                 observer.toString());