apol added a comment.
Other than that, makes sense. INLINE COMMENTS > kservice.h:576 > + */ > + static bool isSubseq(const QString& pattern, const QString& text, > Qt::CaseSensitivity cs = Qt::CaseSensitive); > + Can we give it a better name? How about something like `isContained`? or at least `isSubsequence`. 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