feat(matter): use C++ nullptr instead of NULL C macro

Author: SuGlider

libraries/Matter/src/Matter.cpp

@@ -28,8 +28,8 @@ constexpr auto k_timeout_seconds = 300;
 
 static bool _matter_has_started = false;
 static node::config_t node_config;
-static node_t *deviceNode = NULL;
-ArduinoMatter::matterEventCB ArduinoMatter::_matterEventCB = NULL;
+static node_t *deviceNode = nullptr;
+ArduinoMatter::matterEventCB ArduinoMatter::_matterEventCB = nullptr;
 
 // This callback is called for every attribute update. The callback implementation shall
 // handle the desired attributes and return an appropriate error code. If the attribute
@@ -43,7 +43,7 @@ static esp_err_t app_attribute_update_cb(
   switch (type) {
     case PRE_UPDATE:  // Callback before updating the value in the database
       log_v("Attribute update callback: PRE_UPDATE");
-      if (ep != NULL) {
+      if (ep != nullptr) {
         err = ep->attributeChangeCB(endpoint_id, cluster_id, attribute_id, val) ? ESP_OK : ESP_FAIL;
       }
       break;
@@ -79,7 +79,7 @@ static esp_err_t app_identification_cb(identification::callback_type_t type, uin
     identifyIsActive = false;
     log_v("Identification callback: STOP");
   }
-  if (ep != NULL) {
+  if (ep != nullptr) {
     err = ep->endpointIdentifyCB(endpoint_id, identifyIsActive) ? ESP_OK : ESP_FAIL;
   }
 
@@ -124,7 +124,7 @@ static void app_event_cb(const ChipDeviceEvent *event, intptr_t arg) {
     default:                                                       break;
   }
   // Check if the user-defined callback is set
-  if (ArduinoMatter::_matterEventCB != NULL) {
+  if (ArduinoMatter::_matterEventCB != nullptr) {
     ArduinoMatter::_matterEventCB(static_cast<ArduinoMatter::matterEvent_t>(event->Type), event);
   }
 }

libraries/Matter/src/MatterEndPoint.h

@@ -41,30 +41,30 @@ class MatterEndPoint {
   esp_matter::attribute_t *getAttribute(uint32_t cluster_id, uint32_t attribute_id) {
     if (endpoint_id == 0) {
       log_e("Endpoint ID is not set");
-      return NULL;
+      return nullptr;
     }
     endpoint_t *endpoint = endpoint::get(node::get(), endpoint_id);
-    if (endpoint == NULL) {
+    if (endpoint == nullptr) {
       log_e("Endpoint [%d] not found", endpoint_id);
-      return NULL;
+      return nullptr;
     }
     cluster_t *cluster = cluster::get(endpoint, cluster_id);
-    if (cluster == NULL) {
+    if (cluster == nullptr) {
       log_e("Cluster [%d] not found", cluster_id);
-      return NULL;
+      return nullptr;
     }
     esp_matter::attribute_t *attribute = attribute::get(cluster, attribute_id);
-    if (attribute == NULL) {
+    if (attribute == nullptr) {
       log_e("Attribute [%d] not found", attribute_id);
-      return NULL;
+      return nullptr;
     }
     return attribute;
   }
 
   // get the value of an attribute from its cluster id and attribute it
   bool getAttributeVal(uint32_t cluster_id, uint32_t attribute_id, esp_matter_attr_val_t *attrVal) {
     esp_matter::attribute_t *attribute = getAttribute(cluster_id, attribute_id);
-    if (attribute == NULL) {
+    if (attribute == nullptr) {
       return false;
     }
     if (attribute::get_val(attribute, attrVal) == ESP_OK) {
@@ -78,7 +78,7 @@ class MatterEndPoint {
   // set the value of an attribute from its cluster id and attribute it
   bool setAttributeVal(uint32_t cluster_id, uint32_t attribute_id, esp_matter_attr_val_t *attrVal) {
     esp_matter::attribute_t *attribute = getAttribute(cluster_id, attribute_id);
-    if (attribute == NULL) {
+    if (attribute == nullptr) {
       return false;
     }
     if (attribute::set_val(attribute, attrVal) == ESP_OK) {
@@ -117,6 +117,6 @@ class MatterEndPoint {
 
 protected:
   uint16_t endpoint_id = 0;
-  EndPointIdentifyCB _onEndPointIdentifyCB = NULL;
+  EndPointIdentifyCB _onEndPointIdentifyCB = nullptr;
 };
 #endif /* CONFIG_ESP_MATTER_ENABLE_DATA_MODEL */