microsoft/Terminal

[Performance] Fast path for `RenderData::IsCursorDoubleWidth`

Open

#10.470 aberto em 21 de jun. de 2021

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-PerformanceHelp WantedIssue-TaskProduct-Conhost

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

This is a sister issue of #10393.

Currently an unreasonable amount of CPU is wasted in checking IsCursorDoubleWidth in the buffer, due to the expensive construction of TextBufferCellIterator. I think it's fair to find a fast path for people who don't use width characters at all.

image

Guia do colaborador

Direção de pesquisa
Perfile a função IsCursorDoubleWidth e implemente um caminho rápido que ignore a iteração cara quando não houver caracteres largos.
Pilha de tecnologia
cpp
Domain
performance
Tipo Issue
Performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Windows Terminal development
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60