Use the ID string for comparison in clock_gettime logic.

headius · headius · commit 758cea51598e · 2018-06-29T20:28:40.000-05:00
diff --git a/core/src/main/java/org/jruby/RubyProcess.java b/core/src/main/java/org/jruby/RubyProcess.java
@@ -1419,15 +1419,13 @@ public static IRubyObject times(Ruby runtime) {
                 Block.NULL_BLOCK);
     }
 
-    // this is only in 2.1. See https://bugs.ruby-lang.org/issues/8658
     @JRubyMethod(module = true, visibility = PRIVATE)
     public static IRubyObject clock_gettime(ThreadContext context, IRubyObject self, IRubyObject _clock_id) {
         Ruby runtime = context.runtime;
 
         return makeClockResult(runtime, getTimeForClock(_clock_id, runtime), CLOCK_UNIT_FLOAT_SECOND);
     }
 
-    // this is only in 2.1. See https://bugs.ruby-lang.org/issues/8658
     @JRubyMethod(module = true, visibility = PRIVATE)
     public static IRubyObject clock_gettime(ThreadContext context, IRubyObject self, IRubyObject _clock_id, IRubyObject _unit) {
         Ruby runtime = context.runtime;
@@ -1451,9 +1449,10 @@ private static long getTimeForClock(IRubyObject _clock_id, Ruby runtime) throws
         long nanos;
 
         if (_clock_id instanceof RubySymbol) {
-            if (_clock_id.toString().equals(CLOCK_MONOTONIC)) {
+            RubySymbol clock_id = (RubySymbol) _clock_id;
+            if (clock_id.idString().equals(CLOCK_MONOTONIC)) {
                 nanos = System.nanoTime();
-            } else if (_clock_id.toString().equals(CLOCK_REALTIME)) {
+            } else if (clock_id.idString().equals(CLOCK_REALTIME)) {
                 POSIX posix = runtime.getPosix();
                 if (posix.isNative()) {
                     Timeval tv = posix.allocateTimeval();
@@ -1479,9 +1478,10 @@ private static long getResolutionForClock(IRubyObject _clock_id, Ruby runtime) t
         long nanos;
 
         if (_clock_id instanceof RubySymbol) {
-            if (_clock_id.toString().equals(CLOCK_MONOTONIC)) {
+            RubySymbol clock_id = (RubySymbol) _clock_id;
+            if (clock_id.idString().equals(CLOCK_MONOTONIC)) {
                 nanos = 1;
-            } else if (_clock_id.toString().equals(CLOCK_REALTIME)) {
+            } else if (clock_id.idString().equals(CLOCK_REALTIME)) {
                 nanos = 1000000;
             } else {
                 throw runtime.newErrnoEINVALError("clock_gettime");
