Jiri Danek created ARTEMIS-996:
----------------------------------

             Summary: Simplify and deduplicate lookupHome(path) in 
artemis-maven-plugin
                 Key: ARTEMIS-996
                 URL: https://issues.apache.org/jira/browse/ARTEMIS-996
             Project: ActiveMQ Artemis
          Issue Type: Improvement
          Components: Broker
    Affects Versions: 2.0.0
            Reporter: Jiri Danek
            Priority: Trivial


ArtemisCLIPlugin.java and ArtemisCreatePlugin.java in artemis-maven-plugin both 
contain the following code

{noformat}
 85      Path binFolder = path.resolve("bin");
 86
 87      if (binFolder == null && Files.exists(binFolder, 
LinkOption.NOFOLLOW_LINKS)) {
 88         return false;
 89      }
{noformat}

Check for null is unnecessary, because Path#resolve never returns null, and 
also the condition in if does not make any sense.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to