llvm/llvm-project

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

Open

#34,652 opened on 2017年11月14日

GitHub で見る
 (6 comments) (0 reactions) (0 assignees)C++ (26,378 stars) (10,782 forks)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
Issue 種別
performance
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
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.