Reorder checks to match CRuby behavior

headius · Feb 14, 2025 · 7109a58 · 7109a58
1 parent b6971dc
commit 7109a58
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/core/src/main/java/org/jruby/ext/fiber/ThreadFiber.java b/core/src/main/java/org/jruby/ext/fiber/ThreadFiber.java
@@ -381,12 +381,10 @@ public IRubyObject transfer(ThreadContext context, IRubyObject[] values) {
 
         return processResultData(context, result);
     }
+
     public IRubyObject raise(ThreadContext context, IRubyObject exception) {
         Ruby runtime = context.runtime;
 
-        final FiberData data = this.data;
-        if (data.prev != null) throw runtime.newFiberError("double resume");
-
         if (!alive()) throw runtime.newFiberError("dead fiber called");
 
         FiberData currentFiberData = context.getFiber().data;
@@ -395,6 +393,9 @@ public IRubyObject raise(ThreadContext context, IRubyObject exception) {
             RubyKernel.raise(context, this, exception);
         }
 
+        final FiberData data = this.data;
+        if (data.prev != null) throw runtime.newFiberError("double resume");
+
         if (data.parent != context.getFiberCurrentThread()) fiberCalledAcrossThreads(runtime);
 
         FiberRequest val = new FiberRequest(((RubyException) exception).toThrowable(), RequestType.RAISE);