Consider detaching game from DevToolsService when not needed
I maintainer di solito rispondono entro 1 giorno
@filiph ci sta già lavorando.
Dal 21/9/2024.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
What could be improved
Not a big deal, but currently, when running a game in debug mode, any FlameGame that is ever started will forever be retained in memory unless replaced with a different FlameGame.
From FlameGame's constructor:
if (kDebugMode) {
DevToolsService.initWithGame(this);
}
There is no DevToolsService.detachGame() or anything similar.
Why should this be improved
This can be confusing when one's looking at memory (albeit I agree that one should be normally doing that in profile mode) and sees a game that's supposed to be gone a long time ago, and won't go away even after manual GC.
This almost sent me on a wild goose chase for a giant game-sized memory leak.
Risks
- Some people might want to inspect their game after it's been disposed?
- Could introduce a DevTools connector bug where game's disposed too soon, for example.
More information
I'm not sure myself if this is something worth addressing. Just putting this here before I forget.
Other
- Are you interested in working on a PR for this?
- Lingua principale
- Dart
- Stelle
- 10.8k
- Fork
- 1.1k
- Merge medio
- 1g 21h
- PR unite (30g)
- 50
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di flame-engine/flame
-
`PostProcessComponent` throws `StateError` when it has no `PositionComponent` childrenForse già presa @ufrshubham l’ha presa 1 giorno fa. Apertabug
flame-engine/flame#4182 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
flame-engine/flame#4176 ·
I maintainer di solito rispondono entro 1 giorno
-
discussion enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
flame-engine/flame#4175 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
flame-engine/flame#4174 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
flame-engine/flame#4173 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di flame-engine/flame
Issue simili
-
dart easy first-contributor first-contributors good first issue help wanted up-for-grabs
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
fork-commit-merge/fork-commit-merge#8563 ·
I maintainer di solito rispondono entro 1 giorno
-
from.page-issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
bannzai/Pilll#2080 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
mollykitten/waveform#112 ·
I maintainer di solito rispondono entro 1 giorno
-
auto-implement-claude
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
I maintainer di solito rispondono entro 1 giorno