Fix a bug causing coercion to HeldClass instances to crash.

Author: braxtonmckee

typed_python/PyInstance.hpp

@@ -302,7 +302,7 @@ class PyInstance {
     static PyObject* initializePythonRepresentation(Type* eltType, const init_func& f) {
         Instance instance(eltType, f);
 
-        return extractPythonObject(instance.data(), instance.type());
+        return extractPythonObject(instance.data(), instance.type(), false);
     }
 
     //initialize a PyInstance for 'eltType'. For ints, floats, etc, with

typed_python/compiler/tests/held_class_interpreter_semantics_test.py

@@ -4,7 +4,7 @@
 import time
 from flaky import flaky
 
-from typed_python import Class, Final, ListOf, Held, Member, PointerTo, pointerTo
+from typed_python import Class, Final, ListOf, Held, Member, PointerTo, pointerTo, Function
 from typed_python.test_util import currentMemUsageMb
 
 
@@ -22,6 +22,20 @@ def increment(self):
 
 
 class TestHeldClassInterpreterSemantics(unittest.TestCase):
+    def test_can_return_held_class_from_typed_function(self):
+        @Function
+        def f(x, y) -> H:
+            return H(x=x, y=y)
+
+        res = f(10, 20)
+        res2 = f(12, 22)
+
+        assert res.x == 10
+        assert res.y == 20
+
+        assert res2.x == 12
+        assert res2.y == 22
+
     def test_list_of_held_class_item_type(self):
         assert sys.gettrace() is None