D11745: databasesanitizer: Use flags for filtering

Stefan BrĂ¼ns noreply at phabricator.kde.org
Sun Apr 15 02:42:09 UTC 2018


bruns accepted this revision.
bruns added a comment.
This revision is now accepted and ready to land.


  Save the missing comment for "tmpfs", looks fine, thanks!

INLINE COMMENTS

> databasesanitizer.cpp:250
>          auto mounted = info.isValid();
> -
> -        if (missingOnly && mounted) {
> +        if (info.fileSystemType() == QLatin1String("tmpfs")) {
> +            continue;

This line definitely needs a comment why this is here

REPOSITORY
  R293 Baloo

BRANCH
  sanitize-enums (branched from master)

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

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


More information about the Kde-frameworks-devel mailing list