D11529: Balooctl: Deindex unfound files with check command.

Kai Uwe Broulik noreply at phabricator.kde.org
Wed Mar 28 08:47:33 UTC 2018


broulik requested changes to this revision.
broulik added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> fileindexerconfig.cpp:63
>      , m_indexHidden(false)
> -    , m_devices(nullptr)
> +    , m_devices(new StorageDevices(this))
>      , m_maxUncomittedFiles(40)

Please create the `StorageDevices` object on demand, otherwise this penalizes all users of that class that just want to know whether Baloo itself is enabled, which was the main reason for doing D11279 <https://phabricator.kde.org/D11279>

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D11529

To: smithjd, #baloo, vhanda, michaelh, broulik
Cc: mgallien, #frameworks, ashaposhnikov, michaelh, astippich, spoorun, nicolasfella, ngraham, alexeymin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180328/eb825790/attachment.html>


More information about the Kde-frameworks-devel mailing list