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