From: Kai Koehne <kai.koehne@nokia.com>
Date: Tue, 24 Apr 2012 13:17:24 +0000 (+0200)
Subject: Fix usage of QMutexLocker
X-Git-Tag: 071012131707~548
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2a31da23b459cee39aa26b4384df04daf3d33954;p=profile%2Fivi%2Fqtdeclarative.git

Fix usage of QMutexLocker

Not giving QMutexLocker a name means that it's lifetime is
not guaranteed to be bound to the current scope.

Change-Id: I6e58b0aab814c5e686fac3ff0aa6b671e17db240
Reviewed-by: Aaron Kennedy <aaron.kennedy@nokia.com>
---

diff --git a/src/qml/qml/qquickworkerscript.cpp b/src/qml/qml/qquickworkerscript.cpp
index e689f9d..b00847a 100644
--- a/src/qml/qml/qquickworkerscript.cpp
+++ b/src/qml/qml/qquickworkerscript.cpp
@@ -282,7 +282,7 @@ v8::Handle<v8::Value> QQuickWorkerScriptEnginePrivate::sendMessage(const v8::Arg
 
     QByteArray data = QV8Worker::serialize(args[2], engine);
 
-    QMutexLocker(&engine->p->m_lock);
+    QMutexLocker locker(&engine->p->m_lock);
     WorkerScript *script = engine->p->workers.value(id);
     if (!script)
         return v8::Undefined();
@@ -405,7 +405,7 @@ void QQuickWorkerScriptEnginePrivate::reportScriptException(WorkerScript *script
 {
     QQuickWorkerScriptEnginePrivate *p = QQuickWorkerScriptEnginePrivate::get(workerEngine);
 
-    QMutexLocker(&p->m_lock);
+    QMutexLocker locker(&p->m_lock);
     if (script->owner)
         QCoreApplication::postEvent(script->owner, new WorkerErrorEvent(error));
 }