Restore m_scheduledUploadTime position to before LOCKGUARD (match main)

bmehta001 · Copilot · bmehta001 · commit f4b3bd9e50be · 2026-03-18T23:26:46.000-05:00
Moving m_scheduledUploadTime inside the LOCKGUARD changes the timing
of the delta check in scheduleUpload(), causing fewer events to be
delivered within test timeouts.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/lib/tpm/TransmissionPolicyManager.cpp b/lib/tpm/TransmissionPolicyManager.cpp
@@ -185,9 +185,10 @@ namespace MAT_NS_BEGIN {
             return;
         }
         m_runningLatency = latency;
+        m_scheduledUploadTime = std::numeric_limits<uint64_t>::max();
+
         {
             LOCKGUARD(m_scheduledUploadMutex);
-            m_scheduledUploadTime = std::numeric_limits<uint64_t>::max();
             m_isUploadScheduled = false;  // Allow to schedule another uploadAsync
             if ((m_isPaused) || (m_scheduledUploadAborted))
             {

Original file line number	Diff line number	Diff line change
`@@ -185,9 +185,10 @@ namespace MAT_NS_BEGIN {`
`185`	`185`	`return;`
`186`	`186`	`}`
`187`	`187`	`m_runningLatency = latency;`
	`188`	`+ m_scheduledUploadTime = std::numeric_limits<uint64_t>::max();`
	`189`	`+`
`188`	`190`	`{`
`189`	`191`	`LOCKGUARD(m_scheduledUploadMutex);`
`190`		`- m_scheduledUploadTime = std::numeric_limits<uint64_t>::max();`
`191`	`192`	`m_isUploadScheduled = false; // Allow to schedule another uploadAsync`
`192`	`193`	`if ((m_isPaused) \|\| (m_scheduledUploadAborted))`
`193`	`194`	`{`