Update comments for RemoveSynchronizedDevice

Author: yufengwangca

examples/fabric-admin/rpc/RpcClient.cpp

@@ -90,37 +90,44 @@ CHIP_ERROR AddSynchronizedDevice(chip::NodeId nodeId)
     chip_rpc_SynchronizedDevice device;
     device.node_id = nodeId;
 
-    // The RPC will remain active as long as `addSynchronizedDeviceCall` is alive.
+    // By assigning the returned call to the global 'addSynchronizedDeviceCall', the RPC
+    // call is kept alive until it completes. When a response is received, it
+    // will be logged by the handler function and the call will complete.
     addSynchronizedDeviceCall = fabricBridgeClient.AddSynchronizedDevice(device, OnAddDeviceResponseCompleted);
 
     if (!addSynchronizedDeviceCall.active())
     {
-        return CHIP_ERROR_INTERNAL;
+        {
+            // The RPC call was not sent. This could occur due to, for example, an invalid channel ID. Handle if necessary.
+            return CHIP_ERROR_INTERNAL;
+        }
+
+        return CHIP_NO_ERROR;
     }
 
-    return CHIP_NO_ERROR;
-}
+    CHIP_ERROR RemoveSynchronizedDevice(chip::NodeId nodeId)
+    {
+        ChipLogProgress(NotSpecified, "RemoveSynchronizedDevice");
 
-CHIP_ERROR RemoveSynchronizedDevice(chip::NodeId nodeId)
-{
-    ChipLogProgress(NotSpecified, "RemoveSynchronizedDevice");
+        if (removeSynchronizedDeviceCall.active())
+        {
+            ChipLogError(NotSpecified, "Remove Synchronized Device operation is in progress\n");
+            return CHIP_ERROR_BUSY;
+        }
 
-    if (removeSynchronizedDeviceCall.active())
-    {
-        ChipLogError(NotSpecified, "Remove Synchronized Device operation is in progress\n");
-        return CHIP_ERROR_BUSY;
-    }
+        chip_rpc_SynchronizedDevice device;
+        device.node_id = nodeId;
 
-    chip_rpc_SynchronizedDevice device;
-    device.node_id = nodeId;
+        // By assigning the returned call to the global 'removeSynchronizedDeviceCall', the RPC
+        // call is kept alive until it completes. When a response is received, it
+        // will be logged by the handler function and the call will complete.
+        removeSynchronizedDeviceCall = fabricBridgeClient.RemoveSynchronizedDevice(device, OnRemoveDeviceResponseCompleted);
 
-    // The RPC will remain active as long as `removeSynchronizedDeviceCall` is alive.
-    removeSynchronizedDeviceCall = fabricBridgeClient.RemoveSynchronizedDevice(device, OnRemoveDeviceResponseCompleted);
+        if (!removeSynchronizedDeviceCall.active())
+        {
+            // The RPC call was not sent. This could occur due to, for example, an invalid channel ID. Handle if necessary.
+            return CHIP_ERROR_INTERNAL;
+        }
 
-    if (!removeSynchronizedDeviceCall.active())
-    {
-        return CHIP_ERROR_INTERNAL;
+        return CHIP_NO_ERROR;
     }
-
-    return CHIP_NO_ERROR;
-}