microsoft/Terminal

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

Open

#2.672 aberto em 5 de set. de 2019

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-CodeHealthArea-OutputHelp WantedIssue-TaskProduct-ConhostProduct-Terminal

Métricas do repositório

Stars
 (35.764 stars)
Métricas de merge de PR
 (Mesclagem média 27d 19h) (24 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Inspecione a classe CharRowCellReference e suas definições de iterador. Substitua os marcadores begin/end de ponteiro bruto por um tipo de iterador adequado que evite aritmética de ponteiros, como um wrapper ou iterador baseado em índice.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75