Correctly annotate things that are unused when logging is disabled.

bzbarsky-apple · bzbarsky-apple · commit 9b283f4c9ca4 · 2024-02-15T11:34:16.000-05:00
We don't seem to have very good CI coverage for the logging-disabled configurations. Also removes UNUSED_VAR and UNUSED_PARAMETER use in favor of the standard [[maybe_unused]]. Fixes project-chip#32113
diff --git a/examples/bridge-app/asr/subdevice/SubDeviceManager.cpp b/examples/bridge-app/asr/subdevice/SubDeviceManager.cpp
@@ -91,12 +91,11 @@ CHIP_ERROR RemoveDeviceEndpoint(SubDevice * dev)
     {
         if (gSubDevices[index] == dev)
         {
-            EndpointId ep      = emberAfClearDynamicEndpoint(index);
-            gSubDevices[index] = NULL;
-            ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);
             // Silence complaints about unused ep when progress logging
             // disabled.
-            UNUSED_VAR(ep);
+            [[maybe_unused] EndpointId ep      = emberAfClearDynamicEndpoint(index);
+            gSubDevices[index] = NULL;
+            ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);
             return CHIP_NO_ERROR;
         }
     }
diff --git a/examples/bridge-app/esp32/main/main.cpp b/examples/bridge-app/esp32/main/main.cpp
@@ -201,12 +201,11 @@ CHIP_ERROR RemoveDeviceEndpoint(Device * dev)
     {
         if (gDevices[index] == dev)
         {
-            EndpointId ep   = emberAfClearDynamicEndpoint(index);
-            gDevices[index] = NULL;
-            ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);
             // Silence complaints about unused ep when progress logging
             // disabled.
-            UNUSED_VAR(ep);
+            [[maybe_unused]] EndpointId ep = emberAfClearDynamicEndpoint(index);
+            gDevices[index]                = NULL;
+            ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);
             return CHIP_NO_ERROR;
         }
     }
diff --git a/examples/bridge-app/linux/main.cpp b/examples/bridge-app/linux/main.cpp
@@ -300,12 +300,11 @@ int RemoveDeviceEndpoint(Device * dev)
         {
             // Todo: Update this to schedule the work rather than use this lock
             DeviceLayer::StackLock lock;
-            EndpointId ep   = emberAfClearDynamicEndpoint(index);
-            gDevices[index] = nullptr;
-            ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);
             // Silence complaints about unused ep when progress logging
             // disabled.
-            UNUSED_VAR(ep);
+            [[maybe_unused]] EndpointId ep = emberAfClearDynamicEndpoint(index);
+            gDevices[index]                = nullptr;
+            ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);
             return index;
         }
         index++;
diff --git a/examples/bridge-app/telink/src/AppTask.cpp b/examples/bridge-app/telink/src/AppTask.cpp
@@ -224,12 +224,11 @@ CHIP_ERROR RemoveDeviceEndpoint(Device * dev)
     {
         if (gDevices[index] == dev)
         {
-            EndpointId ep   = emberAfClearDynamicEndpoint(index);
-            gDevices[index] = NULL;
-            ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);
             // Silence complaints about unused ep when progress logging
             // disabled.
-            UNUSED_VAR(ep);
+            [[maybe_unused]] EndpointId ep = emberAfClearDynamicEndpoint(index);
+            gDevices[index]                = NULL;
+            ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);
             return CHIP_NO_ERROR;
         }
     }
diff --git a/examples/chip-tool/commands/common/DeviceScanner.cpp b/examples/chip-tool/commands/common/DeviceScanner.cpp
@@ -218,7 +218,7 @@ void DeviceScanner::Log() const
     auto resultsCount = mDiscoveredResults.size();
     VerifyOrReturn(resultsCount > 0, ChipLogProgress(chipTool, "No device discovered."));
 
-    uint16_t index = 0;
+    [[maybe_unused]] uint16_t index = 0;
     for (auto & instance : mDiscoveredResults)
     {
         ChipLogProgress(chipTool, "Instance Name: %s ", instance.first.c_str());
diff --git a/examples/chip-tool/commands/discover/DiscoverCommissionersCommand.cpp b/examples/chip-tool/commands/discover/DiscoverCommissionersCommand.cpp
@@ -29,7 +29,7 @@ CHIP_ERROR DiscoverCommissionersCommand::RunCommand()
 
 void DiscoverCommissionersCommand::Shutdown()
 {
-    int commissionerCount = 0;
+    [[maybe_unused]] int commissionerCount = 0;
     for (int i = 0; i < CHIP_DEVICE_CONFIG_MAX_DISCOVERED_NODES; i++)
     {
         const Dnssd::DiscoveredNodeData * commissioner = mCommissionableNodeController.GetDiscoveredCommissioner(i);
diff --git a/examples/common/websocket-server/WebSocketServer.cpp b/examples/common/websocket-server/WebSocketServer.cpp
@@ -24,9 +24,9 @@
 #include <deque>
 #include <mutex>
 
-constexpr uint16_t kDefaultWebSocketServerPort = 9002;
-constexpr uint16_t kMaxMessageBufferLen        = 8192;
-constexpr char kWebSocketServerReadyMessage[]  = "== WebSocket Server Ready";
+constexpr uint16_t kDefaultWebSocketServerPort                 = 9002;
+constexpr uint16_t kMaxMessageBufferLen                        = 8192;
+[[maybe_unused]] constexpr char kWebSocketServerReadyMessage[] = "== WebSocket Server Ready";
 
 namespace {
 lws * gWebSocketInstance = nullptr;
diff --git a/examples/light-switch-app/genio/src/main.cpp b/examples/light-switch-app/genio/src/main.cpp
@@ -51,8 +51,6 @@ using namespace ::chip;
 using namespace ::chip::Inet;
 using namespace ::chip::DeviceLayer;
 
-#define UNUSED_PARAMETER(a) (a = a)
-
 volatile int apperror_cnt;
 
 /***************************************************************************
diff --git a/examples/lighting-app/genio/src/AppTask.cpp b/examples/lighting-app/genio/src/AppTask.cpp
@@ -66,8 +66,6 @@
 #error "Must have portYIELD_FROM_ISR or portEND_SWITCHING_ISR"
 #endif
 
-#define UNUSED_PARAMETER(a) (a = a)
-
 #if defined(ENABLE_CHIP_SHELL)
 using chip::Shell::Engine;
 using chip::Shell::PrintCommandHelp;
diff --git a/examples/lighting-app/genio/src/main.cpp b/examples/lighting-app/genio/src/main.cpp
@@ -51,8 +51,6 @@ using namespace ::chip;
 using namespace ::chip::Inet;
 using namespace ::chip::DeviceLayer;
 
-#define UNUSED_PARAMETER(a) (a = a)
-
 volatile int apperror_cnt;
 
 /***************************************************************************
diff --git a/examples/lock-app/genio/src/AppTask.cpp b/examples/lock-app/genio/src/AppTask.cpp
@@ -60,8 +60,6 @@
 #error "Must have portYIELD_FROM_ISR or portEND_SWITCHING_ISR"
 #endif
 
-#define UNUSED_PARAMETER(a) (a = a)
-
 namespace {
 
 TimerHandle_t sFunctionTimer; // FreeRTOS app sw timer.
diff --git a/examples/lock-app/genio/src/main.cpp b/examples/lock-app/genio/src/main.cpp
@@ -51,8 +51,6 @@ using namespace ::chip;
 using namespace ::chip::Inet;
 using namespace ::chip::DeviceLayer;
 
-#define UNUSED_PARAMETER(a) (a = a)
-
 volatile int apperror_cnt;
 
 /***************************************************************************
diff --git a/examples/ota-requestor-app/genio/src/AppTask.cpp b/examples/ota-requestor-app/genio/src/AppTask.cpp
@@ -53,8 +53,6 @@
 #error "Must have portYIELD_FROM_ISR or portEND_SWITCHING_ISR"
 #endif
 
-#define UNUSED_PARAMETER(a) (a = a)
-
 namespace {
 
 TaskHandle_t sAppTaskHandle;
diff --git a/examples/ota-requestor-app/genio/src/main.cpp b/examples/ota-requestor-app/genio/src/main.cpp
@@ -53,8 +53,6 @@ using namespace ::chip;
 using namespace ::chip::Inet;
 using namespace ::chip::DeviceLayer;
 
-#define UNUSED_PARAMETER(a) (a = a)
-
 volatile int apperror_cnt;
 
 static void OTAEventsHandler(const DeviceLayer::ChipDeviceEvent * event, intptr_t arg)
diff --git a/examples/platform/silabs/efr32/rs911x/hal/efx32_ncp_host.c b/examples/platform/silabs/efr32/rs911x/hal/efx32_ncp_host.c
@@ -55,21 +55,18 @@ LDMA_TransferCfg_t ldmaRXConfig;
 
 static osSemaphoreId_t transfer_done_semaphore = NULL;
 
-static bool dma_callback(unsigned int channel, unsigned int sequenceNo, void * userParam)
+static bool dma_callback([[maybe_unused]] unsigned int channel, [[maybe_unused]] unsigned int sequenceNo,
+                         [[maybe_unused]] void * userParam)
 {
-    UNUSED_PARAMETER(channel);
-    UNUSED_PARAMETER(sequenceNo);
-    UNUSED_PARAMETER(userParam);
 #if defined(SL_CATLOG_POWER_MANAGER_PRESENT)
     sl_power_manager_remove_em_requirement(SL_POWER_MANAGER_EM1);
 #endif
     osSemaphoreRelease(transfer_done_semaphore);
     return false;
 }
 
-static void gpio_interrupt(uint8_t interrupt_number)
+static void gpio_interrupt([[maybe_unused]] uint8_t interrupt_number)
 {
-    UNUSED_PARAMETER(interrupt_number);
     sl_si91x_host_set_bus_event(NCP_HOST_BUS_RX_EVENT);
     //  GPIO_IntClear(0xAAAA);
 }
@@ -299,4 +296,4 @@ void sl_si91x_host_disable_bus_interrupt(void)
 bool sl_si91x_host_is_in_irq_context(void)
 {
     return (SCB->ICSR & SCB_ICSR_VECTACTIVE_Msk) != 0U;
-}
+}
diff --git a/examples/platform/silabs/efr32/rs911x/hal/efx_spi.c b/examples/platform/silabs/efr32/rs911x/hal/efx_spi.c
@@ -171,10 +171,7 @@ void sl_wfx_host_reset_chip(void)
     vTaskDelay(pdMS_TO_TICKS(3));
 }
 
-void gpio_interrupt(uint8_t interrupt_number)
-{
-    UNUSED_PARAMETER(interrupt_number);
-}
+void gpio_interrupt([[maybe_unused]] uint8_t interrupt_number) {}
 
 /*****************************************************************
  * @fn   void rsi_hal_board_init(void)
diff --git a/examples/providers/DeviceInfoProviderImpl.h b/examples/providers/DeviceInfoProviderImpl.h
@@ -46,8 +46,8 @@ class DeviceInfoProviderImpl : public DeviceInfoProvider
         void Release() override { chip::Platform::Delete(this); }
 
     private:
-        EndpointId mEndpoint = 0;
-        size_t mIndex        = 0;
+        [[maybe_unused]] EndpointId mEndpoint = 0;
+        size_t mIndex                         = 0;
         char mFixedLabelNameBuf[kMaxLabelNameLength + 1];
         char mFixedLabelValueBuf[kMaxLabelValueLength + 1];
     };
diff --git a/examples/thermostat/genio/src/main.cpp b/examples/thermostat/genio/src/main.cpp
@@ -51,8 +51,6 @@ using namespace ::chip;
 using namespace ::chip::Inet;
 using namespace ::chip::DeviceLayer;
 
-#define UNUSED_PARAMETER(a) (a = a)
-
 volatile int apperror_cnt;
 
 /***************************************************************************
diff --git a/examples/tv-casting-app/linux/CastingUtils.cpp b/examples/tv-casting-app/linux/CastingUtils.cpp
@@ -105,7 +105,7 @@ void InitCommissioningFlow(intptr_t commandArg)
             commissioner->LogDetail();
             if (associatedConnectableVideoPlayer.HasValue())
             {
-                TargetVideoPlayerInfo * targetVideoPlayerInfo = associatedConnectableVideoPlayer.Value();
+                [[maybe_unused]] TargetVideoPlayerInfo * targetVideoPlayerInfo = associatedConnectableVideoPlayer.Value();
                 ChipLogProgress(AppServer, "Previously connected with nodeId 0x" ChipLogFormatX64 " fabricIndex: %d",
                                 ChipLogValueX64(targetVideoPlayerInfo->GetNodeId()), targetVideoPlayerInfo->GetFabricIndex());
             }
@@ -305,7 +305,7 @@ void PrintFabrics()
             ChipLogError(AppServer, " -- Not initialized");
             continue;
         }
-        NodeId myNodeId = fb.GetNodeId();
+        [[maybe_unused]] NodeId myNodeId = fb.GetNodeId();
         ChipLogProgress(NotSpecified,
                         "---- Current Fabric nodeId=0x" ChipLogFormatX64 " fabricId=0x" ChipLogFormatX64 " fabricIndex=%d",
                         ChipLogValueX64(myNodeId), ChipLogValueX64(fb.GetFabricId()), fabricIndex);
diff --git a/examples/tv-casting-app/tv-casting-common/core/CastingPlayer.cpp b/examples/tv-casting-app/tv-casting-common/core/CastingPlayer.cpp
@@ -257,7 +257,7 @@ void CastingPlayer::LogDetail() const
     {
         for (unsigned j = 0; j < mAttributes.numIPs; j++)
         {
-            char * ipAddressOut = mAttributes.ipAddresses[j].ToString(buf);
+            [[maybe_unused]] char * ipAddressOut = mAttributes.ipAddresses[j].ToString(buf);
             ChipLogDetail(AppServer, "\tIP Address #%d: %s", j + 1, ipAddressOut);
         }
     }
diff --git a/examples/tv-casting-app/tv-casting-common/src/CastingServer.cpp b/examples/tv-casting-app/tv-casting-common/src/CastingServer.cpp
@@ -695,7 +695,7 @@ void CastingServer::SetDefaultFabricIndex(std::function<void(TargetVideoPlayerIn
             ChipLogError(AppServer, " -- Not initialized");
             continue;
         }
-        NodeId myNodeId = fb.GetNodeId();
+        [[maybe_unused]] NodeId myNodeId = fb.GetNodeId();
         ChipLogProgress(NotSpecified,
                         "---- Current Fabric nodeId=0x" ChipLogFormatX64 " fabricId=0x" ChipLogFormatX64 " fabricIndex=%d",
                         ChipLogValueX64(myNodeId), ChipLogValueX64(fb.GetFabricId()), fabricIndex);
diff --git a/examples/tv-casting-app/tv-casting-common/src/ZCLCallbacks.cpp b/examples/tv-casting-app/tv-casting-common/src/ZCLCallbacks.cpp
@@ -38,8 +38,8 @@ using namespace ::chip::app::Clusters;
 void MatterPostAttributeChangeCallback(const chip::app::ConcreteAttributePath & attributePath, uint8_t type, uint16_t size,
                                        uint8_t * value)
 {
-    ClusterId clusterId     = attributePath.mClusterId;
-    AttributeId attributeId = attributePath.mAttributeId;
+    [[maybe_unused]] ClusterId clusterId     = attributePath.mClusterId;
+    [[maybe_unused]] AttributeId attributeId = attributePath.mAttributeId;
     ChipLogProgress(Zcl, "AttributeChangeCallback cluster: " ChipLogFormatMEI " attribute:" ChipLogFormatMEI,
                     ChipLogValueMEI(clusterId), ChipLogValueMEI(attributeId));
 }
diff --git a/src/app/app-platform/ContentAppPlatform.cpp b/src/app/app-platform/ContentAppPlatform.cpp
@@ -237,13 +237,12 @@ EndpointId ContentAppPlatform::RemoveContentApp(ContentApp * app)
         if (mContentApps[index] == app)
         {
             EndpointId curEndpoint = app->GetEndpointId();
-            EndpointId ep          = emberAfClearDynamicEndpoint(index);
-            mContentApps[index]    = nullptr;
-            ChipLogProgress(DeviceLayer, "Removed device %d from dynamic endpoint %d (index=%d)",
-                            app->GetApplicationBasicDelegate()->HandleGetVendorId(), ep, index);
             // Silence complaints about unused ep when progress logging
             // disabled.
-            UNUSED_VAR(ep);
+            /*[[maybe_unused]]*/ EndpointId ep = emberAfClearDynamicEndpoint(index);
+            mContentApps[index]                = nullptr;
+            ChipLogProgress(DeviceLayer, "Removed device %d from dynamic endpoint %d (index=%d)",
+                            app->GetApplicationBasicDelegate()->HandleGetVendorId(), ep, index);
             if (curEndpoint == mCurrentAppEndpointId)
             {
                 mCurrentAppEndpointId = kNoCurrentEndpointId;
diff --git a/src/app/server/DefaultAclStorage.cpp b/src/app/server/DefaultAclStorage.cpp
@@ -138,7 +138,7 @@ CHIP_ERROR DefaultAclStorage::Init(PersistentStorageDelegate & persistentStorage
 
     CHIP_ERROR err;
 
-    size_t count = 0;
+    [[maybe_unused]] size_t count = 0;
 
     for (auto it = first; it != last; ++it)
     {
diff --git a/src/app/util/types_stub.h b/src/app/util/types_stub.h
@@ -166,12 +166,6 @@ typedef struct
 
 } EmberEventControl;
 
-/**
- * @description Useful macro for avoiding compiler warnings related to unused
- * function arguments or unused variables.
- */
-#define UNUSED_VAR(x) (void) (x)
-
 /**
  * @brief Returns the value of the bitmask \c bits within
  * the register or byte \c reg.
diff --git a/src/controller/tests/data_model/TestRead.cpp b/src/controller/tests/data_model/TestRead.cpp
@@ -475,7 +475,7 @@ void TestReadInteraction::TestReadSubscribeAttributeResponseWithCache(nlTestSuit
     //
     app::InteractionModelEngine::GetInstance()->RegisterReadHandlerAppCallback(&gTestReadInteraction);
 
-    int testId = 0;
+    [[maybe_unused]] int testId = 0;
 
     // Read of E2C3A1(dedup), E*C3A1(E1C3A1 not exit, E2C3A1 exist), E2C3A* (5 supported attributes)
     // Expect no versions would be cached.
diff --git a/src/crypto/tests/TestChipCryptoPAL.cpp b/src/crypto/tests/TestChipCryptoPAL.cpp
@@ -2715,7 +2715,7 @@ static void TestVIDPID_StringExtraction(nlTestSuite * inSuite, void * inContext)
     };
     // clang-format on
 
-    int caseIdx = 0;
+    [[maybe_unused]] int caseIdx = 0;
     for (const auto & testCase : kTestCases)
     {
         AttestationCertVidPid vidpid;

Original file line number	Diff line number	Diff line change
`@@ -91,12 +91,11 @@ CHIP_ERROR RemoveDeviceEndpoint(SubDevice * dev)`
`91`	`91`	`{`
`92`	`92`	`if (gSubDevices[index] == dev)`
`93`	`93`	`{`
`94`		`- EndpointId ep = emberAfClearDynamicEndpoint(index);`
`95`		`- gSubDevices[index] = NULL;`
`96`		`- ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);`
`97`	`94`	`// Silence complaints about unused ep when progress logging`
`98`	`95`	`// disabled.`
`99`		`- UNUSED_VAR(ep);`
	`96`	`+ [[maybe_unused] EndpointId ep = emberAfClearDynamicEndpoint(index);`
	`97`	`+ gSubDevices[index] = NULL;`
	`98`	`+ ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);`
`100`	`99`	`return CHIP_NO_ERROR;`
`101`	`100`	`}`
`102`	`101`	`}`
Original file line number	Diff line number	Diff line change
`@@ -201,12 +201,11 @@ CHIP_ERROR RemoveDeviceEndpoint(Device * dev)`
`201`	`201`	`{`
`202`	`202`	`if (gDevices[index] == dev)`
`203`	`203`	`{`
`204`		`- EndpointId ep = emberAfClearDynamicEndpoint(index);`
`205`		`- gDevices[index] = NULL;`
`206`		`- ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);`
`207`	`204`	`// Silence complaints about unused ep when progress logging`
`208`	`205`	`// disabled.`
`209`		`- UNUSED_VAR(ep);`
	`206`	`+ [[maybe_unused]] EndpointId ep = emberAfClearDynamicEndpoint(index);`
	`207`	`+ gDevices[index] = NULL;`
	`208`	`+ ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);`
`210`	`209`	`return CHIP_NO_ERROR;`
`211`	`210`	`}`
`212`	`211`	`}`
Original file line number	Diff line number	Diff line change
`@@ -224,12 +224,11 @@ CHIP_ERROR RemoveDeviceEndpoint(Device * dev)`
`224`	`224`	`{`
`225`	`225`	`if (gDevices[index] == dev)`
`226`	`226`	`{`
`227`		`- EndpointId ep = emberAfClearDynamicEndpoint(index);`
`228`		`- gDevices[index] = NULL;`
`229`		`- ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);`
`230`	`227`	`// Silence complaints about unused ep when progress logging`
`231`	`228`	`// disabled.`
`232`		`- UNUSED_VAR(ep);`
	`229`	`+ [[maybe_unused]] EndpointId ep = emberAfClearDynamicEndpoint(index);`
	`230`	`+ gDevices[index] = NULL;`
	`231`	`+ ChipLogProgress(DeviceLayer, "Removed device %s from dynamic endpoint %d (index=%d)", dev->GetName(), ep, index);`
`233`	`232`	`return CHIP_NO_ERROR;`
`234`	`233`	`}`
`235`	`234`	`}`
Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,7 @@ void DeviceScanner::Log() const`
`218`	`218`	`auto resultsCount = mDiscoveredResults.size();`
`219`	`219`	`VerifyOrReturn(resultsCount > 0, ChipLogProgress(chipTool, "No device discovered."));`
`220`	`220`
`221`		`- uint16_t index = 0;`
	`221`	`+ [[maybe_unused]] uint16_t index = 0;`
`222`	`222`	`for (auto & instance : mDiscoveredResults)`
`223`	`223`	`{`
`224`	`224`	`ChipLogProgress(chipTool, "Instance Name: %s ", instance.first.c_str());`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ CHIP_ERROR DiscoverCommissionersCommand::RunCommand()`
`29`	`29`
`30`	`30`	`void DiscoverCommissionersCommand::Shutdown()`
`31`	`31`	`{`
`32`		`- int commissionerCount = 0;`
	`32`	`+ [[maybe_unused]] int commissionerCount = 0;`
`33`	`33`	`for (int i = 0; i < CHIP_DEVICE_CONFIG_MAX_DISCOVERED_NODES; i++)`
`34`	`34`	`{`
`35`	`35`	`const Dnssd::DiscoveredNodeData * commissioner = mCommissionableNodeController.GetDiscoveredCommissioner(i);`
Original file line number	Diff line number	Diff line change
`@@ -55,21 +55,18 @@ LDMA_TransferCfg_t ldmaRXConfig;`
`55`	`55`
`56`	`56`	`static osSemaphoreId_t transfer_done_semaphore = NULL;`
`57`	`57`
`58`		`-static bool dma_callback(unsigned int channel, unsigned int sequenceNo, void * userParam)`
	`58`	`+static bool dma_callback([[maybe_unused]] unsigned int channel, [[maybe_unused]] unsigned int sequenceNo,`
	`59`	`+ [[maybe_unused]] void * userParam)`
`59`	`60`	`{`
`60`		`- UNUSED_PARAMETER(channel);`
`61`		`- UNUSED_PARAMETER(sequenceNo);`
`62`		`- UNUSED_PARAMETER(userParam);`
`63`	`61`	`#if defined(SL_CATLOG_POWER_MANAGER_PRESENT)`
`64`	`62`	`sl_power_manager_remove_em_requirement(SL_POWER_MANAGER_EM1);`
`65`	`63`	`#endif`
`66`	`64`	`osSemaphoreRelease(transfer_done_semaphore);`
`67`	`65`	`return false;`
`68`	`66`	`}`
`69`	`67`
`70`		`-static void gpio_interrupt(uint8_t interrupt_number)`
	`68`	`+static void gpio_interrupt([[maybe_unused]] uint8_t interrupt_number)`
`71`	`69`	`{`
`72`		`- UNUSED_PARAMETER(interrupt_number);`
`73`	`70`	`sl_si91x_host_set_bus_event(NCP_HOST_BUS_RX_EVENT);`
`74`	`71`	`// GPIO_IntClear(0xAAAA);`
`75`	`72`	`}`
`@@ -299,4 +296,4 @@ void sl_si91x_host_disable_bus_interrupt(void)`
`299`	`296`	`bool sl_si91x_host_is_in_irq_context(void)`
`300`	`297`	`{`
`301`	`298`	`return (SCB->ICSR & SCB_ICSR_VECTACTIVE_Msk) != 0U;`
`302`		`-}`
	`299`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ void InitCommissioningFlow(intptr_t commandArg)`
`105`	`105`	`commissioner->LogDetail();`
`106`	`106`	`if (associatedConnectableVideoPlayer.HasValue())`
`107`	`107`	`{`
`108`		`- TargetVideoPlayerInfo * targetVideoPlayerInfo = associatedConnectableVideoPlayer.Value();`
	`108`	`+ [[maybe_unused]] TargetVideoPlayerInfo * targetVideoPlayerInfo = associatedConnectableVideoPlayer.Value();`
`109`	`109`	`ChipLogProgress(AppServer, "Previously connected with nodeId 0x" ChipLogFormatX64 " fabricIndex: %d",`
`110`	`110`	`ChipLogValueX64(targetVideoPlayerInfo->GetNodeId()), targetVideoPlayerInfo->GetFabricIndex());`
`111`	`111`	`}`
`@@ -305,7 +305,7 @@ void PrintFabrics()`
`305`	`305`	`ChipLogError(AppServer, " -- Not initialized");`
`306`	`306`	`continue;`
`307`	`307`	`}`
`308`		`- NodeId myNodeId = fb.GetNodeId();`
	`308`	`+ [[maybe_unused]] NodeId myNodeId = fb.GetNodeId();`
`309`	`309`	`ChipLogProgress(NotSpecified,`
`310`	`310`	`"---- Current Fabric nodeId=0x" ChipLogFormatX64 " fabricId=0x" ChipLogFormatX64 " fabricIndex=%d",`
`311`	`311`	`ChipLogValueX64(myNodeId), ChipLogValueX64(fb.GetFabricId()), fabricIndex);`
Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,7 @@ void CastingPlayer::LogDetail() const`
`257`	`257`	`{`
`258`	`258`	`for (unsigned j = 0; j < mAttributes.numIPs; j++)`
`259`	`259`	`{`
`260`		`- char * ipAddressOut = mAttributes.ipAddresses[j].ToString(buf);`
	`260`	`+ [[maybe_unused]] char * ipAddressOut = mAttributes.ipAddresses[j].ToString(buf);`
`261`	`261`	`ChipLogDetail(AppServer, "\tIP Address #%d: %s", j + 1, ipAddressOut);`
`262`	`262`	`}`
`263`	`263`	`}`
Original file line number	Diff line number	Diff line change
`@@ -695,7 +695,7 @@ void CastingServer::SetDefaultFabricIndex(std::function<void(TargetVideoPlayerIn`
`695`	`695`	`ChipLogError(AppServer, " -- Not initialized");`
`696`	`696`	`continue;`
`697`	`697`	`}`
`698`		`- NodeId myNodeId = fb.GetNodeId();`
	`698`	`+ [[maybe_unused]] NodeId myNodeId = fb.GetNodeId();`
`699`	`699`	`ChipLogProgress(NotSpecified,`
`700`	`700`	`"---- Current Fabric nodeId=0x" ChipLogFormatX64 " fabricId=0x" ChipLogFormatX64 " fabricIndex=%d",`
`701`	`701`	`ChipLogValueX64(myNodeId), ChipLogValueX64(fb.GetFabricId()), fabricIndex);`
Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,8 @@ using namespace ::chip::app::Clusters;`
`38`	`38`	`void MatterPostAttributeChangeCallback(const chip::app::ConcreteAttributePath & attributePath, uint8_t type, uint16_t size,`
`39`	`39`	`uint8_t * value)`
`40`	`40`	`{`
`41`		`- ClusterId clusterId = attributePath.mClusterId;`
`42`		`- AttributeId attributeId = attributePath.mAttributeId;`
	`41`	`+ [[maybe_unused]] ClusterId clusterId = attributePath.mClusterId;`
	`42`	`+ [[maybe_unused]] AttributeId attributeId = attributePath.mAttributeId;`
`43`	`43`	`ChipLogProgress(Zcl, "AttributeChangeCallback cluster: " ChipLogFormatMEI " attribute:" ChipLogFormatMEI,`
`44`	`44`	`ChipLogValueMEI(clusterId), ChipLogValueMEI(attributeId));`
`45`	`45`	`}`