aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBenoit Germain <benoit.germain@ubisoft.com>2024-11-13 10:47:24 +0100
committerBenoit Germain <benoit.germain@ubisoft.com>2024-11-13 10:47:24 +0100
commitf2a3c033fc31332e78aa45d2d9deaf51359b584a (patch)
treeb8e017488af194e1156f2209d9f7fd70ad56f0cb
parent9e8930f02e7a53a41b713c642bcb53b3b61f7cb5 (diff)
downloadlanes-f2a3c033fc31332e78aa45d2d9deaf51359b584a.tar.gz
lanes-f2a3c033fc31332e78aa45d2d9deaf51359b584a.tar.bz2
lanes-f2a3c033fc31332e78aa45d2d9deaf51359b584a.zip
Cleaning up guano
Lane::tracking_next and LaneTracker::trackingFirst do not need to be volatile or anything else, all accesses are mutex-protected
-rw-r--r--src/lane.hpp4
-rw-r--r--src/tracker.hpp2
2 files changed, 3 insertions, 3 deletions
diff --git a/src/lane.hpp b/src/lane.hpp
index 4f87e2a..eb51aa3 100644
--- a/src/lane.hpp
+++ b/src/lane.hpp
@@ -135,8 +135,8 @@ class Lane
135 // is still running 135 // is still running
136 // S: cleans up after itself if non-nullptr at lane exit 136 // S: cleans up after itself if non-nullptr at lane exit
137 137
138 // For tracking only 138 // access is protected by LaneTracker::trackingMutex
139 Lane* volatile tracking_next{ nullptr }; 139 Lane* tracking_next{ nullptr };
140 140
141 ErrorTraceLevel const errorTraceLevel{ Basic }; 141 ErrorTraceLevel const errorTraceLevel{ Basic };
142 142
diff --git a/src/tracker.hpp b/src/tracker.hpp
index dcf3c6e..c1a38e2 100644
--- a/src/tracker.hpp
+++ b/src/tracker.hpp
@@ -11,7 +11,7 @@ class LaneTracker
11{ 11{
12 private: 12 private:
13 mutable std::mutex trackingMutex; 13 mutable std::mutex trackingMutex;
14 Lane* volatile trackingFirst{ nullptr }; // will change to TRACKING_END if we want to activate tracking 14 Lane* trackingFirst{ nullptr }; // will change to TRACKING_END if we want to activate tracking
15 15
16 public: 16 public:
17 void tracking_add(Lane* lane_); 17 void tracking_add(Lane* lane_);