dfaure added a comment.
isContained() sounds like QString::contains(). I like isSubsequence(). But I also just realized, it's a bit odd to have this in KService. I think I should make it part of the KApplicationTrader namespace instead, since it's a utility method for querying. This means merging with D25698 <https://phabricator.kde.org/D25698> then. What do you think? REPOSITORY R309 KService REVISION DETAIL https://phabricator.kde.org/D26740 To: dfaure, michaeleden, aacid, davidedmundson Cc: apol, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns