aboutsummaryrefslogtreecommitdiffstats
path: root/Library/Homebrew/utils.rb
diff options
context:
space:
mode:
authorDaniel Lee Harple2013-05-18 08:34:28 -0400
committerJack Nagel2013-05-22 19:53:05 -0500
commit471502bc06e8dde06618b30885d21d10c53528e5 (patch)
tree55812e3363ac7e3a0c3613731442dd596aa39560 /Library/Homebrew/utils.rb
parent08388b149e5a45d21214170a038a118eeaba850d (diff)
downloadbrew-471502bc06e8dde06618b30885d21d10c53528e5.tar.bz2
Remove unnecessary rescue-all exception handling
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Diffstat (limited to 'Library/Homebrew/utils.rb')
-rw-r--r--Library/Homebrew/utils.rb18
1 files changed, 6 insertions, 12 deletions
diff --git a/Library/Homebrew/utils.rb b/Library/Homebrew/utils.rb
index ad86946aa..768ea2a4c 100644
--- a/Library/Homebrew/utils.rb
+++ b/Library/Homebrew/utils.rb
@@ -286,13 +286,10 @@ module GitHub extend self
uri = URI.parse("https://api.github.com/legacy/issues/search/mxcl/homebrew/open/#{name}")
- open uri do |f|
- begin
- MultiJson.decode(f.read)['issues'].each do |issue|
- # don't include issues that just refer to the tool in their body
- issues << issue['html_url'] if issue['title'].include? name
- end
- rescue
+ GitHub.open uri do |f|
+ MultiJson.decode(f.read)['issues'].each do |issue|
+ # don't include issues that just refer to the tool in their body
+ issues << issue['html_url'] if issue['title'].include? name
end
end
@@ -306,11 +303,8 @@ module GitHub extend self
uri = URI.parse("https://api.github.com/legacy/issues/search/mxcl/homebrew/open/#{query}")
GitHub.open uri do |f|
- begin
- MultiJson.decode(f.read)['issues'].each do |pull|
- yield pull['pull_request_url'] if rx.match pull['title'] and pull['pull_request_url']
- end
- rescue
+ MultiJson.decode(f.read)['issues'].each do |pull|
+ yield pull['pull_request_url'] if rx.match pull['title'] and pull['pull_request_url']
end
end
end