broulik added inline comments.
INLINE COMMENTS
> config.cpp:371
> dbg << "KScreen::Config(";
> - for (const auto output : config->outputs()) {
> + for (const auto &output : config->outputs()) {
> if (output->isConnected()) {
This still calls `config->outputs().begin()` and might detach, wouldn't it?
`qAsConst` also doesn't work for return value of a method call, so looks like
you're gonna have to store it in a `const` variable first
REPOSITORY
R110 KScreen Library
REVISION DETAIL
https://phabricator.kde.org/D14350
To: gladhorn, #plasma
Cc: broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed,
jensreuterberg, abetts, sebas, apol, mart
