diff options
| author | Kui LIU <brucekuiliu@gmail.com> | 2017-10-14 17:20:29 +0200 |
|---|---|---|
| committer | Stefan Bodewig <bodewig@apache.org> | 2017-10-15 07:25:19 +0200 |
| commit | 9ec11ae3b04d5a8389a6f90b70463f16ecd541a2 (patch) | |
| tree | b0ab0758d28940d796e523e08cfa7b5933c54284 | |
| parent | 20fe0ffaf6c8965beccad519c644df2321f5006c (diff) | |
| download | ant-9ec11ae3b04d5a8389a6f90b70463f16ecd541a2.tar.gz | |
Remove the redundant nullcheck of value known to be non-null.
This statement contains a redundant check of a known non-null s against the constant null.
http://findbugs.sourceforge.net/bugDescriptions.html#RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
| -rw-r--r-- | src/main/org/apache/tools/ant/dispatch/DispatchUtils.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/org/apache/tools/ant/dispatch/DispatchUtils.java b/src/main/org/apache/tools/ant/dispatch/DispatchUtils.java index 1a7c1f98a..62dd88fd0 100644 --- a/src/main/org/apache/tools/ant/dispatch/DispatchUtils.java +++ b/src/main/org/apache/tools/ant/dispatch/DispatchUtils.java @@ -64,7 +64,7 @@ public class DispatchUtils { final Object o = actionM.invoke(dispatchable, (Object[]) null); if (o != null) { final String s = o.toString(); - if (s != null && s.trim().length() > 0) { + if (s.trim().length() > 0) { methodName = s.trim(); Method executeM = null; executeM = dispatchable.getClass().getMethod( |