Merge pull request #155 from mrkn/fix_without_gvl_for_exception

Fix without_gvl for exceptions occurred in a block
mrkn · Feb 10, 2022 · f7cde02 · f7cde02
2 parents 69cf4dd + a6fea92
commit f7cde02
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/ext/pycall/pycall.c b/ext/pycall/pycall.c
@@ -107,6 +107,10 @@ pycall_without_gvl(VALUE (* func)(VALUE), VALUE arg)
 
   pycall_set_with_gvl();
 
+  if (state) {
+    rb_jump_tag(state);
+  }
+
   return result;
 }
 

diff --git a/test/pycall/test-gvl.rb b/test/pycall/test-gvl.rb
@@ -0,0 +1,10 @@
+class PyCallGvlTest < Test::Unit::TestCase
+  def test_exception
+    math = PyCall.import_module("math")
+    assert_raise_message(/factorial\(\) not defined for negative values/) do
+      PyCall.without_gvl do
+        math.factorial(-1)
+      end
+    end
+  end
+end