Skip to content

Commit 84669c3

Browse files
committed
hotfix for #367
over-optimization - overlooked nested if when removing MAX_LED_MEMORY limitations
1 parent 49baa0a commit 84669c3

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

wled00/cfg.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -202,9 +202,10 @@ bool deserializeConfig(JsonObject doc, bool fromFS) {
202202
if (fromFS) {
203203
BusConfig bc = BusConfig(ledType, pins, start, length, colorOrder, reversed, skipFirst, AWmode, freqkHz, artnet_outputs, artnet_leds_per_output, artnet_fps_limit);
204204
mem += BusManager::memUsage(bc);
205-
#if !defined(ARDUINO_ARCH_ESP32) || defined(CONFIG_IDF_TARGET_ESP32S2) || defined(CONFIG_IDF_TARGET_ESP32C3) // WLEDMM only for boards with small RAM
206-
if (mem <= MAX_LED_MEMORY) if (busses.add(bc) == -1) break; // finalization will be done in WLED::beginStrip()
205+
#if !defined(ARDUINO_ARCH_ESP32) || defined(CONFIG_IDF_TARGET_ESP32S2) || defined(CONFIG_IDF_TARGET_ESP32C3)
206+
if (mem <= MAX_LED_MEMORY) // WLEDMM only apply memory limits for boards with small RAM
207207
#endif
208+
if (busses.add(bc) == -1) break; // finalization will be done in WLED::beginStrip()
208209
} else {
209210
if (busConfigs[s] != nullptr) delete busConfigs[s];
210211
busConfigs[s] = new BusConfig(ledType, pins, start, length, colorOrder, reversed, skipFirst, AWmode, freqkHz, artnet_outputs, artnet_leds_per_output, artnet_fps_limit);

0 commit comments

Comments
 (0)