GitHub user tabish121 opened a pull request:
https://github.com/apache/activemq-artemis/pull/1524
ARTEMIS-1400 Fix some minor issues spotted with FindBugs
Remove a bit of dead code, and invert two string tests that were testing
for null after comparing the value to empty string (NPE scenario).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tabish121/activemq-artemis ARTEMIS-1400
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/activemq-artemis/pull/1524.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 #1524
----
commit a02187bb8a487853e4020ec54819bf5bca0ee154
Author: Timothy Bish <[email protected]>
Date: 2017-09-08T15:41:28Z
ARTEMIS-1400 Fix some minor issues spotted with FindBugs
Remove a bit of dead code, and invert two string tests that were testing
for null after comparing the value to empty string (NPE scenario).
----
---
