Set up remaining converters as postfix rescue for now.

@enebo improved our runtime to omit stack traces downstream from simple postfix rescues, but simple long-form rescues don't get that optimization yet. This is a temporary measure until they do.
jruby · Sep 29, 2015 · 16dcafe · 16dcafe
1 parent 6bd67c2
commit 16dcafe
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/lib/ruby/stdlib/csv.rb b/lib/ruby/stdlib/csv.rb
@@ -955,17 +955,13 @@ def self.date(f)
       begin
         e = f.encode(ConverterEncoding)
         e =~ DateMatcher ? Date.parse(e) : f
-      rescue  # encoding conversion or date parse errors
-        f
-      end
+      end rescue r # encoding conversion or date parse errors
     end
     def self.date_time(f)
       begin
         e = f.encode(ConverterEncoding)
         e =~ DateTimeMatcher ? DateTime.parse(e) : f
-      rescue  # encoding conversion or date parse errors
-        f
-      end
+      end rescue r # encoding conversion or date parse errors
     end
   end
   Converters  = { integer:   converter_methods.method(:integer),