Reflect using dtor instead

adafruit · Mar 6, 2025 · 393fffa · 393fffa
1 parent c6f15b5
commit 393fffa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 12 deletions.
diff --git a/src/components/pixels/ws_pixels.cpp b/src/components/pixels/ws_pixels.cpp
@@ -69,11 +69,6 @@ void ws_pixels::deallocateStrand(int16_t strandIdx) {
     // Fill with "off"
     strands[strandIdx].neoPixelPtr->clear();
     strands[strandIdx].neoPixelPtr->show();
-#ifdef ARDUINO_ARCH_ESP32
-    // Clear the pin used for RMT
-    strands[strandIdx].neoPixelPtr->updateLength(0);
-    strands[strandIdx].neoPixelPtr->show();
-#endif
     // Delete the NeoPixel object
     delete strands[strandIdx].neoPixelPtr;
   } else if ((strands[strandIdx].dotStarPtr != nullptr)) {

diff --git a/src/components/statusLED/Wippersnapper_StatusLED.cpp b/src/components/statusLED/Wippersnapper_StatusLED.cpp
@@ -103,14 +103,10 @@ void initStatusLED() {
 void releaseStatusLED() {
   WS_DEBUG_PRINTLN("Releasing status LED");
 #ifdef USE_STATUS_NEOPIXEL
-#ifdef ARDUINO_ARCH_ESP32
-  // Release the rmtPin for use by other peripherals
-  statusPixel->updateLength(0);
-  statusPixel->show();
-#endif
-  // Dealloc. NeoPixel object
+  // Deallocate Adafruit_NeoPixel object, set data pin back to INPUT,
+  // and unlock pixel for use by pixels component
   delete statusPixel;
-  WS.lockStatusNeoPixel = false; // unlock
+  WS.lockStatusNeoPixel = false;
 #endif
 
 #ifdef USE_STATUS_DOTSTAR