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

Reply via email to