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-configuration.git


The following commit(s) were added to refs/heads/master by this push:
     new 4935545  No need to initialize to default value.
4935545 is described below

commit 4935545522b18a96827252e6d28cf1b5774d732c
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Nov 20 18:53:36 2020 -0500

    No need to initialize to default value.
---
 .../apache/commons/configuration2/TestNullCompositeConfiguration.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/configuration2/TestNullCompositeConfiguration.java
 
b/src/test/java/org/apache/commons/configuration2/TestNullCompositeConfiguration.java
index 615fdb8..d01363e 100644
--- 
a/src/test/java/org/apache/commons/configuration2/TestNullCompositeConfiguration.java
+++ 
b/src/test/java/org/apache/commons/configuration2/TestNullCompositeConfiguration.java
@@ -239,8 +239,7 @@ public class TestNullCompositeConfiguration
         cc.addConfiguration(conf1);
         cc.addConfiguration(xmlConf);
 
-        Configuration subset = null;
-        subset = cc.subset("test");
+        Configuration subset = cc.subset("test");
         assertNotNull(subset);
         assertFalse("Shouldn't be empty", subset.isEmpty());
         assertEquals("Make sure the initial loaded configs subset overrides 
any later add configs subset", "1", subset.getString("short"));

Reply via email to