admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Check nil? || empty? instead of to_s.empty?

Browse Source
This commit is contained in:
Jack Nagel 2014-07-18 11:43:37 -05:00
parent 3439703c65
commit a5895ad1fe
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
Library/Homebrew/extend/ENV/shared.rb
View File

@ -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