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 2014/06/27 15:03:37 UTC
svn commit: r1606082 -
/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java
Author: ggregory
Date: Fri Jun 27 13:03:36 2014
New Revision: 1606082
URL: http://svn.apache.org/r1606082
Log:
Statement unnecessarily nested within else clause.
Modified:
commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java
Modified: commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java?rev=1606082&r1=1606081&r2=1606082&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java (original)
+++ commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/event/EventUtils.java Fri Jun 27 13:03:36 2014
@@ -111,9 +111,8 @@ public class EventUtils {
if (eventTypes.isEmpty() || eventTypes.contains(method.getName())) {
if (hasMatchingParametersMethod(method)) {
return MethodUtils.invokeMethod(target, methodName, parameters);
- } else {
- return MethodUtils.invokeMethod(target, methodName);
}
+ return MethodUtils.invokeMethod(target, methodName);
}
return null;
}