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

Reply via email to