llvm/llvm-project

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

Open

#34,652 创建于 2017年11月14日

在 GitHub 查看
 (6 评论) (0 反应) (0 负责人)C++ (10,782 fork)batch import
bugzillaclang:toolingconfirmedgood first issue

仓库指标

Star
 (26,378 star)
PR 合并指标
 (平均合并 1天 2小时) (30 天内合并 1,000 个 PR)

描述
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)

贡献者指南

研究方向
定位文件 include/clang/Tooling/Refactoring/Rename/RenamingAction.h，找到 QualifiedRenameRule 的构造函数，并将第二个参数从 std::string 改为 const std::string&，以避免潜在的高成本复制。
技术栈
cpp
领域
backend
议题类型
性能
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
1
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1 小时以内
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
85