Elimated unncessary iteration of file system.

Caching the palette count globally, rather than iterating the SPFFS each time a count was needed.  Update cached count on OTA upload of SPIFFS, or when edited via the web. Improves performance when switching palettes.

Griswold-LED-Controller.ino

@@ -173,6 +173,7 @@ void setup() {
   ArduinoOTA.onEnd([]() { 
     DBG_OUTPUT_PORT.println("\nEnd... remounting SPIFFS");
     SPIFFS.begin();
+    paletteCount = getPaletteCount();
   });
   ArduinoOTA.onProgress([](unsigned int progress, unsigned int total) {
     DBG_OUTPUT_PORT.printf("Progress: %u%%\r", (progress / (total / 100)));
@@ -420,9 +421,8 @@ void setup() {
     settings.mode = PALETTE_ANIMS;
     if (server.arg("p") != "") {
       uint8_t pal = (uint8_t) strtol(server.arg("p").c_str(), NULL, 10);
-      int numberOfPalettes = getPaletteCount();
-      if (pal > numberOfPalettes) 
-        pal = numberOfPalettes;
+      if (pal > paletteCount) 
+        pal = paletteCount;
 
       settings.palette_ndx = pal;
       loadPaletteFromFile(settings.palette_ndx, &targetPalette);
@@ -433,6 +433,8 @@ void setup() {
   });
 
   server.begin();
+
+  paletteCount = getPaletteCount();
 }
 
 void loop() {

colormodes.h

@@ -139,7 +139,7 @@ void ChangePalettePeriodically(bool forceNow) {
   if (forceNow || millis() - paletteMillis > (settings.show_length * 1000)) {
     paletteMillis = millis();
 
-    targetPaletteIndex = random(0, getPaletteCount());
+    targetPaletteIndex = random(0, paletteCount);
 
     currentPalette = targetPalette;
 

palettes.h

@@ -20,6 +20,8 @@
 // The LEDLAMP project is a fork of the McLighting Project at
 //        https://github.com/toblum/McLighting
 
+int paletteCount = 0;
+
 int getPaletteCount() {
   Dir dir = SPIFFS.openDir("/palettes");
   int palette_count = 0;

request_handlers.h

@@ -375,8 +375,7 @@ void webSocketEvent(uint8_t num, WStype_t type, uint8_t * payload, size_t length
           if (payload[1] == '+') {
             DBG_OUTPUT_PORT.printf("Current pallet_ndx=%d\n", settings.palette_ndx);      
             settings.palette_ndx++;
-            int numberOfPalettes = getPaletteCount();
-            if (settings.palette_ndx >= numberOfPalettes) {
+            if (settings.palette_ndx >= paletteCount) {
               settings.palette_ndx = 0;              
             }
 

spiffs_webserver.h

@@ -95,6 +95,10 @@ void handleFileUpload() {
 			fsUploadFile.close();
 		DBG_OUTPUT_PORT.print("handleFileUpload Size: "); DBG_OUTPUT_PORT.println(upload.totalSize);
 	}
+
+  // Update the paletteCount in case someone uploaded one.
+  paletteCount = getPaletteCount();
+
 }
 
 void handleFileDelete() {
@@ -108,6 +112,10 @@ void handleFileDelete() {
 	SPIFFS.remove(path);
 	server.send(200, "text/plain", "");
 	path = String();
+
+  // Update the paletteCount in case someone deleted one.
+  paletteCount = getPaletteCount();
+
 }
 
 void handleFileCreate() {
@@ -126,6 +134,9 @@ void handleFileCreate() {
 		return server.send(500, "text/plain", "CREATE FAILED");
 	server.send(200, "text/plain", "");
 	path = String();
+
+  // Update the paletteCount in case someone created one.
+  paletteCount = getPaletteCount();
 }
 
 void handleFileList() {