Minor refactor: replace for break with MIN()

mrv96 · mrv96 · commit 59592f04bb65 · 2025-08-26T11:46:32.000Z
diff --git a/wled00/cfg.cpp b/wled00/cfg.cpp
@@ -751,14 +751,12 @@ bool deserializeConfig(JsonObject doc, bool fromFS) {
   CJSON(DMXStartLED,dmx[F("start-led")]);
 
   JsonArray dmx_fixmap = dmx[F("fixmap")];
-  for (int i = 0; i < dmx_fixmap.size(); i++) {
-    if (i > 14) break;
+  for (int i = 0; i < MIN(dmx_fixmap.size(), 15); i++) {
     CJSON(DMXFixtureMap[i],dmx_fixmap[i]);
   }
 
   JsonArray dmx_chsval = dmx[F("chsval")];
-  for (int i = 0; i < dmx_chsval.size(); i++) {
-    if (i > 14) break;
+  for (int i = 0; i < MIN(dmx_chsval.size(), 15); i++) {
     CJSON(DMXChannelsValue[i],dmx_chsval[i]);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -751,14 +751,12 @@ bool deserializeConfig(JsonObject doc, bool fromFS) {`
`751`	`751`	`CJSON(DMXStartLED,dmx[F("start-led")]);`
`752`	`752`
`753`	`753`	`JsonArray dmx_fixmap = dmx[F("fixmap")];`
`754`		`- for (int i = 0; i < dmx_fixmap.size(); i++) {`
`755`		`- if (i > 14) break;`
	`754`	`+ for (int i = 0; i < MIN(dmx_fixmap.size(), 15); i++) {`
`756`	`755`	`CJSON(DMXFixtureMap[i],dmx_fixmap[i]);`
`757`	`756`	`}`
`758`	`757`
`759`	`758`	`JsonArray dmx_chsval = dmx[F("chsval")];`
`760`		`- for (int i = 0; i < dmx_chsval.size(); i++) {`
`761`		`- if (i > 14) break;`
	`759`	`+ for (int i = 0; i < MIN(dmx_chsval.size(), 15); i++) {`
`762`	`760`	`CJSON(DMXChannelsValue[i],dmx_chsval[i]);`
`763`	`761`	`}`
`764`	`762`