From f0790cee70bb95209f42ea54e2a96f17b02110d6 Mon Sep 17 00:00:00 2001
From: Stanislaw Halik <sthalik@misaki.pl>
Date: Thu, 12 Jun 2014 03:51:52 +0200
Subject: Fix lock order reversal

---
 FTNoIR_Tracker_PT/ftnoir_tracker_pt.cpp | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

(limited to 'FTNoIR_Tracker_PT/ftnoir_tracker_pt.cpp')

diff --git a/FTNoIR_Tracker_PT/ftnoir_tracker_pt.cpp b/FTNoIR_Tracker_PT/ftnoir_tracker_pt.cpp
index 787a5128..e3823291 100644
--- a/FTNoIR_Tracker_PT/ftnoir_tracker_pt.cpp
+++ b/FTNoIR_Tracker_PT/ftnoir_tracker_pt.cpp
@@ -23,12 +23,13 @@ const float deg2rad = 1.0/rad2deg;
 
 //-----------------------------------------------------------------------------
 Tracker::Tracker()
-    : commands(0),
+    : mutex(QMutex::Recursive),
+      commands(0),
 	  video_widget(NULL), 
 	  video_frame(NULL),
 	  tracking_valid(false),
-	  need_apply(false),
-	  new_settings(nullptr)
+      new_settings(nullptr)
+
 {
 	qDebug()<<"Tracker::Tracker";
 }
@@ -115,21 +116,18 @@ void Tracker::run()
 }
 void Tracker::apply(settings& s)
 {
-	QMutexLocker lock(&mutex);
-	need_apply = true;
-	// caller guarantees object lifetime
+    // caller guarantees object lifetime
 	new_settings = &s;
 }
 
 void Tracker::apply_inner()
 {
-	QMutexLocker lock(&mutex);
-	if (!need_apply)
-		return;
 	qDebug()<<"Tracker:: Applying settings";
-	auto& s = *new_settings;
+    settings* tmp = new_settings;
+    if (tmp == nullptr)
+        return;
+    auto& s = *tmp;
 	new_settings = nullptr;
-	need_apply = false;
     camera.set_device_index(s.cam_index);
     camera.set_res(s.cam_res_x, s.cam_res_y);
     camera.set_fps(s.cam_fps);
-- 
cgit v1.2.3