microsoft/Terminal

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

Aperta

#2675 aperta il 5 set 2019

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (9275 fork)batch import
Area-CodeHealthArea-OutputHelp WantedIssue-TaskProduct-ConhostProduct-Terminal

Metriche repository

Star
 (103.173 stelle)
Metriche merge PR
 (Merge medio 27g 19h) (24 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Ispeziona l'avviso soppresso in OutputCellIterator e identifica perché l'iteratore avanzato attiva l'avviso. Quindi risolvi l'avviso o implementa una corretta inizializzazione/distruzione.
Tech stack
cpp
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65