GitHub user BruceKuiLiu opened a pull request:
https://github.com/apache/aries/pull/76
Remove the useless conditional sub-expression.
This statement contains a useless conditional expression, where this
conditional expression returns a false value regardless of what the value of
isValid is.
http://findbugs.sourceforge.net/bugDescriptions.html#UCF_USELESS_CONTROL_FLOW
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/BruceKuiLiu/aries trunk
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/aries/pull/76.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #76
----
commit 1c4387018fa18d9cc81ef8395430e42ce1de4671
Author: Kui LIU <[email protected]>
Date: 2017-10-09T08:25:47Z
Remove the useless conditional sub-expression.
This statement contains a useless conditional expression, where this
conditional expression returns a false value regardless of what the value of
isValid is.
http://findbugs.sourceforge.net/bugDescriptions.html#UCF_USELESS_CONTROL_FLOW
----
---