Author: ggregory
Date: Sat May 31 06:22:31 2014
New Revision: 1598828
URL: http://svn.apache.org/r1598828
Log:
Statement unnecessarily nested within else clause.
Modified:
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/visitors/PluginElementVisitor.java
Modified:
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/visitors/PluginElementVisitor.java
URL:
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/visitors/PluginElementVisitor.java?rev=1598828&r1=1598827&r2=1598828&view=diff
==============================================================================
---
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/visitors/PluginElementVisitor.java
(original)
+++
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/visitors/PluginElementVisitor.java
Sat May 31 06:22:31 2014
@@ -76,15 +76,14 @@ public class PluginElementVisitor extend
array[i] = values.get(i);
}
return array;
- } else {
- final Node namedNode = findNamedNode(name, node.getChildren());
- if (namedNode == null) {
- return null;
- }
- LOGGER.debug("{}({})", namedNode.getName(), namedNode.toString());
- node.getChildren().remove(namedNode);
- return namedNode.getObject();
}
+ final Node namedNode = findNamedNode(name, node.getChildren());
+ if (namedNode == null) {
+ return null;
+ }
+ LOGGER.debug("{}({})", namedNode.getName(), namedNode.toString());
+ node.getChildren().remove(namedNode);
+ return namedNode.getObject();
}
private Node findNamedNode(final String name, final Iterable<Node>
children) {
