Refactor Time#from_array and get rid of recursive call

Author: andrykonchin

src/main/java/org/truffleruby/core/time/TimeNodes.java

@@ -527,22 +527,6 @@ protected RubyTime timeSFromArray(
             return buildTime(language, timeClass, sec, min, hour, mday, month, year, nsec, isdst, isutc, utcoffset);
         }
 
-        @Specialization(guards = "!isInteger(sec) || !isInteger(nsec)")
-        protected Object timeSFromArrayFallback(
-                RubyClass timeClass,
-                Object sec,
-                int min,
-                int hour,
-                int mday,
-                int month,
-                int year,
-                Object nsec,
-                int isdst,
-                boolean isutc,
-                Object utcoffset) {
-            return FAILURE;
-        }
-
         @TruffleBoundary
         private RubyTime buildTime(RubyLanguage language, RubyClass timeClass, int sec, int min, int hour, int mday,
                 int month,

src/main/ruby/truffleruby/core/time.rb

@@ -380,9 +380,7 @@ def from_array(sec, min, hour, mday, month, year, nsec, is_dst, is_utc, utc_offs
       Primitive.rb_num2int(month)
       Primitive.rb_num2int(year)
 
-      time = Primitive.time_s_from_array(self, sec, min, hour, mday, month, year, nsec, is_dst, is_utc, utc_offset)
-      return time unless Primitive.undefined?(time)
-
+      # handle sec and nsec
       if Primitive.is_a?(sec, String)
         sec = sec.to_i
       elsif nsec
@@ -401,10 +399,8 @@ def from_array(sec, min, hour, mday, month, year, nsec, is_dst, is_utc, utc_offs
       end
 
       nsec ||= 0
-      sec += nsec / 1_000_000_000
-      nsec %= 1_000_000_000
 
-      from_array(sec, min, hour, mday, month, year, nsec, is_dst, is_utc, utc_offset)
+      Primitive.time_s_from_array(self, sec, min, hour, mday, month, year, nsec, is_dst, is_utc, utc_offset)
     end
     private :from_array