D11753: baloodb: Add clean command

Stefan BrĂ¼ns noreply at phabricator.kde.org
Sun Apr 15 21:32:06 UTC 2018


bruns added inline comments.

INLINE COMMENTS

> databasesanitizer.cpp:344
> +            } else {
> +                m_pimpl->m_transaction->removeDocument(info.id);
> +                out << i18n("Removing:");

do the calls to m_transaction using a wrapper, e.g. `DatabaseSanitizer::removeDocument(info.id)`

> databasesanitizer.cpp:354
> +    if (dryRun) {
> +        m_pimpl->m_transaction->abort();
> +    } else {

dito

> main.cpp:110
>      }
> +
>  };

remove empty line

REPOSITORY
  R293 Baloo

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

To: michaelh, #baloo, #frameworks, bruns
Cc: bruns, cfeck, smithjd, ashaposhnikov, michaelh, astippich, spoorun, ngraham, alexeymin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180415/42472c9b/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list