dfaure added inline comments. INLINE COMMENTS
> apol wrote in kservice.cpp:658 > This adds a "KDE" allocation on every which isn't ideal either. I don't know, > maybe it doesn't matter either way... Well, it's a QStringLiteral, AFAIK the QString just points to pre-allocated UTF-16 data. Not sure if it's zero-allocation but it's pretty cheap. REPOSITORY R309 KService BRANCH xdg_current_deskto REVISION DETAIL https://phabricator.kde.org/D26900 To: dfaure, apol, aacid Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
