Possible memory leak when reloading images in MarkdownTextBlock
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- csharp
- Área
- desktop-dev
Línea de trabajo
Comienza con las rutas de los handlers MarkdownTextBlock Init() y LoadImage descritas en el issue, centrándote en la sustitución de la instancia _image mediante IImageProvider. Establece un caso reproducible de recarga de imagen y, a continuación, verifica mediante una comprobación de fugas adecuada que al sustituir imágenes no se conserven handlers Loaded ni objetos obsoletos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
Found by Copilot in https://github.com/CommunityToolkit/Labs-Windows/pull/771/
When using IImageProvider, the _image instance is replaced with a new Image object from the provider. This means the Loaded event handler attached in Init() is lost, since it was attached to the original _image instance. However, since this code is already executing within the LoadImage handler and _loaded is set to true immediately after, this might work correctly. Consider whether the event handler should be detached from the old image or if there's a risk of memory leaks from the orphaned handler on the original image object.
Steps to reproduce
Unknown, need investigation.
Expected behavior
No memory leaks when images are changed.
Screenshots
No response
Code Platform
- UWP
- WinAppSDK / WinUI 3
- Web Assembly (WASM)
- Android
- iOS
- MacOS
- Linux / GTK
Windows Build Number
- Windows 10 1809 (Build 17763)
- Windows 10 1903 (Build 18362)
- Windows 10 1909 (Build 18363)
- Windows 10 2004 (Build 19041)
- Windows 10 20H2 (Build 19042)
- Windows 10 21H1 (Build 19043)
- Windows 11 21H2 (Build 22000)
- Other (specify)
Other Windows Build number
No response
App minimum and target SDK version
- Windows 10, version 1809 (Build 17763)
- Windows 10, version 1903 (Build 18362)
- Windows 10, version 1909 (Build 18363)
- Windows 10, version 2004 (Build 19041)
- Other (specify)
Other SDK version
No response
Visual Studio Version
No response
Visual Studio Build Number
No response
Device form factor
No response
Additional context
No response
Help us help you
Yes, I'd like to be assigned to work on this item.
- Lenguaje dominante
- C#
- Estrellas
- 483
- Forks
- 89
- Merge medio
- 4 d 5 h
- PR fusionados (30 d)
- 2
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 CommunityToolkit/Labs-Windows
-
bug :bug:
Dificultad 1/5 1-3 horas Aptitud para principiantes 80/100
CommunityToolkit/Labs-Windows#815 ·
-
bug :bug:
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
CommunityToolkit/Labs-Windows#816 ·
-
bug :bug:
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
CommunityToolkit/Labs-Windows#814 ·
-
experiment :test_tube:
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
CommunityToolkit/Labs-Windows#812 ·
-
ux :paintbrush:
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
CommunityToolkit/Labs-Windows#808 · 1 comentario ·
Todos los issues de CommunityToolkit/Labs-Windows
Issues similares
-
Bug pulumi/pulumi
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
activescott/lessmsi#306 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
HTML sitemap lists unpublished pagesPosiblemente ocupada @KrzysztofPajak la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
grandnode/grandnode2#883 ·
Los mantenedores suelen responder en 1 día
-
[Bug]: `winapp ui <command> --on sandbox --help` starts sandbox setup instead of showing helpAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día