----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3097/#review4364 -----------------------------------------------------------
Ship it! some very minor issues, nothing majorly important though. this is ready to go in! :) if you have an svn account, pls commit, otherwise let us know and one of us can do so on your behalf. /branches/KDE/4.4/kdebase/workspace/plasma/generic/runners/webshortcuts/webshortcutrunner.cpp <http://reviewboard.kde.org/r/3097/#comment3853> this line is probably not necessary; though i'd probably write it as removeAll(QString()) if it were :) /branches/KDE/4.4/kdebase/workspace/plasma/generic/runners/webshortcuts/webshortcutrunner.cpp <http://reviewboard.kde.org/r/3097/#comment3854> given the continue above, the else is superfluous. harmless, but superfluous :) /branches/KDE/4.4/kdebase/workspace/plasma/generic/runners/webshortcuts/webshortcutrunner.cpp <http://reviewboard.kde.org/r/3097/#comment3856> don't actually need to initialize this with an assignment. /branches/KDE/4.4/kdebase/workspace/plasma/generic/runners/webshortcuts/webshortcutrunner.cpp <http://reviewboard.kde.org/r/3097/#comment3855> spacing: foreach ( - Aaron On 2010-03-04 01:20:41, Nikolaus Waxweiler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/3097/ > ----------------------------------------------------------- > > (Updated 2010-03-04 01:20:41) > > > Review request for Plasma. > > > Summary > ------- > > Solution: find out which ones are enabled and compare before presenting a > match/advertising a search engine. > > I added a QStringList m_enabledSearchEngines which is updated after every > relevant change to sycoca and changed 2 places in the runner to query it: > when building the runner-help-string to be displayed via krunner's [?]-button > and when looking if the first word of the user's string could possibly be a > shorthand for/keyword of a search engine. > > > Diffs > ----- > > > /branches/KDE/4.4/kdebase/workspace/plasma/generic/runners/webshortcuts/webshortcutrunner.h > 1098255 > > /branches/KDE/4.4/kdebase/workspace/plasma/generic/runners/webshortcuts/webshortcutrunner.cpp > 1098255 > > Diff: http://reviewboard.kde.org/r/3097/diff > > > Testing > ------- > > Used rev. 1098255 as the starting point. Compiled it and overwrote the 4.4.1 > .so I had installed. Because I'm lazy. I experienced some krunner-crashes, > maybe because of this :p Didn't check though. Because I'm lazy. > > I then typed the shorthand for some search engine I had disabled ("amg") to > check whether the runner would respond. Nope. I then enabled it and ran > kbuildsycoca4 manually (the web shortcut dialog in konq. is unreliable with > this.. I think there's a few bug reports for this), and the runner responded > appropriately. Other shortcuts worked fine. > > > Thanks, > > Nikolaus > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
