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
Fix chatters recursion not working #4114
Conversation
Can be tested & verified with the `/test-chatters` command
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
auto formatError = [](HelixGetChattersError error, QString message) { | ||
using Error = HelixGetChattersError; | ||
auto formatChattersError = [](HelixGetChattersError error, | ||
QString message) { |
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 'message' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param]
QString message) { | |
const QString& message) { |
|
||
auto twitchChannel = dynamic_cast<TwitchChannel *>(channel.get()); | ||
this->registerCommand( | ||
"/chatters", [formatChattersError](const auto &words, auto channel) { |
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 'words' is unused [misc-unused-parameters]
"/chatters", [formatChattersError](const auto &words, auto channel) { | |
"/chatters", [formatChattersError](const auto & /*words*/, auto channel) { |
Pull request checklist:
CHANGELOG.md
was updated, if applicableDescription
Can be tested & verified with the
/test-chatters
command