Simplify the test case.

Apparently, StringIO() objects are serialized.

Author: ZeroIntensity

Lib/test/test_interpreters/test_api.py

@@ -2416,14 +2416,10 @@ def test_object_collected_after_proxy_closes(self):
         buffer = io.StringIO()
         interp = interpreters.create()
         try:
-            def write_to_proxy(obj):
-                obj.file.write("hello")
+            with self.share(self.unshareable(buffer)):
+                pass
 
-            with self.share(self.unshareable(buffer)) as proxy:
-                self.assertIsNone(interp.call(write_to_proxy, proxy))
-                self.assertEqual(str(proxy.file), buffer)
-
-            self.assertEqual(buffer.getvalue(), "hellodone")
+            self.assertEqual(buffer.getvalue(), "done")
         finally:
             interp.close()
 

Modules/_interpretersmodule.c

@@ -805,10 +805,15 @@ _sharedobjectproxy_as_shared(_PyXI_proxy_share *share)
 static void
 _sharedobjectproxy_finish_share(_PyXI_proxy_share *share)
 {
-    _PyXIData_Free(share->xidata);
+    if (share->xidata != NULL) {
+        _PyXIData_Free(share->xidata);
+    }
 #ifdef Py_DEBUG
     share->xidata = NULL;
-    share->object = NULL;
+    if (share->object != NULL) {
+        assert(PyUnstable_IsImmortal(share->object));
+        share->object = NULL;
+    }
 #endif
 }
 
@@ -836,7 +841,7 @@ _sharedobjectproxy_wrap_result(SharedObjectProxy *self, PyObject *result,
     }
 
     PyObject *ret = _sharedobjectproxy_as_shared(&shared_result);
-    _sharedobjectproxy_finish_share(&shared_result);
+    //_sharedobjectproxy_finish_share(&shared_result);
     return ret;
 }