diff options
| author | Jack Nagel | 2014-05-27 21:02:00 -0500 |
|---|---|---|
| committer | Jack Nagel | 2014-05-27 21:03:47 -0500 |
| commit | 28fa5b0261b3efef222e8f92f5b3906cccfeb388 (patch) | |
| tree | 31b355954b417954535f9ebaef09e9049e199bb2 /Library | |
| parent | 152490b7b00ddeac71eb5cb27d145797c981d55c (diff) | |
| download | brew-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.rb | 8 | ||||
| -rw-r--r-- | Library/Homebrew/resource.rb | 8 |
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={} |
