cleanup

esphome/components/esp32_ble/ble_event.h

@@ -203,6 +203,7 @@ class BLEEvent {
         break;
 
       default:
+        // We only handle 4 GAP event types, others are dropped
         break;
     }
   }

esphome/components/esp32_ble/ble_event_pool.h

@@ -42,14 +42,17 @@ template<uint8_t SIZE> class BLEEventPool {
       RAMAllocator<BLEEvent> allocator(RAMAllocator<BLEEvent>::ALLOC_INTERNAL);
       event = allocator.allocate(1);
 
-      if (event != nullptr) {
-        // Placement new to construct the object
-        new (event) BLEEvent();
-        this->total_created_++;
+      if (event == nullptr) {
+        // Memory allocation failed
+        return nullptr;
       }
+
+      // Placement new to construct the object
+      new (event) BLEEvent();
+      this->total_created_++;
     }
 
-    return event;  // Will be nullptr if allocation failed
+    return event;
   }
 
   // Return an event to the pool