aboutsummaryrefslogtreecommitdiffstats
path: root/examples/widgets/animation
diff options
context:
space:
mode:
authorChristian Tismer <tismer@stackless.com>2021-04-18 18:58:18 +0200
committerChristian Tismer <tismer@stackless.com>2022-01-24 19:29:23 +0100
commit3c3595e1e196204bfaec8c5f82db7f0fc84a1702 (patch)
treed6f9f64af8feb64c4d09960add362b806dea19b3 /examples/widgets/animation
parent9b4d1f76128bc46a5ea85d33ce67f81029b8017e (diff)
PyPySide: handle QPainter correctly with GC
While testing the examples with PyPy, a number of examples used QPainter without explicitly calling painter.end() and crashed. This works in standard Python, but leaves the painter open in other implementations, because the implicit deletion of objects when going out of scope does not work in the wrapper when garbage collection is used. Fixed by providing the missing painter.end() calls. This problem should finally be fixed by making QPainter a context manager. The same approach was taken by Python.org and the file open/close functions. The context manager was needed for implementations like IronPython, Jython and PyPy. [ChangeLog][PySide6] The examples were adapted to PyPy's need to close QPainter, explicitly. Eventually, we may turn this into a context manager. Change-Id: I18eeeff7df800bafce91a1e5c98c469aa3bcc41b Pick-to: 6.2 Task-number: PYSIDE-535 Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io> Reviewed-by: Cristian Maureira-Fredes <cristian.maureira-fredes@qt.io>
Diffstat (limited to 'examples/widgets/animation')
-rw-r--r--examples/widgets/animation/easing/easing.py1
1 files changed, 1 insertions, 0 deletions
diff --git a/examples/widgets/animation/easing/easing.py b/examples/widgets/animation/easing/easing.py
index ba7f2d363..a7bff7842 100644
--- a/examples/widgets/animation/easing/easing.py
+++ b/examples/widgets/animation/easing/easing.py
@@ -218,6 +218,7 @@ class Window(QWidget):
item.setText(curve_name)
self._ui.easingCurvePicker.addItem(item)
+ # QPainter needs an explicit end() in PyPy. This will become a context manager in 6.3.
painter.end()
def start_animation(self):