Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Kiel Oleson <[email protected]>
  • Loading branch information
woody-apple and kiel-apple authored Aug 8, 2024
1 parent 6f09140 commit d1ad575
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,8 @@ CHIP_ERROR CastingPlayerDiscovery::StartDiscovery(uint32_t deviceTypeFilter)

CHIP_ERROR CastingPlayerDiscovery::StopDiscovery()
{
ChipLogProgress(Discovery, "CastingPlayerDiscovery::StopDiscovery() mCastingPlayers: %zu, mCastingPlayersInternal: %zu",
mCastingPlayers.size(), mCastingPlayersInternal.size());
ChipLogProgress(Discovery, "CastingPlayerDiscovery::StopDiscovery() mCastingPlayers: %u, mCastingPlayersInternal: %u",
static_cast<unsigned int>mCastingPlayers.size(), static_cast<unsigned int>mCastingPlayersInternal.size());
VerifyOrReturnError(mState == DISCOVERY_RUNNING, CHIP_ERROR_INCORRECT_STATE);
ReturnErrorOnFailure(mCommissionableNodeController.StopDiscovery());

Expand All @@ -80,8 +80,8 @@ CHIP_ERROR CastingPlayerDiscovery::StopDiscovery()

void CastingPlayerDiscovery::ClearDisconnectedCastingPlayersInternal()
{
ChipLogProgress(Discovery, "CastingPlayerDiscovery::ClearDisconnectedCastingPlayersInternal() mCastingPlayersInternal: %zu",
mCastingPlayersInternal.size());
ChipLogProgress(Discovery, "CastingPlayerDiscovery::ClearDisconnectedCastingPlayersInternal() mCastingPlayersInternal: %u",
static_cast<unsigned int>mCastingPlayersInternal.size());
// Only clear the CastingPlayers in mCastingPlayersInternal with ConnectionState == CASTING_PLAYER_NOT_CONNECTED
for (auto it = mCastingPlayersInternal.begin(); it != mCastingPlayersInternal.end();)
{
Expand Down

0 comments on commit d1ad575

Please sign in to comment.