-
-
Notifications
You must be signed in to change notification settings - Fork 449
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Finish renaming Viewer list to Chatter list #4974
Finish renaming Viewer list to Chatter list #4974
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
@@ -1364,8 +1364,8 @@ | |||
|
|||
HotkeyController::HotkeyMap actions{ | |||
{"delete", | |||
[viewerDock](std::vector<QString>) -> QString { | |||
viewerDock->close(); | |||
[chatterDock](std::vector<QString>) -> QString { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: the parameter #1 is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param]
[chatterDock](std::vector<QString>) -> QString { | |
[chatterDock](const std::vector<QString>&) -> QString { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
item->setText(text); | ||
item->setFont(getApp()->fonts->getFont(FontStyle::ChatMedium, 1.0)); | ||
return item; | ||
}; | ||
|
||
auto addLabel = [this, formatListItemText, chattersList](QString label) { | ||
auto formattedLabel = formatListItemText(label); | ||
auto *formattedLabel = formatListItemText(label); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: parameter 'label' is passed by value and only copied once; consider moving it to avoid unnecessary copies [performance-unnecessary-value-param]
auto *formattedLabel = formatListItemText(label); | |
auto *formattedLabel = formatListItemText(std::move(label)); |
Description
Closes #4640.