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

(Updated Sept. 10, 2015, 8:51 p.m.)


Review request for KDE Frameworks and Albert Astals Cid.


Changes
-------

remove the static bool


Repository: kservice


Description (updated)
-------

77a998e37 removed the call to KSycoca::disableAutoRebuild(),
we can just check that we are in kbuildsycoca using qAppName
like two other places in ksycoca.cpp already do.
And this way get rid of more unnecessary "public" API for ksycoca.

REVIEW: 125145


Diffs (updated)
-----

  src/sycoca/ksycoca.h 5e762fddffa03c4e25218ed608d483dcd4b35bf4 
  src/sycoca/ksycoca.cpp 2048c84660bc36e9a56b747eaee27757089363b4 

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


Testing
-------

compiles.

we don't have a unittest for a corrupt ksycoca file ;)


Thanks,

David Faure

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to