Review Request 111795: Remove three character restriction on the services runner

[Sebastian Kügler]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111795/#review36804
-----------------------------------------------------------


Not so sure about this. The 3 char restriction is there for performance, it might still work as expected on many system, yet cause stalling on others. At least doing the query on the first character seems *way* overdone.

- Sebastian Kügler


On July 30, 2013, 12:46 p.m., Milo Mordaunt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111795/
> -----------------------------------------------------------
> 
> (Updated July 30, 2013, 12:46 p.m.)
> 
> 
> Review request for kde-workspace.
> 
> 
> Description
> -------
> 
> The services runner now queries with terms of less than three characters.
> 
> Previously with a term of less than three characters this runner would be idle. Now it gives more immediate results, ie gp now returns gparted., and since it is only the services runner they are likely to be relevant.
> 
> 
> Diffs
> -----
> 
>   plasma/generic/runners/services/servicerunner.cpp 7b292efe40bbddb23cfc317648db0e645c579879 
> 
> Diff: http://git.reviewboard.kde.org/r/111795/diff/
> 
> 
> Testing
> -------
> 
> I have tested a variety of terms in my krunner, and the services runner appears to be working as expected.
> 
> 
> Thanks,
> 
> Milo Mordaunt
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130730/26671a2e/attachment.htm>