Fix rb_time_nano_new - don't call Time.at directly

Author: andrykonchin

CHANGELOG.md

@@ -23,6 +23,7 @@ Bug fixes:
 * Fix `\P{}` matching in regular expressions (#2798, @andrykonchin).
 * Fix constants lookup when `BasicObject#instance_eval` method is called with a String (#2810, @andrykonchin).
 * Don't trigger the `method_added` event when changing a method's visibility or calling `module_function` (@paracycle, @nirvdrum).
+* Fix `rb_time_timespec_new` function to not call `Time.at` method directly (@andrykonchin).
 
 Compatibility:
 

lib/truffle/truffle/cext.rb

@@ -112,6 +112,8 @@ module Truffle::CExt
     LIBTRUFFLERUBY = libtruffleruby
   end
 
+  ORIGINAL_TIME_AT = Time.method(:at)
+
   def self.register_libtruffleruby(libtruffleruby)
     SET_LIBTRUFFLERUBY.call(libtruffleruby)
   end
@@ -1673,21 +1675,21 @@ def warning?
   end
 
   def rb_time_nano_new(sec, nsec)
-    Time.at sec, nsec, :nanosecond
+    ORIGINAL_TIME_AT.call(sec, nsec, :nanosecond)
   end
 
   def rb_time_timespec_new(sec, nsec, offset, is_utc, is_local)
     if is_local
-      Time.at(sec, nsec, :nanosecond)
+      ORIGINAL_TIME_AT.call(sec, nsec, :nanosecond)
     elsif is_utc
-      Time.at(sec, nsec, :nanosecond, in: 0).getgm
+      ORIGINAL_TIME_AT.call(sec, nsec, :nanosecond, in: 0).getgm
     else
-      Time.at(sec, nsec, :nanosecond, in: offset)
+      ORIGINAL_TIME_AT.call(sec, nsec, :nanosecond, in: offset)
     end
   end
 
   def rb_time_num_new(timev, off)
-    Time.at(timev).getlocal(off)
+    ORIGINAL_TIME_AT.call(timev).getlocal(off)
   end
 
   def rb_time_interval_acceptable(time_val)

spec/ruby/optional/capi/time_spec.rb

@@ -283,6 +283,11 @@
       -> { @s.rb_time_timespec_new(1447087832, 476451125, 86400) }.should raise_error(ArgumentError)
       -> { @s.rb_time_timespec_new(1447087832, 476451125, -86400) }.should raise_error(ArgumentError)
     end
+
+    it "doesn't call Time.at directly" do
+      Time.should_not_receive(:at)
+      @s.rb_time_timespec_new(1447087832, 476451125, 32400).should be_kind_of(Time)
+    end
   end
 
   describe "rb_timespec_now" do