diff options
| author | Christian Tismer <tismer@stackless.com> | 2024-05-09 22:43:59 +0200 |
|---|---|---|
| committer | Christian Tismer <tismer@stackless.com> | 2024-05-10 14:09:42 +0200 |
| commit | 3e1cbc0a0cc3961564c5161fad775c450068c8fb (patch) | |
| tree | cebde1768e8c111eff9b4613983682f200eba3b2 /sources/pyside6/PySide6/QtQml/pysideqmlvolatilebool.cpp | |
| parent | 2176967547d1ea64aeeb396cd5bd25af301cf4a0 (diff) | |
Shiboken: Simplify Python Error Messages
The function PyErr_SetString is used quite often, which has
no return value. The bracketed sequence
PyErr_SetString(...);
return nullptr;
can in most cases be replaced by a single call to
return PyErr_Format(...);
To simplify matters, PyErr_Format is now used everywhere.
Task-number: PYSIDE-2404
Change-Id: I5988fcd2430be700415d14d7a5cc740211e61d08
Pick-to: 6.7
Reviewed-by: Adrian Herrmann <adrian.herrmann@qt.io>
Diffstat (limited to 'sources/pyside6/PySide6/QtQml/pysideqmlvolatilebool.cpp')
| -rw-r--r-- | sources/pyside6/PySide6/QtQml/pysideqmlvolatilebool.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/sources/pyside6/PySide6/QtQml/pysideqmlvolatilebool.cpp b/sources/pyside6/PySide6/QtQml/pysideqmlvolatilebool.cpp index 6e403ab72..52a45e026 100644 --- a/sources/pyside6/PySide6/QtQml/pysideqmlvolatilebool.cpp +++ b/sources/pyside6/PySide6/QtQml/pysideqmlvolatilebool.cpp @@ -63,10 +63,8 @@ QtQml_VolatileBoolObject_set(QtQml_VolatileBoolObject *self, PyObject *args) } ok = PyObject_IsTrue(value); - if (ok < 0) { - PyErr_SetString(PyExc_TypeError, "Not a boolean value."); - return nullptr; - } + if (ok < 0) + return PyErr_Format(PyExc_TypeError, "Not a boolean value."); *self->flag = ok > 0; |