Better parameter name

Better instance variable name

Author: garydgregory

src/main/java/org/apache/commons/cli/Option.java

@@ -417,7 +417,7 @@ public static Builder builder(final String option) {
     private List<String> values = new ArrayList<>();
 
     /** The character that is the value separator. */
-    private char valuesep;
+    private char valueSeparator;
 
     /**
      * Private constructor used by the nested Builder class.
@@ -435,7 +435,7 @@ private Option(final Builder builder) {
         this.required = builder.required;
         this.since = builder.since;
         this.type = builder.type;
-        this.valuesep = builder.valueSeparator;
+        this.valueSeparator = builder.valueSeparator;
         this.converter = builder.converter;
     }
 
@@ -727,7 +727,7 @@ public String[] getValues() {
      * @return the value separator character.
      */
     public char getValueSeparator() {
-        return valuesep;
+        return valueSeparator;
     }
 
     /**
@@ -805,7 +805,7 @@ public boolean hasOptionalArg() {
      * @since 1.1
      */
     public boolean hasValueSeparator() {
-        return valuesep > 0;
+        return valueSeparator > 0;
     }
 
     /**
@@ -969,10 +969,10 @@ public void setType(final Object type) {
     /**
      * Sets the value separator. For example if the argument value was a Java property, the value separator would be '='.
      *
-     * @param sep The value separator.
+     * @param valueSeparator The value separator.
      */
-    public void setValueSeparator(final char sep) {
-        this.valuesep = sep;
+    public void setValueSeparator(final char valueSeparator) {
+        this.valueSeparator = valueSeparator;
     }
 
     String toDeprecatedString() {