allow re-enabling profiler

include/ProfilerLib/Profiler.hpp

@@ -67,11 +67,16 @@ class Profiler {
 
     void disable();
 
+    void enable();
+
     [[nodiscard]] bool isEnabled() const;
 
-  private:
     void save();
-    void submitEvent(const TraceEvent &event);
+
+  private:
+    // Forcing the event to be submitted causes it to be written even if the profiler is disabled. This is useful when a
+    // task consists of multiple events (such as scopeevent) where the profiler might be disabled during task execution.
+    void submitEvent(const TraceEvent &event, bool force = false);
 
     std::string name;
     std::filesystem::path outputPath;

include/ProfilerLib/ScopeEvent.hpp

@@ -30,6 +30,7 @@ class ScopeEvent {
     ~ScopeEvent();
 
   private:
+    bool active;
     Profiler &p;
 };
 

src/Profiler.cpp

@@ -23,8 +23,8 @@ Profiler::Profiler(std::string name, std::filesystem::path outputPath) :
     submitInstantEvent("Profiler \"" + this->name + "\" starting", Scope::Global);
 }
 
-void Profiler::submitEvent(const TraceEvent &event) {
-    if (!this->enabled) {
+void Profiler::submitEvent(const TraceEvent &event, bool force) {
+    if (!this->enabled && !force) {
         return;
     }
     std::lock_guard<std::mutex> lock(eventListMutex);
@@ -116,6 +116,10 @@ void Profiler::disable() {
     this->enabled = false;
 }
 
+void Profiler::enable() {
+    this->enabled = true;
+}
+
 [[nodiscard]] bool Profiler::isEnabled() const {
     return enabled;
 };

src/ProfilerUtil.cpp

@@ -22,6 +22,6 @@ namespace profilerUtil {
     }
 
     std::size_t tidHash() {
-        return std::hash<std::thread::id>{}(tid());
+        return std::hash<std::thread::id>{}(tid()) & 0xFFFFFFF;
     }
 } // namespace profilerUtil

src/ScopeEvent.cpp

@@ -12,14 +12,23 @@
 #include "TraceEvent.hpp"           // for TraceEvent, TraceEventType, Trac...
 
 ScopeEvent::ScopeEvent(Profiler &profiler, std::string name) : p{profiler} {
+    active = p.isEnabled();
+    if (!active) {
+        return;
+    }
     TraceEvent e;
     e.ph = TraceEventType::DurationBegin;
     e.name = p.name + ": " + std::move(name);
     p.submitEvent(e);
 }
 
 ScopeEvent::~ScopeEvent() {
+    if (!active) {
+        // Prevent submitting DurationEnd when the profiler was disabled at DurationBegin
+        return;
+    }
     TraceEvent e;
     e.ph = TraceEventType::DurationEnd;
-    p.submitEvent(e);
+    // Submit the event even if profiler was disabled in the meantime (only consider enabled state at start)
+    p.submitEvent(e, true);
 }