D20383: [KRunner] Avoid writing history if addToHistory is idempotent
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Mon Apr 8 16:19:37 BST 2019
bruns created this revision.
bruns added reviewers: Plasma, broulik, apol.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
bruns requested review of this revision.
REVISION SUMMARY
It is not uncommon to run the same command repeatedly. In this case,
the history is actually unchanged - the item is removed from the first
position, and prepended again.
TEST PLAN
run the same command twice, config file is not rewritten
run a new command, config is updated
REPOSITORY
R120 Plasma Workspace
BRANCH
krunner_history
REVISION DETAIL
https://phabricator.kde.org/D20383
AFFECTED FILES
krunner/view.cpp
To: bruns, #plasma, broulik, apol
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190408/9874c5d1/attachment.html>
More information about the Plasma-devel
mailing list