Update comment in src/app/InteractionModelEngine.cpp

dvdm-qorvo · bzbarsky-apple · web-flow · commit 034ea73a4245 · 2025-02-25T19:51:28.000+01:00
Co-authored-by: Boris Zbarsky &lt;bzbarsky@apple.com&gt;
diff --git a/src/app/InteractionModelEngine.cpp b/src/app/InteractionModelEngine.cpp
@@ -754,8 +754,8 @@ Protocols::InteractionModel::Status InteractionModelEngine::OnReadInitialRequest
                 }
                 iterator->Release();
 
-                // If we have no subscriptions to resume, we can cancel the Timer, to make sure it is cleared in the case,
-                // we deleted a subscription in resumption mode.
+                // If we have no subscriptions to resume, we can cancel the timer, which might be armed
+                // if one of the subscriptions we deleted was about to be resumed.
                 if (!HasSubscriptionsToResume())
                 {
                     mpExchangeMgr->GetSessionManager()->SystemLayer()->CancelTimer(ResumeSubscriptionsTimerCallback, this);

Original file line number	Diff line number	Diff line change
`@@ -754,8 +754,8 @@ Protocols::InteractionModel::Status InteractionModelEngine::OnReadInitialRequest`
`754`	`754`	`}`
`755`	`755`	`iterator->Release();`
`756`	`756`
`757`		`- // If we have no subscriptions to resume, we can cancel the Timer, to make sure it is cleared in the case,`
`758`		`- // we deleted a subscription in resumption mode.`
	`757`	`+ // If we have no subscriptions to resume, we can cancel the timer, which might be armed`
	`758`	`+ // if one of the subscriptions we deleted was about to be resumed.`
`759`	`759`	`if (!HasSubscriptionsToResume())`
`760`	`760`	`{`
`761`	`761`	`mpExchangeMgr->GetSessionManager()->SystemLayer()->CancelTimer(ResumeSubscriptionsTimerCallback, this);`