Refactor tile worker startup for robustness and efficiency (#88)

CelliesProjects · web-flow · commit 8688ba8e18a6 · 2025-05-28T15:08:04.000+02:00
- Moved PNG decoder availability check to startTileWorkerTasks()
diff --git a/src/OpenStreetMap-esp32.cpp b/src/OpenStreetMap-esp32.cpp
@@ -513,6 +513,9 @@ void OpenStreetMap::tileFetcherTask(void *param)
 
 bool OpenStreetMap::startTileWorkerTasks()
 {
+    if (tasksStarted)
+        return true;
+
     if (!jobQueue)
     {
         jobQueue = xQueueCreate(OSM_JOB_QUEUE_SIZE, sizeof(TileJob));
@@ -523,9 +526,6 @@ bool OpenStreetMap::startTileWorkerTasks()
         }
     }
 
-    if (tasksStarted)
-        return true;
-
     numberOfWorkers = OSM_FORCE_SINGLECORE ? 1 : ESP.getChipCores();
     for (int core = 0; core < numberOfWorkers; ++core)
     {

Original file line number	Diff line number	Diff line change
`@@ -513,6 +513,9 @@ void OpenStreetMap::tileFetcherTask(void *param)`
`513`	`513`
`514`	`514`	`bool OpenStreetMap::startTileWorkerTasks()`
`515`	`515`	`{`
	`516`	`+ if (tasksStarted)`
	`517`	`+ return true;`
	`518`	`+`
`516`	`519`	`if (!jobQueue)`
`517`	`520`	`{`
`518`	`521`	`jobQueue = xQueueCreate(OSM_JOB_QUEUE_SIZE, sizeof(TileJob));`
`@@ -523,9 +526,6 @@ bool OpenStreetMap::startTileWorkerTasks()`
`523`	`526`	`}`
`524`	`527`	`}`
`525`	`528`
`526`		`- if (tasksStarted)`
`527`		`- return true;`
`528`		`-`
`529`	`529`	`numberOfWorkers = OSM_FORCE_SINGLECORE ? 1 : ESP.getChipCores();`
`530`	`530`	`for (int core = 0; core < numberOfWorkers; ++core)`
`531`	`531`	`{`