Remove "scalar" call entirely

Context with the lhs of the == operator forces this into scalar context
anyway, making this call redundant.

diff --git a/lib/Monitoring/Plugin/Getopt.pm b/lib/Monitoring/Plugin/Getopt.pm
index 65c872f..c8033d0 100644 (file)
--- a/lib/Monitoring/Plugin/Getopt.pm
+++ b/lib/Monitoring/Plugin/Getopt.pm
@@ -392,7 +392,7 @@ sub arg
   );
 
   # Named args
-  if (exists $params{$_[0]} && scalar @_ % 2 == 0) {
+  if (exists $params{$_[0]} && @_ % 2 == 0) {
     %args = validate( @_, \%params );
   }