Consider detaching game from DevToolsService when not needed
Los mantenedores suelen responder en 1 día
@filiph ya está trabajando en esto.
Desde el 21/9/2024.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
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?
- Lenguaje dominante
- Dart
- Estrellas
- 10.8k
- Forks
- 1.1k
- Merge medio
- 1 d 21 h
- PR fusionados (30 d)
- 62
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de flame-engine/flame
-
`Viewfinder.visibleGameSize` is not reapplied when the viewport changes sizePosiblemente ocupada @ufrshubham la tomó hoy. Abiertobug
flame-engine/flame#4204 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
`Viewfinder.visibleGameSize` zooms in too far with a `FixedResolutionViewport`Posiblemente ocupada @ufrshubham la tomó hoy. Abiertobug
flame-engine/flame#4203 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Camera bounds with `considerViewport: true` are not updated when the viewport is resizedPosiblemente ocupada @ufrshubham la tomó hoy. Abiertobug
flame-engine/flame#4202 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
`CameraComponent.setBounds` with `considerViewport: true` only works for bounds that start at (0, 0), at zoom 1Posiblemente ocupada @ufrshubham la tomó hoy. Abiertobug
flame-engine/flame#4201 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
`findPostProcessFromContext` returns `null` for components rendered with `CameraComponent.postProcess`Posiblemente ocupada @ufrshubham la tomó hoy. Abiertobug
flame-engine/flame#4200 · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de flame-engine/flame
Issues similares
-
[CarouselView] Crash (RangeError / ArgumentError) when children or itemCount is empty with infinite: true or CarouselController.animateToItemPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertop: material_ui package platform-android platform-ios platform-linux platform-macos platform-web team-design
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
flutter/flutter#195878 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[genkit_google_genai] Gemini tool call with no arguments fails with "requires a non-null input"Posiblemente ocupada @kenzieschmoll la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
genkit-ai/genkit-dart#692 ·
Los mantenedores suelen responder en 11 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
lollipopkit/flutter_server_box#1851 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día