| davidedmundson |
| Plasma |
Match applications in krunner using subsequences instead of substrings. This allows matching applications using parts of its name, instead of having to type its full name.
This is also an attempt to implement bug #262837 (Applications runner - match subsequence instead of substring)
Examples :
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| runners/services/servicerunner.cpp | ||
|---|---|---|
| 101 | I'm skipping the first string in strList because that string has already been accounted for relavace right before the call to increaseMatchRelavance. Relevance is incremented by 0.01 only based on the second string onwards. I don't think you can skip iterations based on indices in a foreach loop. | |
| 203 | Could you explain why would we want to use splitRef instead of split? I'm not sure I understand the need for it. | |
| runners/services/servicerunner.cpp | ||
|---|---|---|
| 203 | split copies the strings. Result is the same but faster. | |
No, I don't have commit access yet. Can I apply for a developer account now? I haven't done so until now.
This has no repo assigned, so i was going to commit it but could not.
You can apply for a developer account if you think you need it at https://community.kde.org/Infrastructure/Get_a_Developer_Account
@aacid : I have assigned the Plasma-Workspace repo to this revision. Please commit this patch on my behalf. I think I'll hold on for a while before applying for commit access.