microsoft/Terminal

[Audit] Eliminate raw pointers as begin/end markers in the CharRowCellReference class

Open

#2672 aperta il 5 set 2019

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

Metriche repository

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

Descrizione

When turning on audit mode for the output buffer project, I ran into the circumstance where CharRowCellReference is using raw pointers as its iterator definitions for begin and end, which makes static analysis very unhappy as it still results in pointer math.

This represents adjusting CharRowCellReference to perform iteration in a way that audit finds palatable.

Guida contributor

Direzione di ricerca
Ispeziona la classe CharRowCellReference e le sue definizioni di iteratore. Sostituisci i marcatori begin/end a puntatori grezzi con un tipo di iteratore appropriato che eviti l'aritmetica dei puntatori, come un wrapper o un iteratore basato su indice.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75