Use StringBuilder instead of std::stringstream

erwinpan1 · Feb 13, 2024 · f32ad21 · f32ad21
1 parent 4af0b95
commit f32ad21
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/examples/chef/common/clusters/channel/ChannelManager.cpp b/examples/chef/common/clusters/channel/ChannelManager.cpp
@@ -137,14 +137,13 @@ CHIP_ERROR ChannelManager::HandleGetCurrentChannel(AttributeValueEncoder & aEnco
 
 bool ChannelManager::isChannelMatched(const ChannelInfoType & channel, const chip::CharSpan & match)
 {
-    std::stringstream ss;
-    ss << channel.majorNumber << "." << channel.minorNumber;
-    std::string number = ss.str();
+    StringBuilder<16> channelNum;
+    channelNum.AddFormat("%d.%d", channel.majorNumber, channel.minorNumber);
 
     auto isMatch = [&match](const Optional<chip::CharSpan> & a) { return a.HasValue() && a.Value().data_equal(match); };
 
     return isMatch(channel.name) || isMatch(channel.affiliateCallSign) || isMatch(channel.callSign) ||
-        match.data_equal(chip::CharSpan::fromCharString(number.c_str()));
+        match.data_equal(chip::CharSpan::fromCharString(channelNum.c_str()));
 }
 
 void ChannelManager::HandleChangeChannel(CommandResponseHelper<ChangeChannelResponseType> & helper, const CharSpan & match)