Review Request 125417: Don't sync the config n times when saving

David Edmundson david at davidedmundson.co.uk
Sat Sep 26 23:29:18 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125417/
-----------------------------------------------------------

Review request for KDE Frameworks.


Repository: knotifyconfig


Description
-------

Because knotifyconfigelement has a cache, we end up calling writeEntry; sync() ; writeEntry() ; sync() ..which sounds expensive 

This change makes that we only sync after all the writeEntry.


Diffs
-----

  src/knotifyconfigelement.cpp 93fade5b2b02ea9acfb8f11dee5879aca6c0d807 
  src/knotifyeventlist.cpp dd6b6e9d5557fde0b46d980e4078921dd978f13f 

Diff: https://git.reviewboard.kde.org/r/125417/diff/


Testing
-------

Still works.
Nothing else called NotifyConfigElement::save and it's private API.


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150926/23797a57/attachment.html>


More information about the Kde-frameworks-devel mailing list