fix locker issue

libraries/recording/src/recording/RecordingScriptingInterface.cpp

@@ -38,33 +38,33 @@ using namespace recording;
 static const QString HFR_EXTENSION = "hfr";
 
 RecordingScriptingInterface::RecordingScriptingInterface() {
-    Locker(_mutex);
+    Locker lock(_mutex);
     _player = DependencyManager::get<Deck>();
     _recorder = DependencyManager::get<Recorder>();
 }
 
 bool RecordingScriptingInterface::isPlaying() const {
-    Locker(_mutex);
+    Locker lock(_mutex);
     return _player->isPlaying();
 }
 
 bool RecordingScriptingInterface::isPaused() const {
-    Locker(_mutex);
+    Locker lock(_mutex);
     return _player->isPaused();
 }
 
 float RecordingScriptingInterface::playerElapsed() const {
-    Locker(_mutex);
+    Locker lock(_mutex);
     return _player->position();
 }
 
 float RecordingScriptingInterface::playerLength() const {
-    Locker(_mutex);
+    Locker lock(_mutex);
     return _player->length();
 }
 
 void RecordingScriptingInterface::playClip(NetworkClipLoaderPointer clipLoader, const QString& url, const ScriptValue& callback) {
-    Locker(_mutex);
+    Locker lock(_mutex);
     _player->queueClip(clipLoader->getClip());
 
     if (callback.isFunction()) {
@@ -75,7 +75,7 @@ void RecordingScriptingInterface::playClip(NetworkClipLoaderPointer clipLoader,
 }
 
 void RecordingScriptingInterface::loadRecording(const QString& url, const ScriptValue& callback) {
-    Locker(_mutex);
+    Locker lock(_mutex);
 
     auto clipLoader = DependencyManager::get<recording::ClipCache>()->getClipLoader(url);
 
@@ -134,12 +134,12 @@ void RecordingScriptingInterface::startPlaying() {
         return;
     }
 
-    Locker(_mutex);
+    Locker lock(_mutex);
     _player->play();
 }
 
 void RecordingScriptingInterface::setPlayerVolume(float volume) {
-    Locker(_mutex);
+    Locker lock(_mutex);
     _player->setVolume(std::min(std::max(volume, 0.0f), 1.0f));
 }
 
@@ -153,7 +153,7 @@ void RecordingScriptingInterface::setPlayerTime(float time) {
         return;
     }
 
-    Locker(_mutex);
+    Locker lock(_mutex);
     _player->seek(time);
 }
 
@@ -162,7 +162,7 @@ void RecordingScriptingInterface::setPlayFromCurrentLocation(bool playFromCurren
 }
 
 void RecordingScriptingInterface::setPlayerLoop(bool loop) {
-    Locker(_mutex);
+    Locker lock(_mutex);
     _player->loop(loop);
 }
 
@@ -183,7 +183,7 @@ void RecordingScriptingInterface::setPlayerUseSkeletonModel(bool useSkeletonMode
 }
 
 void RecordingScriptingInterface::pausePlayer() {
-    Locker(_mutex);
+    Locker lock(_mutex);
     _player->pause();
 }
 
@@ -193,7 +193,7 @@ void RecordingScriptingInterface::stopPlaying() {
         return;
     }
 
-    Locker(_mutex);
+    Locker lock(_mutex);
     _player->stop();
 }
 
@@ -211,7 +211,7 @@ void RecordingScriptingInterface::startRecording() {
         return;
     }
 
-    Locker(_mutex);
+    Locker lock(_mutex);
     _recorder->start();
 }
 
@@ -221,7 +221,7 @@ void RecordingScriptingInterface::stopRecording() {
         return;
     }
 
-    Locker(_mutex);
+    Locker lock(_mutex);
     _recorder->stop();
     _lastClip = _recorder->getClip();
     _lastClip->seek(0);
@@ -236,7 +236,7 @@ QString RecordingScriptingInterface::getDefaultRecordingSaveDirectory() {
 }
 
 void RecordingScriptingInterface::saveRecording(const QString& filename) {
-    Locker(_mutex);
+    Locker lock(_mutex);
     if (!_lastClip) {
         qWarning() << "There is no recording to save";
         return;
@@ -251,7 +251,7 @@ bool RecordingScriptingInterface::saveRecordingToAsset(const ScriptValue& getCli
         return false;
     }
 
-    Locker(_mutex);
+    Locker lock(_mutex);
     if (!_lastClip) {
         qWarning() << "There is no recording to save";
         return false;
@@ -293,7 +293,7 @@ void RecordingScriptingInterface::loadLastRecording() {
         return;
     }
 
-    Locker(_mutex);
+    Locker lock(_mutex);
 
     if (!_lastClip) {
         qCDebug(scriptengine) << "There is no recording to load";

libraries/recording/src/recording/RecordingScriptingInterface.h

@@ -357,7 +357,7 @@ protected:
     using Locker = std::unique_lock<Mutex>;
     using Flag = std::atomic<bool>;
 
-    Mutex _mutex;
+    mutable Mutex _mutex;
 
     QSharedPointer<recording::Deck> _player;
     QSharedPointer<recording::Recorder> _recorder;