----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125121/ -----------------------------------------------------------
Review request for KDE Frameworks and Albert Astals Cid. Repository: kservice Description ------- This leads to more self-contained API: recreate checks if it can indeed reuse the existing sycoca or if it should skip it (when language or $XDG_DATA_{HOME,DIRS} has changed) And remove the call to clearCaches which closes KSycoca::self, doesn't matter now that KBuildSycoca is a different instance anyway. Diffs ----- src/kbuildsycoca/kbuildsycoca.h 65e4efb6d30ec57be1c5fd0d2a2b4dfcb7e760a7 src/kbuildsycoca/kbuildsycoca.cpp 3ce7ae425cfdfeac3264c5f4ecff440402402f7d Diff: https://git.reviewboard.kde.org/r/125121/diff/ Testing ------- Wrote (and pushed) a new unittest (KSycocaTest::testOtherAppDir) for the case where kbuildsycoca detects a mismatch in the global header. Works before and after this patch. Thanks, David Faure
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel