From a5895ad1fe1ae4ae296c71483e7a77d87606cdd8 Mon Sep 17 00:00:00 2001
From: Jack Nagel
Date: Fri, 18 Jul 2014 11:43:37 -0500
Subject: Check nil? || empty? instead of to_s.empty?

---
 Library/Homebrew/extend/ENV/shared.rb | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

(limited to 'Library')

diff --git a/Library/Homebrew/extend/ENV/shared.rb b/Library/Homebrew/extend/ENV/shared.rb
index 87ea397f3..583c3c639 100644
--- a/Library/Homebrew/extend/ENV/shared.rb
+++ b/Library/Homebrew/extend/ENV/shared.rb
@@ -47,20 +47,22 @@ module SharedEnvExtension
   def append keys, value, separator = ' '
     value = value.to_s
     Array(keys).each do |key|
-      unless self[key].to_s.empty?
-        self[key] = self[key] + separator + value
-      else
+      old = self[key]
+      if old.nil? || old.empty?
         self[key] = value
+      else
+        self[key] += separator + value
       end
     end
   end
   def prepend keys, value, separator = ' '
     value = value.to_s
     Array(keys).each do |key|
-      unless self[key].to_s.empty?
-        self[key] = value + separator + self[key]
-      else
+      old = self[key]
+      if old.nil? || old.empty?
         self[key] = value
+      else
+        self[key] = value + separator + old
       end
     end
   end
-- 
cgit v1.2.3