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