microsoft/Terminal

Memory leak and other issues when saving settings

Open

#10 609 ouverte le 11 juil. 2021

Voir sur GitHub
 (9 commentaires) (0 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-SettingsUIExternal-Blocked-WinUI3Help WantedIssue-BugPriority-1Product-Terminal

Métriques du dépôt

Stars
 (35 764 stars)
Métriques de merge PR
 (Merge moyen 27j 19h) (24 PRs mergées en 30 j)

Description

Windows Terminal version (or Windows build number)

1.8.1521.0 (also observed on source build 2021-07-11)

Other Software

No response

Steps to reproduce

Open the settings dialog and spam click save. No option needs to be changed.

Video: 13 seconds in, the settings could not be reloaded from file (not sure if it is not loading or file corruption). https://user-images.githubusercontent.com/2277504/125179830-a31f3a00-e1f2-11eb-8b6f-6f6aa07896c5.mp4

Expected Behavior

Memory stays roughly the same, settings file can be loaded after saving, main UI does not extend past its borders.

Actual Behavior

Memory grows and is not reclaimed, settings file can sometimes not be loaded after saving, UI grows past its borders during reloading.

(Second and third only discovered after deciding to record a video for the memory leak)

Not sure if you want separate tracking issues for these, let me know.

Guide contributeur

Direction de recherche
Examinez le chemin du code de sauvegarde des paramètres dans Windows Terminal. Recherchez les modèles d'allocation de mémoire sans désallocation, en particulier dans l'opération de sauvegarde de la boîte de dialogue des paramètres. La vidéo montre une croissance de la mémoire et des problèmes d'interface utilisateur ; examinez le rechargement du fichier après la sauvegarde. Les fichiers se trouvent probablement dans le répertoire src/ liés aux paramètres et au contrôle du terminal. Vérifiez les commentaires pour des indices supplémentaires.
Stack technique
cpp
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++Windows Terminal internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30