aboutsummaryrefslogtreecommitdiffstats
path: root/Library
diff options
context:
space:
mode:
authorJack Nagel2014-05-27 21:02:00 -0500
committerJack Nagel2014-05-27 21:03:47 -0500
commit28fa5b0261b3efef222e8f92f5b3906cccfeb388 (patch)
tree31b355954b417954535f9ebaef09e9049e199bb2 /Library
parent152490b7b00ddeac71eb5cb27d145797c981d55c (diff)
downloadbrew-28fa5b0261b3efef222e8f92f5b3906cccfeb388.tar.bz2
Remove some evals from formula DSL
The default arguments were removed from these methods long ago, so nothing is standing in the way of using define_method here.
Diffstat (limited to 'Library')
-rw-r--r--Library/Homebrew/formula.rb8
-rw-r--r--Library/Homebrew/resource.rb8
2 files changed, 4 insertions, 12 deletions
diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb
index f0217bb74..19def4337 100644
--- a/Library/Homebrew/formula.rb
+++ b/Library/Homebrew/formula.rb
@@ -633,12 +633,8 @@ class Formula
stable.mirror(val)
end
- Checksum::TYPES.each do |cksum|
- class_eval <<-EOS, __FILE__, __LINE__ + 1
- def #{cksum}(val)
- stable.#{cksum}(val)
- end
- EOS
+ Checksum::TYPES.each do |type|
+ define_method(type) { |val| stable.send(type, val) }
end
def bottle *, &block
diff --git a/Library/Homebrew/resource.rb b/Library/Homebrew/resource.rb
index 4fabb33e3..6a72a6e99 100644
--- a/Library/Homebrew/resource.rb
+++ b/Library/Homebrew/resource.rb
@@ -95,12 +95,8 @@ class Resource
puts "For your reference the SHA1 is: #{fn.sha1}"
end
- Checksum::TYPES.each do |cksum|
- class_eval <<-EOS, __FILE__, __LINE__ + 1
- def #{cksum}(val)
- @checksum = Checksum.new(:#{cksum}, val)
- end
- EOS
+ Checksum::TYPES.each do |type|
+ define_method(type) { |val| @checksum = Checksum.new(type, val) }
end
def url val=nil, specs={}