Port from QMutex::Recursive to QRecursiveMutex

Change-Id: I5bf128b4479971e87d377707f2ebf267ccba1f1d
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>

src/qml/debugger/qqmlconfigurabledebugservice_p.h

@@ -64,7 +64,7 @@ class QQmlConfigurableDebugService : public Base
 {
 protected:
     QQmlConfigurableDebugService(float version, QObject *parent = nullptr) :
-        Base(version, parent), m_configMutex(QMutex::Recursive)
+        Base(version, parent)
     {
         init();
     }
@@ -103,7 +103,7 @@ protected:
             emit Base::attachedToEngine(engine);
     }
 
-    QMutex m_configMutex;
+    QRecursiveMutex m_configMutex;
     QList<QJSEngine *> m_waitingEngines;
     bool m_waitingForConfiguration;
 };

src/qml/jsapi/qjsengine_p.h

@@ -74,7 +74,7 @@ public:
     static const QJSEnginePrivate* get(const QJSEngine*e) { return e->d_func(); }
     static QJSEnginePrivate* get(QV4::ExecutionEngine *e);
 
-    QJSEnginePrivate() : mutex(QMutex::Recursive) {}
+    QJSEnginePrivate() = default;
     ~QJSEnginePrivate() override;
 
     static void addToDebugServer(QJSEngine *q);
@@ -105,7 +105,7 @@ public:
     };
 
     // Shared by QQmlEngine
-    mutable QMutex mutex;
+    mutable QRecursiveMutex mutex;
 
 
     // These methods may be called from the QML loader thread

src/qml/jsruntime/qv4executableallocator.cpp

@@ -147,9 +147,7 @@ bool ExecutableAllocator::ChunkOfPages::contains(Allocation *alloc) const
 }
 
 ExecutableAllocator::ExecutableAllocator()
-    : mutex(QMutex::NonRecursive)
-{
-}
+    = default;
 
 ExecutableAllocator::~ExecutableAllocator()
 {

src/qml/jsruntime/qv4executableallocator_p.h

@@ -130,7 +130,7 @@ public:
 private:
     QMultiMap<size_t, Allocation*> freeAllocations;
     QMap<quintptr, ChunkOfPages*> chunks;
-    mutable QMutex mutex;
+    mutable QRecursiveMutex mutex;
 };
 
 }

src/qml/qml/qqmlmetatype.cpp

@@ -60,7 +60,7 @@ struct LockedData : private QQmlMetaTypeData
 };
 
 Q_GLOBAL_STATIC(LockedData, metaTypeData)
-Q_GLOBAL_STATIC_WITH_ARGS(QMutex, metaTypeDataLock, (QMutex::Recursive))
+Q_GLOBAL_STATIC(QRecursiveMutex, metaTypeDataLock)
 
 class QQmlMetaTypeDataPtr
 {
@@ -804,7 +804,7 @@ QQmlType QQmlMetaType::typeForUrl(const QString &urlString,
     return QQmlType();
 }
 
-QMutex *QQmlMetaType::typeRegistrationLock()
+QRecursiveMutex *QQmlMetaType::typeRegistrationLock()
 {
     return metaTypeDataLock();
 }

src/qml/qml/qqmlmetatype_p.h

@@ -58,7 +58,7 @@
 QT_BEGIN_NAMESPACE
 
 class QQmlTypeModule;
-class QMutex;
+class QRecursiveMutex;
 class QQmlError;
 
 namespace QV4 { class ExecutableCompilationUnit; }
@@ -160,7 +160,7 @@ public:
     static void prependCachedUnitLookupFunction(QQmlPrivate::QmlUnitCacheLookupFunction handler);
     static void removeCachedUnitLookupFunction(QQmlPrivate::QmlUnitCacheLookupFunction handler);
 
-    static QMutex *typeRegistrationLock();
+    static QRecursiveMutex *typeRegistrationLock();
 
     static QString prettyTypeName(const QObject *object);