D5870: Match subsequences in application runner

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon May 15 23:16:10 UTC 2017


apol added a comment.


  +1

INLINE COMMENTS

> servicerunner.cpp:101
> +
> +        for(int i=1; i<strList.size(); i++)
> +        {

use a foreach

> servicerunner.cpp:203
> +        //Splitting the query term to match using subsequences
> +        QStringList queryList = term.split(" ");
> +

`splitRef(QLatin1Char(' '))`?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D5870

To: rthomas, #plasma
Cc: apol, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170515/f91f9b0e/attachment.html>


More information about the Plasma-devel mailing list