[
https://issues.apache.org/jira/browse/GEODE-2842?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16142377#comment-16142377
]
ASF subversion and git services commented on GEODE-2842:
--------------------------------------------------------
Commit 0af313241b9e6125443468752bd18d2104acc627 in geode's branch
refs/heads/develop from [~prhomberg]
[ https://git-wip-us.apache.org/repos/asf?p=geode.git;h=0af3132 ]
GEODE-2842: Removed redundant default annotation parameter values.
This closes #731
> Remove unnecessary @CliOption arguments
> ---------------------------------------
>
> Key: GEODE-2842
> URL: https://issues.apache.org/jira/browse/GEODE-2842
> Project: Geode
> Issue Type: Bug
> Components: gfsh
> Reporter: Jared Stewart
> Assignee: Jared Stewart
>
> The {{@CliOption}} annotation specifies default values for some of its fields
> (like {{mandatory}} and {{unspecifiedDefaultValue}}). Yet, we often
> explicitly set those fields to the default value. For example,
> {noformat} @CliOption(key = CliStrings.START_LOCATOR__MEMBER_NAME,
> mandatory = false,
> unspecifiedDefaultValue = CliMetaData.ANNOTATION_NULL_VALUE,
> help = CliStrings.START_LOCATOR__MEMBER_NAME__HELP) String
> memberName {noformat}
> could simply read
> {noformat} @CliOption(key = CliStrings.START_LOCATOR__MEMBER_NAME,
> help = CliStrings.START_LOCATOR__MEMBER_NAME__HELP) String
> memberName {noformat}
> Removing these redundant/unnecessary arguments will make the code a lot more
> readable.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
