11 files changed, 18 insertions, 33 deletions
diff --git a/Library/Homebrew/cask/lib/hbc.rb b/Library/Homebrew/cask/lib/hbc.rb
index 62b391638..69b6e8f21 100644
--- a/Library/Homebrew/cask/lib/hbc.rb
+++ b/Library/Homebrew/cask/lib/hbc.rb
@@ -1,7 +1,6 @@
 module Hbc; end
 
 require "hardware"
-require "hbc/extend"
 require "hbc/artifact"
 require "hbc/audit"
 require "hbc/auditor"
diff --git a/Library/Homebrew/cask/lib/hbc/artifact/artifact.rb b/Library/Homebrew/cask/lib/hbc/artifact/artifact.rb
index cb35821cc..b42db877d 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/artifact.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/artifact.rb
@@ -1,5 +1,7 @@
 require "hbc/artifact/moved"
 
+require "hbc/utils/hash_validator"
+
 module Hbc
   module Artifact
     class Artifact < Moved
@@ -16,7 +18,7 @@ module Hbc
         raise CaskInvalidError.new(@cask.token, "no source given for artifact") if source_string.nil?
         @source = @cask.staged_path.join(source_string)
         raise CaskInvalidError.new(@cask.token, "target required for generic artifact #{source_string}") unless target_hash.is_a?(Hash)
-        target_hash.assert_valid_keys(:target)
+        target_hash.extend(HashValidator).assert_valid_keys(:target)
         @target = Pathname.new(target_hash[:target])
       end
     end
diff --git a/Library/Homebrew/cask/lib/hbc/artifact/pkg.rb b/Library/Homebrew/cask/lib/hbc/artifact/pkg.rb
index e590a9082..372740631 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/pkg.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/pkg.rb
@@ -1,5 +1,7 @@
 require "hbc/artifact/base"
 
+require "hbc/utils/hash_validator"
+
 module Hbc
   module Artifact
     class Pkg < Base
@@ -14,7 +16,7 @@ module Hbc
         @pkg_install_opts = pkg_description.shift
         begin
           if @pkg_install_opts.respond_to?(:keys)
-            @pkg_install_opts.assert_valid_keys(:allow_untrusted)
+            @pkg_install_opts.extend(HashValidator).assert_valid_keys(:allow_untrusted)
           elsif @pkg_install_opts
             raise
           end
diff --git a/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb b/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
index 953045b32..1b5a4ff47 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
@@ -1,5 +1,7 @@
 require "hbc/artifact/base"
 
+require "hbc/utils/hash_validator"
+
 module Hbc
   module Artifact
     class Relocated < Base
@@ -46,7 +48,7 @@ module Hbc
         @source = @cask.staged_path.join(source_string)
         if target_hash
           raise CaskInvalidError unless target_hash.respond_to?(:keys)
-          target_hash.assert_valid_keys(:target)
+          target_hash.extend(HashValidator).assert_valid_keys(:target)
           @target = Hbc.send(self.class.artifact_dirmethod).join(target_hash[:target])
         else
           @target = Hbc.send(self.class.artifact_dirmethod).join(source.basename)
diff --git a/Library/Homebrew/cask/lib/hbc/cli.rb b/Library/Homebrew/cask/lib/hbc/cli.rb
index f637ae7af..2173a52d0 100644
--- a/Library/Homebrew/cask/lib/hbc/cli.rb
+++ b/Library/Homebrew/cask/lib/hbc/cli.rb
@@ -1,6 +1,8 @@
 require "optparse"
 require "shellwords"
 
+require "extend/optparse"
+
 require "hbc/cli/base"
 require "hbc/cli/audit"
 require "hbc/cli/cat"
diff --git a/Library/Homebrew/cask/lib/hbc/extend.rb b/Library/Homebrew/cask/lib/hbc/extend.rb
deleted file mode 100644
index e836b6bc8..000000000
--- a/Library/Homebrew/cask/lib/hbc/extend.rb
+++ /dev/null
@@ -1,6 +0,0 @@
-# monkeypatching
-require "hbc/extend/hash"
-require "hbc/extend/io"
-require "hbc/extend/optparse"
-require "extend/pathname"
-require "hbc/extend/string"
diff --git a/Library/Homebrew/cask/lib/hbc/extend/io.rb b/Library/Homebrew/cask/lib/hbc/extend/io.rb
deleted file mode 100644
index 1357293cd..000000000
--- a/Library/Homebrew/cask/lib/hbc/extend/io.rb
+++ /dev/null
@@ -1,10 +0,0 @@
-class IO
-  def readline_nonblock(sep = $INPUT_RECORD_SEPARATOR)
-    buffer = ""
-    buffer.concat(read_nonblock(1)) while buffer[-1] != sep
-    buffer
-  rescue IO::WaitReadable, EOFError => e
-    raise e if buffer.empty?
-    buffer
-  end
-end
diff --git a/Library/Homebrew/cask/lib/hbc/extend/optparse.rb b/Library/Homebrew/cask/lib/hbc/extend/optparse.rb
deleted file mode 100644
index 784d6d699..000000000
--- a/Library/Homebrew/cask/lib/hbc/extend/optparse.rb
+++ /dev/null
@@ -1,6 +0,0 @@
-require "optparse"
-require "pathname"
-
-OptionParser.accept Pathname do |path|
-  Pathname(path).expand_path if path
-end
diff --git a/Library/Homebrew/cask/lib/hbc/extend/string.rb b/Library/Homebrew/cask/lib/hbc/extend/string.rb
deleted file mode 100644
index 38c284194..000000000
--- a/Library/Homebrew/cask/lib/hbc/extend/string.rb
+++ /dev/null
@@ -1,5 +0,0 @@
-class String
-  def undent
-    gsub(%r{^.{#{(slice(%r{^ +}) || '').length}}}, "")
-  end
-end
diff --git a/Library/Homebrew/cask/lib/hbc/system_command.rb b/Library/Homebrew/cask/lib/hbc/system_command.rb
index 160aadff9..aa4b9b295 100644
--- a/Library/Homebrew/cask/lib/hbc/system_command.rb
+++ b/Library/Homebrew/cask/lib/hbc/system_command.rb
@@ -1,6 +1,10 @@
 require "open3"
 require "shellwords"
 
+require "extend/io"
+
+require "hbc/utils/hash_validator"
+
 module Hbc
   class SystemCommand
     attr_reader :command
@@ -43,7 +47,8 @@ module Hbc
     attr_reader :executable, :options, :processed_output, :processed_status
 
     def process_options!
-      options.assert_valid_keys :input, :print_stdout, :print_stderr, :args, :must_succeed, :sudo, :bsexec
+      options.extend(HashValidator)
+             .assert_valid_keys :input, :print_stdout, :print_stderr, :args, :must_succeed, :sudo, :bsexec
       sudo_prefix = %w[/usr/bin/sudo -E --]
       bsexec_prefix = ["/bin/launchctl", "bsexec", options[:bsexec] == :startup ? "/" : options[:bsexec]]
       @command = [executable]
diff --git a/Library/Homebrew/cask/lib/hbc/extend/hash.rb b/Library/Homebrew/cask/lib/hbc/utils/hash_validator.rb
index 3a06ba3af..b9e11720d 100644
--- a/Library/Homebrew/cask/lib/hbc/extend/hash.rb
+++ b/Library/Homebrew/cask/lib/hbc/utils/hash_validator.rb
@@ -1,4 +1,4 @@
-class Hash
+module HashValidator
   def assert_valid_keys(*valid_keys)
     unknown_keys = keys - valid_keys
     return if unknown_keys.empty?