Fixed module import for python 3.8

Fixed import module for windows and python 3.7

.codespellrc

@@ -1,4 +1,4 @@
 [codespell]
-skip = ./build-*,.git
+skip = ./build-*,.git,*.svg,rc_assets.py
 interactive = 3
 ignore-words-list = overlay,overlayed

examples/dockwidgets/MyTitleBar_CSS.h

@@ -9,6 +9,11 @@
   Contact KDAB at <info@kdab.com> for commercial licensing options.
 */
 
+#ifndef EXAMPLETITLEBAR_CSS_H
+#define EXAMPLETITLEBAR_CSS_H
+
+#pragma once
+
 #include <kddockwidgets/private/widgets/TitleBarWidget_p.h>
 
 /**
@@ -77,3 +82,5 @@ public:
 MyTitleBar_CSS::~MyTitleBar_CSS()
 {
 }
+
+#endif

python/PyKDDockWidgets/__init__.py.cmake

@@ -15,15 +15,22 @@ import os
 __all__ = ['KDDockWidgets']
 
 def setupLibraryPath():
-    package_dir = os.path.abspath(os.path.dirname(__file__))
     if sys.platform != 'win32':
         return
 
-    if sys.version_info[0] == 3 and sys.version_info[1] >= 8:
+    from shiboken2 import shiboken2
-        os.add_dll_directory(package_dir)
+    from PySide@PYSIDE_MAJOR_VERSION@ import QtCore
-        return
 
-    os.environ['PATH'] = os.fspath(package_dir) + os.pathsep + os.environ['PATH']
+    extra_dll_dirs = [ os.path.abspath(os.path.dirname(shiboken2.__file__)),
+                       os.path.abspath(os.path.dirname(QtCore.__file__)),
+                       os.path.abspath(os.path.dirname(__file__)) ]
+
+    if sys.version_info[0] == 3 and sys.version_info[1] >= 8:
+        for dll_dir in extra_dll_dirs:
+            os.add_dll_directory(dll_dir)
+
+    for dll_dir in extra_dll_dirs:
+        os.environ['PATH'] = os.fspath(dll_dir) + os.pathsep + os.environ['PATH']
 
 # Preload PySide libraries to avoid missing libraries while loading KDDockWidgets
 try:
@@ -36,15 +43,3 @@ except Exception:
     raise
 
 setupLibraryPath()
-
-
-#
-# This file is part of KDDockWidgets.
-#
-# SPDX-FileCopyrightText: 2020-2021 Klarälvdalens Datakonsult AB, a KDAB Group company <info@kdab.com>
-# Author: Renato Araujo Oliveira Filho <renato.araujo@kdab.com>
-#
-# SPDX-License-Identifier: GPL-2.0-only OR GPL-3.0-only
-#
-# Contact KDAB at <info@kdab.com> for commercial licensing options.
-#

python/tests/config.py.cmake

@@ -1,2 +1,19 @@
+# This file is part of KDDockWidgets.
+#
+# SPDX-FileCopyrightText: 2021 Klarälvdalens Datakonsult AB, a KDAB Group company <info@kdab.com>
+# Author: Renato Araujo <renato.araujo@kdab.com>
+#
+# SPDX-License-Identifier: GPL-2.0-only OR GPL-3.0-only
+#
+# Contact KDAB at <info@kdab.com> for commercial licensing options.
+#
+
+import os
+import sys
+
 class TstConfig(object):
     bindingsNamespace = "@PYTHON_BINDING_NAMESPACE@"
+
+    def initLibraryPath():
+        if sys.platform == 'win32' and sys.version_info[0] == 3 and sys.version_info[1] >= 8:
+            os.add_dll_directory("@CMAKE_RUNTIME_OUTPUT_DIRECTORY@")

python/tests/tst_importModule.py

@@ -1,3 +1,13 @@
+# This file is part of KDDockWidgets.
+#
+# SPDX-FileCopyrightText: 2021 Klarälvdalens Datakonsult AB, a KDAB Group company <info@kdab.com>
+# Author: Renato Araujo <renato.araujo@kdab.com>
+#
+# SPDX-License-Identifier: GPL-2.0-only OR GPL-3.0-only
+#
+# Contact KDAB at <info@kdab.com> for commercial licensing options.
+#
+
 import unittest
 import importlib
 import inspect
@@ -16,4 +26,5 @@ class TestImportModules(unittest.TestCase):
             self.assertIn(symbol, moduleSymbols)
 
 if __name__ == '__main__':
+    TstConfig.initLibraryPath()
     unittest.main()

src/private/FloatingWindow_p.h

@@ -205,7 +205,7 @@ public:
 
     ///@brief By default equivalent to QWidget::normalGeometry()
     /// Derived classes can implement something different here, to workaround window manager issues with Qt::Tool
-    /// Also useful for QtQuick to eventually perserve normal geometry uppon save/restore of a maximized window. As
+    /// Also useful for QtQuick to eventually preserve normal geometry upon save/restore of a maximized window. As
     /// QWindow has no notion of normal geometry, so we need to implement it here.
     /// @sa QTBUG-95478
     virtual QRect normalGeometry() const;

src/private/widgets/TabWidgetWidget.cpp

@@ -212,7 +212,7 @@ void TabWidgetWidget::showContextMenu(QPoint pos)
         return;
 
     QTabBar *tabBar = QTabWidget::tabBar();
-    // We dont want context menu if there is only one tab
+    // We don't want context menu if there is only one tab
     if (tabBar->count() <= 1)
         return;
 

tests/fuzzer/Operations.cpp

@@ -27,9 +27,9 @@ using namespace KDDockWidgets;
 using namespace KDDockWidgets::Testing;
 using namespace KDDockWidgets::Testing::Operations;
 
-static QString operationTypeStr(OperationType ot)
+static QString operationTypeStr(OperationType optype)
 {
-    return QMetaEnum::fromType<OperationType>().valueToKey(ot);
+    return QMetaEnum::fromType<OperationType>().valueToKey(optype);
 }
 
 OperationBase::OperationBase(KDDockWidgets::Testing::Operations::OperationType type, Fuzzer *fuzzer)