Moved batching logging for clarity

project-chip · Jun 1, 2023 · 2987aa1 · 2987aa1
1 parent bf6cff5
commit 2987aa1
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/darwin/Framework/CHIP/MTRAsyncCallbackWorkQueue.mm b/src/darwin/Framework/CHIP/MTRAsyncCallbackWorkQueue.mm
@@ -181,8 +181,6 @@ - (void)_callNextReadyWorkItem
 
                 BOOL fullyMerged = NO;
                 workItem.batchingHandler(workItem.batchableData, nextWorkItem.batchableData, &fullyMerged);
-                MTR_LOG_DEFAULT(
-                    "MTRAsyncCallbackWorkQueue: merged items for batching - fully merged %@", fullyMerged ? @"YES" : @"NO");
                 if (!fullyMerged) {
                     // if some parts of the next item is
                     break;

diff --git a/src/darwin/Framework/CHIP/MTRDevice.mm b/src/darwin/Framework/CHIP/MTRDevice.mm
@@ -840,6 +840,7 @@ static BOOL AttributeHasChangesOmittedQuality(MTRAttributePath * attributePath)
             }
 
             if (readRequestsNext.count == 0) {
+                MTR_LOG_DEFAULT("%@ batching - fully merged next item %@", logPrefix, fullyMerged ? @"YES" : @"NO");
                 *fullyMerged = YES;
             }
         };
-Original file line number
+Diff line change
@@ Expand Up @@
                 }
                 if (readRequestsNext.count == 0) {
+                    MTR_LOG_DEFAULT("%@ batching - fully merged next item %@", logPrefix, fullyMerged ? @"YES" : @"NO");
                     *fullyMerged = YES;
                 }
             };
@@ Expand Down @@