From 6a58dd11fa19ce99c279dd98164a5ccbc705a804 Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Thu, 28 Aug 2014 22:11:08 -0500 Subject: Simplify implementation of ARGV.flag? and ARGV.switch? --- Library/Homebrew/extend/ARGV.rb | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) (limited to 'Library/Homebrew/extend') diff --git a/Library/Homebrew/extend/ARGV.rb b/Library/Homebrew/extend/ARGV.rb index 648f2a29e..1bb0b9701 100644 --- a/Library/Homebrew/extend/ARGV.rb +++ b/Library/Homebrew/extend/ARGV.rb @@ -135,9 +135,7 @@ module HomebrewArgvExtension end def flag? flag - options_only.any? do |arg| - arg == flag || arg[1, 1] != "-" && arg.include?(flag[2, 1]) - end + options_only.include?(flag) || switch?(flag[2, 1]) end def force_bottle? @@ -145,11 +143,9 @@ module HomebrewArgvExtension end # eg. `foo -ns -i --bar` has three switches, n, s and i - def switch? switch_character - return false if switch_character.length > 1 - options_only.any? do |arg| - arg[1, 1] != "-" && arg.include?(switch_character) - end + def switch? char + return false if char.length > 1 + options_only.any? { |arg| arg[1, 1] != "-" && arg.include?(char) } end def usage -- cgit v1.2.3