Skip to content

Commit ec92429

Browse files
committed
Address copilot review comments
1 parent 81ee2a0 commit ec92429

File tree

3 files changed

+3
-4
lines changed

3 files changed

+3
-4
lines changed

examples/oven-app/oven-app-common/include/TemperatureControlledCabinetEndpoint.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,6 @@ namespace app {
2828
namespace Clusters {
2929
namespace TemperatureControlledCabinet {
3030

31-
// Add TemperatureControlDelegate here
32-
3331
class TemperatureControlledCabinetEndpoint
3432
{
3533
public:

examples/oven-app/oven-app-common/src/AppSupportedTemperatureLevelsDelegate.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@ uint8_t AppSupportedTemperatureLevelsDelegate::Size()
4747
return endpointPair.mSize;
4848
}
4949
}
50-
ChipLogProgress(AppServer, "No matching endpoint found for %d", mEndpoint);
50+
ChipLogError(AppServer, "No matching endpoint found for %d in Size()", mEndpoint);
51+
// Returning 0 indicates that the endpoint was not found; this should be treated as an error.
5152
return 0;
5253
}
5354

examples/oven-app/silabs/src/OvenManager.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ using namespace chip::app;
3636
using namespace chip::app::Clusters;
3737
using namespace chip::DeviceLayer;
3838

39-
// add tag-lists
39+
// TODO: Implement tag-lists.
4040

4141
OvenManager OvenManager::sOvenMgr;
4242

0 commit comments

Comments
 (0)