From cbff393dc1c60916d6bfe00ea28769a271160118 Mon Sep 17 00:00:00 2001 From: Ladislav Michl Date: Sun, 25 Feb 2024 18:24:27 +0100 Subject: [PATCH] Gui: PythonWrapper: Use getPyTypeObjectForTypeName consistently --- src/Gui/PythonWrapper.cpp | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/Gui/PythonWrapper.cpp b/src/Gui/PythonWrapper.cpp index a77115e2f6..56f18d2b87 100644 --- a/src/Gui/PythonWrapper.cpp +++ b/src/Gui/PythonWrapper.cpp @@ -576,9 +576,9 @@ QGraphicsObject* PythonWrapper::toQGraphicsObject(const Py::Object& pyobject) Py::Object PythonWrapper::fromQImage(const QImage& img) { #if defined (HAVE_SHIBOKEN) && defined(HAVE_PYSIDE) - PyObject* pyobj = Shiboken::Conversions::copyToPython(getPyTypeObjectForTypeName(), - const_cast(&img)); - if (pyobj) { + auto type = getPyTypeObjectForTypeName(); + if (type) { + PyObject* pyobj = Shiboken::Conversions::copyToPython(type, const_cast(&img)); return Py::asObject(pyobj); } #else @@ -600,9 +600,9 @@ Py::Object PythonWrapper::fromQIcon(const QIcon* icon) { #if defined (HAVE_SHIBOKEN) && defined(HAVE_PYSIDE) const char* typeName = typeid(*const_cast(icon)).name(); - PyObject* pyobj = Shiboken::Object::newObject(getPyTypeObjectForTypeName(), - const_cast(icon), true, false, typeName); - if (pyobj) { + auto type = getPyTypeObjectForTypeName(); + if (type) { + PyObject* pyobj = Shiboken::Object::newObject(type, const_cast(icon), true, false, typeName); return Py::asObject(pyobj); } #else @@ -624,9 +624,9 @@ Py::Object PythonWrapper::fromQDir(const QDir& dir) { #if defined (HAVE_SHIBOKEN) && defined(HAVE_PYSIDE) const char* typeName = typeid(dir).name(); - PyObject* pyobj = Shiboken::Object::newObject(getPyTypeObjectForTypeName(), - const_cast(&dir), false, false, typeName); - if (pyobj) { + auto type = getPyTypeObjectForTypeName(); + if (type) { + PyObject* pyobj = Shiboken::Object::newObject(type, const_cast(&dir), false, false, typeName); return Py::asObject(pyobj); } #else