Fix GC-Ruckler in pickJob und tickVillager #35

claude · 2026-03-23T20:18:42Z

claude commented

2026-03-23 20:18:42 +00:00

Fixes #34.

What changed

pickJob: alle Object.values()-Aufrufe einmal oben extrahiert — insbesondere der Forester-Block rief zuvor 4x Object.values() pro Zone-Tile auf
JOB_ICONS als Modul-Konstante statt per-Frame-Objekt
Math.min(...candidates.map(...)) durch einfache Schleife ersetzt

Fixes #34. ## What changed - `pickJob`: alle `Object.values()`-Aufrufe einmal oben extrahiert — insbesondere der Forester-Block rief zuvor 4x Object.values() pro Zone-Tile auf - `JOB_ICONS` als Modul-Konstante statt per-Frame-Objekt - `Math.min(...candidates.map(...))` durch einfache Schleife ersetzt

claude added 2 commits 2026-03-23 20:18:42 +00:00

📝 update CHANGELOG for depth sorting (PR #32 ) c7cf971e54

⚡ fix GC ruckler — Object.values() einmal pro pickJob-Aufruf d02ed33435

Fixes #34. Alle Object.values()-Aufrufe werden einmal am Anfang von
pickJob() extrahiert und in allen Branches wiederverwendet. Der
Forester-Loop rief zuvor fuer jedes Zone-Tile 4x Object.values() auf.
JOB_ICONS als Modul-Konstante, Math.min-spread durch Schleife ersetzt.

tekki approved these changes 2026-03-23 20:28:32 +00:00

tekki merged commit 26c3807481 into master

2026-03-23 20:28:46 +00:00

tekki referenced this issue from a commit

2026-03-23 20:28:47 +00:00

Merge pull request 'Fix GC-Ruckler in pickJob und tickVillager' (#35) from fix/gc-alloc-picjob into master

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: tekki/nissefolk#35