[QUALITY] Remove dead code and empty placeholder methods #23

New issue

Open

opened 2026-03-12 00:03:38 +00:00 by lavarius · 0 comments

lavarius commented

2026-03-12 00:03:38 +00:00

Owner

🟡 Medium — Code Quality

Dead code and placeholder stubs throughout the project add noise and confusion:

FindMainCameraForCanvas.cs — empty Update() method
BoothModeExitButton.cs, Railing.cs — all methods empty
CharacterStore.ChangeLayerRecursively() — never called
GridMapGenerator.MakeBlocksFromRoads(), .ScatterProps() — empty placeholder stubs
MovementManager.cs L225-243 — large block of commented-out old zone logic

Fix:

Delete truly dead files/methods
Mark placeholder stubs with // TODO: and a brief description of intended behaviour
Remove commented-out code (it's in git history if needed)

## 🟡 Medium — Code Quality Dead code and placeholder stubs throughout the project add noise and confusion: - `FindMainCameraForCanvas.cs` — empty `Update()` method - `BoothModeExitButton.cs`, `Railing.cs` — all methods empty - `CharacterStore.ChangeLayerRecursively()` — never called - `GridMapGenerator.MakeBlocksFromRoads()`, `.ScatterProps()` — empty placeholder stubs - `MovementManager.cs` L225-243 — large block of commented-out old zone logic **Fix:** - Delete truly dead files/methods - Mark placeholder stubs with `// TODO:` and a brief description of intended behaviour - Remove commented-out code (it's in git history if needed)