microsoft/Terminal

[Audit] Unsuppress OuptutCellIterator warning and solve custom construction/destruction warning

Open

#2 675 ouverte le 5 sept. 2019

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-CodeHealthArea-OutputHelp WantedIssue-TaskProduct-ConhostProduct-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

I suppressed the warning on the OutputCellIterator as a part of turning on audit mode because I couldn't figure it out (in a reasonable amount of time for how many other things I was fixing and trying to get turned on).

This represents removing the warning suppression and figuring out what the actual issue is that is causing the warning to flag when the iterator is advanced.

Guide contributeur

Direction de recherche
Inspectez l'avertissement supprimé dans OutputCellIterator et identifiez pourquoi l'itérateur avancé déclenche l'avertissement. Corrigez ensuite l'avertissement ou implémentez une initialisation/destruction appropriée.
Stack technique
cpp
Domaine
cli
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-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65