Clarified comments following reviews.

project-chip · Dec 13, 2023 · 3d8a267 · 3d8a267
1 parent 020b326
commit 3d8a267
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/examples/rvc-app/rvc-common/src/rvc-device.cpp b/examples/rvc-app/rvc-common/src/rvc-device.cpp
@@ -106,7 +106,6 @@ void RvcDevice::HandleOpStatePauseCallback(Clusters::OperationalState::GenericOp
 
 void RvcDevice::HandleOpStateResumeCallback(Clusters::OperationalState::GenericOperationalError & err)
 {
-    // This method is only called if the device is in a resume-compatible state, i.e. `Charging`, `Docked` or `Paused`.
     auto error = CHIP_NO_ERROR;
     switch (mOperationalStateInstance.GetCurrentOperationalState())
     {
@@ -136,9 +135,13 @@ void RvcDevice::HandleOpStateResumeCallback(Clusters::OperationalState::GenericO
         }
     }
     break;
+    default:
+        // This method is only called if the device is in a resume-compatible state, i.e. `Charging`, `Docked` or
+        // `Paused`. Therefor, we do not expect to ever enter this branch.
+        err.Set(to_underlying(OperationalState::ErrorStateEnum::kCommandInvalidInState));
+        return;
     }
 
-    // populate the response.
     if (error == CHIP_NO_ERROR)
     {
         err.Set(to_underlying(OperationalState::ErrorStateEnum::kNoError));