diff options
| author | Christoph Reiter <reiter.christoph@gmail.com> | 2018-02-08 09:02:37 +0100 |
|---|---|---|
| committer | Christoph Reiter <reiter.christoph@gmail.com> | 2018-02-08 09:11:16 +0100 |
| commit | d371fd77d33c2987cc450bd68ce4ee4835e09bd8 (patch) | |
| tree | 5b2c84c2d985b6cf6520c729d89b5dc152d55c4a /tests/test_everything.py | |
| parent | 024ac583538976330b429b9b203d412fec0426e3 (diff) | |
| download | pygobject-d371fd77d33c2987cc450bd68ce4ee4835e09bd8.tar.gz | |
Revert "to-py-struct: don't copy the boxed if we are the sole owner of the wrapper"
This reverts commit a4a68b3c7a4bb64c71e1bdbe52ede109b6bcd0bb.
This was wrong. In the context of a boxed return value we also have only one
ref but it doesn't get freed after.
We could decide this based on PYGI_CALLING_CONTEXT_IS_FROM_C from the callable cache,
but let's leave that for now.
Diffstat (limited to 'tests/test_everything.py')
| -rw-r--r-- | tests/test_everything.py | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/tests/test_everything.py b/tests/test_everything.py index b7fcb2af..e206954a 100644 --- a/tests/test_everything.py +++ b/tests/test_everything.py @@ -1129,7 +1129,6 @@ class TestBoxed(unittest.TestCase): self.assertEqual(boxed, copy) self.assertNotEqual(id(boxed), id(copy)) - @unittest.expectedFailure def test_boxed_c_wrapper(self): wrapper = Everything.TestBoxedCWrapper() obj = wrapper.get() @@ -1142,7 +1141,6 @@ class TestBoxed(unittest.TestCase): del wrapper self.assertEqual(obj.refcount, 1) - @unittest.expectedFailure def test_boxed_c_wrapper_copy(self): wrapper = Everything.TestBoxedCWrapper() wrapper_copy = wrapper.copy() |