[
https://issues.apache.org/jira/browse/ARTEMIS-996?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
clebert suconic closed ARTEMIS-996.
-----------------------------------
> 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
> Fix For: 2.0.0
>
>
> 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.4.14#64029)
