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 2017/04/07 17:55:14 UTC
svn commit: r1790603 -
/commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/tree/ImmutableNode.java
Author: ggregory
Date: Fri Apr 7 17:55:14 2017
New Revision: 1790603
URL: http://svn.apache.org/viewvc?rev=1790603&view=rev
Log:
Simplify if-else.
Modified:
commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/tree/ImmutableNode.java
Modified: commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/tree/ImmutableNode.java
URL: http://svn.apache.org/viewvc/commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/tree/ImmutableNode.java?rev=1790603&r1=1790602&r2=1790603&view=diff
==============================================================================
--- commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/tree/ImmutableNode.java (original)
+++ commons/proper/configuration/trunk/src/main/java/org/apache/commons/configuration2/tree/ImmutableNode.java Fri Apr 7 17:55:14 2017
@@ -550,17 +550,11 @@ public final class ImmutableNode
{
return directChildren;
}
- else
+ if (children != null)
{
- if (children != null)
- {
- return Collections.unmodifiableList(children);
- }
- else
- {
- return Collections.emptyList();
- }
+ return Collections.unmodifiableList(children);
}
+ return Collections.emptyList();
}
/**
@@ -577,17 +571,11 @@ public final class ImmutableNode
{
return directAttributes;
}
- else
+ if (attributes != null)
{
- if (attributes != null)
- {
- return Collections.unmodifiableMap(attributes);
- }
- else
- {
- return Collections.emptyMap();
- }
+ return Collections.unmodifiableMap(attributes);
}
+ return Collections.emptyMap();
}
/**