Change to inline errors and remove message arguments

lib/tty/option/parser/arguments.rb

@@ -92,8 +92,7 @@ def process_exact_arity(arg)
 
           if 0 < values.size && values.size < arg.arity &&
               Array(@defaults[arg.name]).size < arg.arity
-            error = InvalidArity.new(arg, values.size)
-            @error_aggregator.(error, error.message)
+            @error_aggregator.(InvalidArity.new(arg, values.size))
           end
 
           values
@@ -123,8 +122,7 @@ def process_infinite_arity(arg)
           end
 
           if values.size < arity && Array(@defaults[arg.name]).size < arity
-            error = InvalidArity.new(arg, values.size)
-            @error_aggregator.(error, error.message)
+            @error_aggregator.(InvalidArity.new(arg, values.size))
           end
 
           values
@@ -201,8 +199,7 @@ def check_required
           return if @required.empty?
 
           @required.each do |param|
-            error = MissingParameter.new(param)
-            @error_aggregator.(error, error.message)
+            @error_aggregator.(MissingParameter.new(param))
           end
         end
       end # Arguments

lib/tty/option/parser/environments.rb

@@ -180,8 +180,7 @@ def check_arity
             min_arity = param.arity < 0 ? param.arity.abs - 1 : param.arity
 
             if arity < min_arity
-              error = InvalidArity.new(param, arity)
-              @error_aggregator.(error, error.message)
+              @error_aggregator.(InvalidArity.new(param, arity))
             end
           end
         end
@@ -195,8 +194,7 @@ def check_required
           return if @required.empty?
 
           @required.each do |param|
-            error = MissingParameter.new(param)
-            @error_aggregator.(error, error.message)
+            @error_aggregator.(MissingParameter.new(param))
           end
         end
       end # Environments

lib/tty/option/parser/keywords.rb

@@ -165,8 +165,7 @@ def check_arity
             min_arity = param.arity < 0 ? param.arity.abs - 1 : param.arity
 
             if arity < min_arity
-              error = InvalidArity.new(param, arity)
-              @error_aggregator.(error, error.message)
+              @error_aggregator.(InvalidArity.new(param, arity))
             end
           end
         end
@@ -180,8 +179,7 @@ def check_required
           return if @required.empty?
 
           @required.each do |param|
-            error = MissingParameter.new(param)
-            @error_aggregator.(error, error.message)
+            @error_aggregator.(MissingParameter.new(param))
           end
         end
       end # Keywords

lib/tty/option/parser/options.rb

@@ -132,8 +132,7 @@ def process_double_option(long, sep, rest)
               elsif !@argv.empty?
                 value = opt.multi_argument? ? consume_arguments : @argv.shift
               else
-                error = MissingArgument.new(opt, long)
-                @error_aggregator.(error, error.message)
+                @error_aggregator.(MissingArgument.new(opt, long))
               end
             elsif opt.argument_optional?
               if !rest.empty?
@@ -273,8 +272,7 @@ def check_arity
             min_arity = param.arity < 0 ? param.arity.abs - 1 : param.arity
 
             if arity < min_arity
-              error = InvalidArity.new(param, arity)
-              @error_aggregator.(error, error.message)
+              @error_aggregator.(InvalidArity.new(param, arity))
             end
           end
         end
@@ -288,8 +286,7 @@ def check_required
           return if @required.empty?
 
           @required.each do |param|
-            error = MissingParameter.new(param)
-            @error_aggregator.(error, error.message)
+            @error_aggregator.(MissingParameter.new(param))
           end
         end
       end # Options