From 750d7bb2c81a58f277fa6813a7286ed68b345ebd Mon Sep 17 00:00:00 2001
From: Jack Nagel <jacknagel@gmail.com>
Date: Fri, 29 Aug 2014 19:38:32 -0500
Subject: [PATCH] Options internals no longer need to support switches

---
 Library/Homebrew/options.rb           | 18 ++----------------
 Library/Homebrew/test/test_options.rb | 13 -------------
 2 files changed, 2 insertions(+), 29 deletions(-)

diff --git a/Library/Homebrew/options.rb b/Library/Homebrew/options.rb
index e43dc3a5a6..f15af418a1 100644
--- a/Library/Homebrew/options.rb
+++ b/Library/Homebrew/options.rb
@@ -4,7 +4,8 @@ class Option
   attr_reader :name, :description, :flag
 
   def initialize(name, description="")
-    @name, @flag = split_name(name)
+    @name = name
+    @flag = "--#{name}"
     @description = description
   end
 
@@ -29,19 +30,6 @@ class Option
   def inspect
     "#<#{self.class.name}: #{flag.inspect}>"
   end
-
-  private
-
-  def split_name(name)
-    case name
-    when /^[a-zA-Z]$/
-      [name, "-#{name}"]
-    when /^-[a-zA-Z]$/
-      [name[1..1], name]
-    else
-      [name, "--#{name}"]
-    end
-  end
 end
 
 class Options
@@ -51,8 +39,6 @@ class Options
     options = new
     array.each do |e|
       case e
-      when /^-[^-]+$/
-        e[1..-1].split(//).each { |o| options << Option.new(o) }
       when /^--(.+)$/
         options << Option.new($1)
       else
diff --git a/Library/Homebrew/test/test_options.rb b/Library/Homebrew/test/test_options.rb
index 917ad590c7..73ff2b752f 100644
--- a/Library/Homebrew/test/test_options.rb
+++ b/Library/Homebrew/test/test_options.rb
@@ -23,12 +23,6 @@ class OptionTests < Homebrew::TestCase
     assert_empty @option.description
     assert_equal "foo", Option.new("foo", "foo").description
   end
-
-  def test_preserves_short_options
-    option = Option.new("-d")
-    assert_equal "-d", option.flag
-    assert_equal "d", option.name
-  end
 end
 
 class OptionsTests < Homebrew::TestCase
@@ -108,11 +102,4 @@ class OptionsTests < Homebrew::TestCase
     option2 = Option.new("bar")
     assert_equal [option1, option2].sort, Options.create(array).sort
   end
-
-  def test_create_splits_multiple_switches_with_single_dash
-    array = %w{-vd}
-    verbose = Option.new("-v")
-    debug = Option.new("-d")
-    assert_equal [verbose, debug].sort, Options.create(array).sort
-  end
 end