Remove unnecessary rescue-all exception handling

Signed-off-by: Jack Nagel <jacknagel@gmail.com>

2 files changed, 12 insertions, 21 deletions

diff --git a/Library/Homebrew/cmd/search.rb b/Library/Homebrew/cmd/search.rb
index 2ceefd2ce..e86594253 100644
--- a/Library/Homebrew/cmd/search.rb
+++ b/Library/Homebrew/cmd/search.rb
@@ -64,16 +64,13 @@ module Homebrew extend self
 
     results = []
     GitHub.open "https://api.github.com/repos/#{user}/homebrew-#{repo}/git/trees/HEAD?recursive=1" do |f|
-      begin
-        user.downcase! if user == "Homebrew" # special handling for the Homebrew organization
-        MultiJson.decode(f.read)["tree"].map{ |hash| hash['path'] }.compact.each do |file|
-          name = File.basename(file, '.rb')
-          if file =~ /\.rb$/ and name =~ rx
-            results << "#{user}/#{repo}/#{name}"
-            $found += 1
-          end
+      user.downcase! if user == "Homebrew" # special handling for the Homebrew organization
+      MultiJson.decode(f.read)["tree"].map{ |hash| hash['path'] }.compact.each do |file|
+        name = File.basename(file, '.rb')
+        if file =~ /\.rb$/ and name =~ rx
+          results << "#{user}/#{repo}/#{name}"
+          $found += 1
         end
-      rescue
       end
     end
     results
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