llvm/llvm-project

include/clang/Tooling/Refactoring/Rename/RenamingAction.h:79]: (performance)

Open

#34.652 aberto em 14 de nov. de 2017

Ver no GitHub
 (6 comments) (0 reactions) (0 assignees)C++ (10.782 forks)batch import
bugzillaclang:toolingconfirmedgood first issue

Métricas do repositório

Stars
 (26.378 stars)
Métricas de merge de PR
 (Mesclagem média 1d 2h) (1.000 fundiu PRs em 30d)

Description
Bugzilla Link 35304
Version trunk
OS Linux
Reporter LLVM Bugzilla Contributor
CC @JoeLoser

Extended Description

trunk/llvm/tools/clang/include/clang/Tooling/Refactoring/Rename/RenamingAction.h:79]: (performance) Function parameter 'NewQualifiedName' should be passed by reference.

Source code is

QualifiedRenameRule(const NamedDecl *ND, std::string NewQualifiedName)

Maybe better code

QualifiedRenameRule(const NamedDecl *ND, const std::string & NewQualifiedName)

Guia do colaborador

Direção de pesquisa
Localize o arquivo include/clang/Tooling/Refactoring/Rename/RenamingAction.h, encontre o construtor de QualifiedRenameRule e altere o segundo parâmetro de std::string para const std::string& para evitar uma cópia potencialmente cara.
Pilha de tecnologia
cpp
Domain
backend
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.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85