ENH: Removing obsolete python2 support

`#ifndef PY3K` conditionals and related logic Removal of PY3k ifdef ifndef not done correctly.
MeVisLab · hjmjohnson · Nov 26, 2024 · Jun 25, 2025 · Jun 25, 2025 · Jun 25, 2025
commit 317240f617b111dfc0c1ab5e8eb019c1163a1c92
diff --git a/src/PythonQt.cpp b/src/PythonQt.cpp
@@ -101,15 +101,13 @@ void PythonQt::init(int flags, const QByteArray& pythonQtModuleName)
       _self->_p->_pySourcelessFileLoader = importlib.getVariable("SourcelessFileLoader");
     }
 
-#ifdef PY3K
     PythonQtObjectPtr asyncio;
     asyncio.setNewRef(PyImport_ImportModule("asyncio"));
     if (asyncio)
     {
       _self->_p->_pyEnsureFuture = asyncio.getVariable("ensure_future");
       _self->_p->_pyFutureClass = asyncio.getVariable("Future");
     }
-#endif
 
     PythonQt::priv()->setupSharedLibrarySuffixes();
 
@@ -336,11 +334,8 @@ PythonQt::PythonQt(int flags, const QByteArray& pythonQtModuleName)
   _p->_initFlags = flags;
 
   if ((flags & PythonAlreadyInitialized) == 0) {
-#ifdef PY3K
     Py_SetProgramName(const_cast<wchar_t*>(L"PythonQt"));
-#else
-    Py_SetProgramName(const_cast<char*>("PythonQt"));
-#endif
+
     if (flags & IgnoreSiteModule) {
       // this prevents the automatic importing of Python site files
       Py_NoSiteFlag = 1;
@@ -430,7 +425,6 @@ void PythonQtPrivate::setTaskDoneCallback(const PythonQtObjectPtr & callable)
 PythonQtObjectPtr PythonQtPrivate::checkAndRunCoroutine(const PythonQtObjectPtr& object)
 {
   PythonQtObjectPtr result;
-#ifdef PY3K
   if (!PyCoro_CheckExact(object))
   {
     return result;
@@ -453,9 +447,6 @@ PythonQtObjectPtr PythonQtPrivate::checkAndRunCoroutine(const PythonQtObjectPtr&
     Py_XDECREF(methodName);
   }
   Py_XDECREF(args);
-#else
-  Q_UNUSED(object)
-#endif
   return result;
 }
 
@@ -989,11 +980,7 @@ QVariant PythonQt::evalCode(PyObject* object, PyObject* pycode) {
     }
     PyObject* r = nullptr;
     if (dict) {
-#ifdef PY3K
       r = PyEval_EvalCode(pycode, globals, dict);
-#else
-      r = PyEval_EvalCode((PyCodeObject*)pycode, globals, dict);
-#endif
     }
     if (r) {
       result = PythonQtConv::PyObjToQVariant(r);
@@ -1253,14 +1240,7 @@ QStringList PythonQt::introspectObject(PyObject* object, ObjectType type)
       keys = PyDict_Keys(object);
       isDict = true;
     } else {
-#if defined(MEVISLAB) && !defined(PY3K)
-      int oldPy3kWarningFlag = Py_Py3kWarningFlag;
-      Py_Py3kWarningFlag = 0;  // temporarily disable Python 3 warnings
-      keys = PyObject_Dir(object);
-      Py_Py3kWarningFlag = oldPy3kWarningFlag;
-#else
       keys = PyObject_Dir(object);
-#endif
     }
     if (keys) {
       int count = PyList_Size(keys);
@@ -1296,9 +1276,6 @@ QStringList PythonQt::introspectObject(PyObject* object, ObjectType type)
               && value->ob_type != &PyModule_Type
               && value->ob_type != &PyType_Type
               && value->ob_type != &PythonQtSlotFunction_Type
-#ifndef PY3K
-              && value->ob_type != &PyClass_Type
-#endif
               ) {
               results << keystr;
             }
@@ -1647,13 +1624,8 @@ int custom_system_exit_exception_handler()
 //    return exitcode;
 
   PyErr_Fetch(&exception, &value, &tb);
-#ifndef PY3K
-  if (Py_FlushLine()) {
-    PyErr_Clear();
-  }
-#else
+
   // TODO: unclear what to do, since Py_FlushLine is gone...
-#endif
   fflush(stdout);
   if (value == nullptr || value == Py_None)
     goto done;
@@ -1831,7 +1803,6 @@ static PyMethodDef PythonQtMethods[] = {
   {nullptr, nullptr, 0, nullptr}
 };
 
-#ifdef PY3K
 static PyModuleDef PythonQtModuleDef = {
   PyModuleDef_HEAD_INIT,
   "",
@@ -1843,20 +1814,15 @@ static PyModuleDef PythonQtModuleDef = {
   nullptr,
   nullptr
 };
-#endif
 
 void PythonQt::initPythonQtModule(bool redirectStdOut, const QByteArray& pythonQtModuleName)
 {
   QByteArray name = "PythonQt";
   if (!pythonQtModuleName.isEmpty()) {
     name = pythonQtModuleName;
   }
-#ifdef PY3K
   PythonQtModuleDef.m_name = name.constData();
   _p->_pythonQtModule = PyModule_Create(&PythonQtModuleDef);
-#else
-  _p->_pythonQtModule = Py_InitModule(name.constData(), PythonQtMethods);
-#endif
   _p->_pythonQtModuleName = name;
 
   Py_INCREF((PyObject*)&PythonQtBoolResult_Type);
@@ -1892,13 +1858,11 @@ void PythonQt::initPythonQtModule(bool redirectStdOut, const QByteArray& pythonQ
   }
   Py_XDECREF(old_module_names);
 
-#ifdef PY3K
   PyObject* modulesAttr = PyObject_GetAttrString(sys.object(), "modules");
   PyObject* pyUnicodeObject = PyUnicode_FromString(name.constData());
   PyDict_SetItem(modulesAttr, pyUnicodeObject, _p->_pythonQtModule.object());
   Py_XDECREF(modulesAttr);
   Py_XDECREF(pyUnicodeObject);
-#endif
 }
 
 QString PythonQt::getReturnTypeOfWrappedMethod(PyObject* module, const QString& name)
@@ -2232,9 +2196,6 @@ bool PythonQtPrivate::isMethodDescriptor(PyObject* object) const
       !PyObject_HasAttrString(object, "__set__") &&
       !PyMethod_Check(object) &&
       !PyFunction_Check(object)
-#ifndef PY3K
-      && !PyClass_Check(object)
-#endif
       ) {
     return true;
   }
@@ -2596,20 +2557,12 @@ void PythonQtPrivate::shellClassDeleted( void* shellClass )
 
 PyObject* PythonQtPrivate::wrapMemoryAsBuffer( const void* data, Py_ssize_t size )
 {
-#ifdef PY3K
   return PyMemoryView_FromMemory((char*)data, size, PyBUF_READ);
-#else
-  return PyBuffer_FromMemory((char*)data, size);
-#endif
 }
 
 PyObject* PythonQtPrivate::wrapMemoryAsBuffer( void* data, Py_ssize_t size )
 {
-#ifdef PY3K
   return PyMemoryView_FromMemory((char*)data, size, PyBUF_WRITE);
-#else
-  return PyBuffer_FromReadWriteMemory((char*)data, size);
-#endif
 }
 
 PythonQtClassInfo* PythonQtPrivate::getClassInfo( const QMetaObject* meta )

diff --git a/src/PythonQt.h b/src/PythonQt.h
@@ -147,15 +147,9 @@ typedef QObject* PythonQtQObjectCreatorFunctionCB();
 template<class T> QObject* PythonQtCreateObject() { return new T(); }
 
 //! Helper define to convert from QString to Python C-API
-#ifdef PY3K
 #define QStringToPythonConstCharPointer(arg) ((arg).toUtf8().constData())
 #define QStringToPythonCharPointer(arg) ((arg).toUtf8().data())
 #define QStringToPythonEncoding(arg) ((arg).toUtf8())
-#else
-#define QStringToPythonConstCharPointer(arg) ((arg).toLatin1().constData())
-#define QStringToPythonCharPointer(arg) ((arg).toLatin1().data())
-#define QStringToPythonEncoding(arg) ((arg).toLatin1())
-#endif
 
 //! The main interface to the Python Qt binding, realized as a singleton
 /*!

diff --git a/src/PythonQtBoolResult.cpp b/src/PythonQtBoolResult.cpp
@@ -69,9 +69,6 @@ static PyNumberMethods PythonQtBoolResult_as_number = {
   nullptr,      /* nb_add */
   nullptr,      /* nb_subtract */
   nullptr,      /* nb_multiply */
-#ifndef PY3K
-  nullptr,      /* nb_divide */
-#endif
   nullptr,      /* nb_remainder */
   nullptr,      /* nb_divmod */
   nullptr,      /* nb_power */
@@ -85,22 +82,12 @@ static PyNumberMethods PythonQtBoolResult_as_number = {
   nullptr,      /* nb_and */
   nullptr,      /* nb_xor */
   nullptr,      /* nb_or */
-#ifndef PY3K   
-  nullptr,      /* nb_coerce */
-#endif
   nullptr,      /* nb_int */
   nullptr,      /* nb_long  / nb_reserved in Py3K */
   nullptr,      /* nb_float */
-#ifndef PY3K
-  nullptr,      /* nb_oct */
-  nullptr,      /* nb_hex */
-#endif
   nullptr,      /* nb_inplace_add */
   nullptr,      /* nb_inplace_subtract */
   nullptr,      /* nb_inplace_multiply */
-#ifndef PY3K
-  nullptr,      /* nb_inplace_divide */
-#endif
   nullptr,      /* nb_inplace_remainder */
   nullptr,      /* nb_inplace_power */
   nullptr,      /* nb_inplace_lshift */
@@ -112,9 +99,7 @@ static PyNumberMethods PythonQtBoolResult_as_number = {
   nullptr,      /* nb_true_divide */
   nullptr,      /* nb_inplace_floor_divide */
   nullptr,      /* nb_inplace_true_divide */
-#ifdef PY3K
   nullptr,      /* nb_index in Py3K */
-#endif
 };
 
 PyTypeObject PythonQtBoolResult_Type = {
@@ -155,4 +140,3 @@ PyTypeObject PythonQtBoolResult_Type = {
     0,            /* tp_dictoffset */
     (initproc)&PythonQtBoolResult_init,      /* tp_init */
 };
-
diff --git a/src/PythonQtClassWrapper.cpp b/src/PythonQtClassWrapper.cpp
@@ -260,9 +260,6 @@ static void initializeSlots(PythonQtClassWrapper* wrap)
       wrap->_base.as_number.nb_multiply = (binaryfunc)PythonQtInstanceWrapper_mul;
     }
     if (typeSlots & PythonQt::Type_Divide) {
-#ifndef PY3K
-      wrap->_base.as_number.nb_divide = (binaryfunc)PythonQtInstanceWrapper_div;
-#endif
       wrap->_base.as_number.nb_true_divide = (binaryfunc)PythonQtInstanceWrapper_div;
     }
     if (typeSlots & PythonQt::Type_And) {
@@ -294,9 +291,6 @@ static void initializeSlots(PythonQtClassWrapper* wrap)
       wrap->_base.as_number.nb_inplace_multiply = (binaryfunc)PythonQtInstanceWrapper_imul;
     }
     if (typeSlots & PythonQt::Type_InplaceDivide) {
-#ifndef PY3K
-      wrap->_base.as_number.nb_inplace_divide = (binaryfunc)PythonQtInstanceWrapper_idiv;
-#endif
       wrap->_base.as_number.nb_inplace_true_divide = (binaryfunc)PythonQtInstanceWrapper_idiv;
     }
     if (typeSlots & PythonQt::Type_InplaceAnd) {
@@ -321,11 +315,7 @@ static void initializeSlots(PythonQtClassWrapper* wrap)
       wrap->_base.as_number.nb_invert = (unaryfunc)PythonQtInstanceWrapper_invert;
     }
     if (typeSlots & PythonQt::Type_NonZero) {
-#ifdef PY3K
       wrap->_base.as_number.nb_bool = (inquiry)PythonQtInstanceWrapper_nonzero;
-#else
-      wrap->_base.as_number.nb_nonzero = (inquiry)PythonQtInstanceWrapper_nonzero;
-#endif
     }
   }
 }
@@ -564,11 +554,7 @@ static PyObject *PythonQtClassWrapper_getattro(PyObject *obj, PyObject *name)
   }
 
   // look for the internal methods (className(), help())
-#ifdef PY3K
   PyObject* internalMethod = PyObject_GenericGetAttr(obj, name);
-#else
-  PyObject* internalMethod = Py_FindMethod( PythonQtClassWrapper_methods, obj, (char*)attributeName);
-#endif
   if (internalMethod) {
     return internalMethod;
   }
@@ -635,11 +621,7 @@ PyTypeObject PythonQtClassWrapper_Type = {
     0,                               /* tp_weaklistoffset */
     nullptr,                         /* tp_iter */
     nullptr,                         /* tp_iternext */
-    #ifdef PY3K
     PythonQtClassWrapper_methods,    /* tp_methods */
-    #else
-    nullptr,                         /* tp_methods */
-    #endif
     nullptr,                         /* tp_members */
     nullptr,                         /* tp_getset */
     nullptr,                         /* tp_base */
@@ -654,4 +636,3 @@ PyTypeObject PythonQtClassWrapper_Type = {
 };
 
 //-------------------------------------------------------
-