broulik added inline comments.
INLINE COMMENTS
> graesslin wrote in package.cpp:594
> Just curious: what does the modulo operator on a string?
It explicitly uses `QStringBuilder` for concatenation which `operator+` only
does if you define `QT_USE_QSTRINGBUILDER`
REPOSITORY
R290 KPackage
graesslin added a comment.
There are a few unrelated changes, but otherwise looks good.
INLINE COMMENTS
> package.cpp:360
> //Nested loop, but in the medium case resolves to just one iteration
> -//qDebug() << "prefixes:" << prefixes.count() << prefixes;
> +// qDebug() <<
This revision was automatically updated to reflect the committed changes.
Closed by commit R290:8336ad60d9a5: Properly do strings in the kpackage
framework (authored by apol).
REPOSITORY
R290 KPackage
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D9104?vs=23262=23266
REVISION
davidedmundson accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R290 KPackage
BRANCH
strings
REVISION DETAIL
https://phabricator.kde.org/D9104
To: apol, #frameworks, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed,
apol created this revision.
apol added reviewers: Frameworks, Plasma.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Instead of using implicit casts, use the ones we need as suggested for KF5.
TEST PLAN
