cfeck added a comment.
Regarding the comments, I prefer a space after '//' for better readability.
INLINE COMMENTS
> powerdevilapp.cpp:169
> + });
> + for(const QString &actionId: actionIds)
> + {
Missing space after 'for'.
> powerdevilapp.cpp:170
> + for(const QString &actionId: actionIds)
> + {
> + QAction oldAction;
Please move brace to previous line.
REPOSITORY
R122 Powerdevil
REVISION DETAIL
https://phabricator.kde.org/D10668
To: davidedmundson, #plasma
Cc: cfeck, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg,
abetts, sebas, apol, mart
