cleanedup my boneheaded 'rescue {}'

aberant · Dec 8, 2009 · 60d9c38 · 60d9c38
1 parent aff0269
commit 60d9c38
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 7 deletions.
diff --git a/lib/spittle/mtime_tracker.rb b/lib/spittle/mtime_tracker.rb
@@ -5,7 +5,7 @@ def initialize(dir, options = {})
   end
 
   def cleanup
-    File.delete(mtime_file) rescue {}
+    File.delete(mtime_file) rescue nil
   end
 
   def fresh?
@@ -30,7 +30,6 @@ def current_mtimes
   end
 
   def file_changed?(file)
-    #puts "#{file} #{mtimes[file]} != #{current_mtimes[file]}"
     mtimes[file] != current_mtimes[file]
   end
 

diff --git a/lib/spittle/stylesheet_builder.rb b/lib/spittle/stylesheet_builder.rb
@@ -17,6 +17,6 @@ def write
   end
 
   def cleanup
-    File.delete(@output_file) rescue {}
+    File.delete(@output_file) rescue nil
   end
 end
diff --git a/spec/integration_spec.rb b/spec/integration_spec.rb
@@ -59,7 +59,7 @@
     end
 
     after do 
-      File.delete(@template) rescue {}
+      File.delete(@template) rescue nil
     end
 
     it "should compose class names" do 

diff --git a/spec/mtime_tracking_spec.rb b/spec/mtime_tracking_spec.rb
@@ -10,7 +10,7 @@
     end
 
     after do
-      File.delete(@img_dir + "/.mtimes") rescue {}
+      File.delete(@img_dir + "/.mtimes") rescue nil
     end
 
     it "tells me there are changes" do 
@@ -31,7 +31,7 @@
     end
 
     after do
-      File.delete(@img_dir + "/.mtimes") rescue {}
+      File.delete(@img_dir + "/.mtimes") rescue nil
     end
 
     it "tells me nothing has changed" do 
@@ -57,7 +57,7 @@
       end
 
       after do 
-        File.delete(@img_dir + "/.mtimes") rescue {}
+        File.delete(@img_dir + "/.mtimes") rescue nil
       end
 
       it "does not report excluded files as changed" do