diff options
| author | Daniel Lee Harple | 2013-05-18 08:34:28 -0400 |
|---|---|---|
| committer | Jack Nagel | 2013-05-22 19:53:05 -0500 |
| commit | 471502bc06e8dde06618b30885d21d10c53528e5 (patch) | |
| tree | 55812e3363ac7e3a0c3613731442dd596aa39560 /Library/Homebrew/utils.rb | |
| parent | 08388b149e5a45d21214170a038a118eeaba850d (diff) | |
| download | brew-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.rb | 18 |
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 |
