edit: tweak default editors.

Add `atom` (not my editor of choice [TextMate for life!]) but widely
used enough to probably warrant it. Also, reorder based on my best
guestimates of current usage and cleanup the editor code while we're
here.

1 files changed, 9 insertions, 14 deletions

diff --git a/Library/Homebrew/utils.rb b/Library/Homebrew/utils.rb
index cde2ee306..826cf4c5c 100644
--- a/Library/Homebrew/utils.rb
+++ b/Library/Homebrew/utils.rb
@@ -328,21 +328,16 @@ def which_all(cmd, path = ENV["PATH"])
 end
 
 def which_editor
-  editor = ENV.values_at("HOMEBREW_EDITOR", "HOMEBREW_VISUAL").compact.reject(&:empty?).first
-  return editor unless editor.nil?
-
-  # Find Textmate, BBEdit / TextWrangler, or vim
-  %w[mate edit vim].each do |candidate|
-    editor = candidate if which(candidate, ENV["HOMEBREW_PATH"])
+  editor = ENV.values_at("HOMEBREW_EDITOR", "HOMEBREW_VISUAL")
+              .compact
+              .reject(&:empty?)
+              .first
+  return editor if editor
+
+  # Find Atom, Sublime Text, Textmate, BBEdit / TextWrangler, or vim
+  editor = %w[atom subl mate edit vim].find do |candidate|
+    candidate if which(candidate, ENV["HOMEBREW_PATH"])
   end
-
-  # Find Textmate
-  editor = which("mate", ENV["HOMEBREW_PATH"])
-  # Find BBEdit/TextWrangler
-  editor ||= which("edit", ENV["HOMEBREW_PATH"])
-  # Find vim
-  editor ||= which("vim", ENV["HOMEBREW_PATH"])
-  # Default to standard vim
   editor ||= "/usr/bin/vim"
 
   opoo <<-EOS.undent