llvm/llvm-project

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

Open

#34,652 建立於 2017年11月14日

在 GitHub 查看
 (6 留言) (0 反應) (0 負責人)C++ (26,378 star) (10,782 fork)batch import
bugzillaclang:toolingconfirmedgood first issue

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

貢獻者指南

技術棧
cpp
領域
toolingperformance
議題類型
performance
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
under 1 hour
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
stale
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
basic C++ knowledge
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
90
研究方向
Locate the file include/clang/Tooling/Refactoring/Rename/RenamingAction.h and inspect line 79. The constructor of QualifiedRenameRule takes a std::string NewQualifiedName by value. Change the parameter to const std::string& NewQualifiedName to avoid an unnecessary copy. Ensure that the rest of the code does not require a mutable copy; if so, adjust accordingly. This is a localized, one line change that improves performance.