----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/694/#review1114 -----------------------------------------------------------
one small tweak to the api and then i think this can go in; note that we are in feature freeze, however, so features can't go in right now. i consider this a bug fix, however, since i spent much of last week accidentally opening an image i'd taken with ksnapshot that was named the same as an application i use (guess what the snapshot was of? hehe..) so having this preferential ranking in makes krunner more reliable imo. /trunk/KDE/kdelibs/plasma/runnercontext.h <http://reviewboard.kde.org/r/694/#comment726> how about just save(const KConfigGroup &config) and restore(const KConfigGroup &config) that way we can use those methods for other data storage/retrieval if we wish, and if we change how launch counts are managed we aren't stuck with lame API. it would also be consistent with Applet, Wallpaper, etc. - Aaron On 2009-05-13 10:17:27, Ryan Bitanga wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/694/ > ----------------------------------------------------------- > > (Updated 2009-05-13 10:17:27) > > > Review request for Plasma. > > > Summary > ------- > > This enables adaptive search for KRunner by tracking user behavior in terms > of programs run, documents opened, etc. via KRunner. It boosts relevance of a > match whenever a user runs that match, allowing the match to appear before > other matches obtained of similar nature as long as the front-end correctly > sorts the matches. > > For example, running a query of "kon" on the services runner may result in > matches for Konqueror and Konsole. Both matches will have a relevance of 1.3, > but Konqueror will appear before Konsole. If a user launches Konsole via > KRunner at least one time more than Konqueror, Konsole will have its > relevance boosted and will appear before Konqueror in the list of matches. > > This partially addresses bug #180158. The other part (changes to quicksand) > will be committed soon. > > > Diffs > ----- > > /trunk/KDE/kdelibs/plasma/querymatch.cpp 967300 > /trunk/KDE/kdelibs/plasma/runnercontext.h 967300 > /trunk/KDE/kdelibs/plasma/runnercontext.cpp 967300 > /trunk/KDE/kdelibs/plasma/runnermanager.cpp 967300 > > Diff: http://reviewboard.kde.org/r/694/diff > > > Testing > ------- > > > Thanks, > > Ryan > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel