Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

Xuetian Weng wengxt at gmail.com
Tue Oct 6 09:53:18 UTC 2015


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

(Updated Oct. 6, 2015, 9:53 a.m.)


Review request for KDE Frameworks, Albert Astals Cid and David Faure.


Changes
-------

Added a template helper function to merge similar logics in TimestampFetcher and TimestampChecker.


Repository: kservice


Description
-------

In 0e4d7247e27f82a9b79b19dbceb35b603baceb76, a new feature saving directory mtime in ksycoca db is introduced.

In the related code, TimestampChecker() will check the mtime recursively, but the building process will only read the top level directories' mtime, which causes ksycoca db being recreated again and again.

I hit this because my ~/.config/menus/applications-merged mtime newer than ~/.config/menus. And plasmashell is running with 100% since it's creating ksycoca db repeatedly.

This patch change the building db procedure to use the same logic as in TimestampChecker(), so building and checking are consistent now.


Diffs (updated)
-----

  autotests/ksycocatest.cpp 67f0700 
  src/sycoca/ksycocautils_p.h c14057b 
  src/sycoca/ksycoca.cpp 5209389 
  src/sycoca/kbuildsycoca.cpp 00aaac3 

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


Testing
-------

Now a sub directory with larger timestamp than parent will not cause ksycoca data being recreated again and again.


Thanks,

Xuetian Weng

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151006/67c53f24/attachment.html>


More information about the Kde-frameworks-devel mailing list