From 9b7d9d7e51b981be6341c564c6cd38b733cce2d5 Mon Sep 17 00:00:00 2001
From: erwinpan1 <erwinpan1@gmail.com>
Date: Fri, 9 Feb 2024 18:39:04 +0800
Subject: [PATCH] Rename index to iFirstMatchedChannel

---
 .../chef/common/clusters/channel/ChannelManager.cpp | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/examples/chef/common/clusters/channel/ChannelManager.cpp b/examples/chef/common/clusters/channel/ChannelManager.cpp
index 2d458aaf0ea40f..50c18266b89dd1 100644
--- a/examples/chef/common/clusters/channel/ChannelManager.cpp
+++ b/examples/chef/common/clusters/channel/ChannelManager.cpp
@@ -152,7 +152,7 @@ bool isChannelMatched(const ChannelInfoType & channel, const CharSpan & match)
 void ChannelManager::HandleChangeChannel(CommandResponseHelper<ChangeChannelResponseType> & helper, const CharSpan & match)
 {
     std::vector<ChannelInfoType> matchedChannels;
-    uint16_t index = 0;
+    uint16_t iFirstMatchedChannel = 0;
     for (auto const & channel : mChannels)
     {
         // verify if CharSpan matches channel name
@@ -161,12 +161,9 @@ void ChannelManager::HandleChangeChannel(CommandResponseHelper<ChangeChannelResp
         {
             matchedChannels.push_back(channel);
         }
-        else if (matchedChannels.size() == 0)
+        else if (matchedChannels.empty())
         {
-            // "index" is only used when we end up with matchedChannels.size() == 1.
-            // In that case, we want it to be the number of non-matching channels we saw before
-            // the matching one.
-            index++;
+            iFirstMatchedChannel++;
         }
     }
 
@@ -178,7 +175,7 @@ void ChannelManager::HandleChangeChannel(CommandResponseHelper<ChangeChannelResp
         response.status = chip::app::Clusters::Channel::StatusEnum::kMultipleMatches;
         helper.Success(response);
     }
-    else if (matchedChannels.size() == 0)
+    else if (matchedChannels.empty())
     {
         // Error: Found no match
         response.status = chip::app::Clusters::Channel::StatusEnum::kNoMatches;
@@ -189,7 +186,7 @@ void ChannelManager::HandleChangeChannel(CommandResponseHelper<ChangeChannelResp
         response.status      = chip::app::Clusters::Channel::StatusEnum::kSuccess;
         response.data        = chip::MakeOptional(CharSpan::fromCharString("data response"));
         mCurrentChannel      = matchedChannels[0];
-        mCurrentChannelIndex = index;
+        mCurrentChannelIndex = iFirstMatchedChannel;
         helper.Success(response);
     }
 }