D28745: Skip caching thumbnails on encrypted filesystems

Méven Car noreply at phabricator.kde.org
Sun Apr 26 14:47:29 BST 2020


meven added inline comments.

INLINE COMMENTS

> thumbnail.cpp:738
> +                if (!m_thumbnailDirDeviceId) {
> +                    struct stat baseStat;
> +                    if (!lstat(QFile::encodeName(m_thumbBasePath).data(), &baseStat)) {

Can't you move this out of the loop ? Since `m_thumbBasePath` does not change.

> thumbnail.h:87
> +    dev_t m_thumbnailDirDeviceId = 0;
> +    short allowDirCatche;
>  };

s/allowDirCatche/allowDirCached

REPOSITORY
  R320 KIO Extras

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

To: marcingu, ivan, broulik, #dolphin, ngraham, meven, bruns
Cc: bruns, meven, ngraham, kde-frameworks-devel, kfm-devel, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, LeGast00n, cblack, fbampaloukas, alexde, Codezela, feverfew, michaelh, spoorun, navarromorales, firef, andrebarros, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200426/f4b09b13/attachment.html>


More information about the Kde-frameworks-devel mailing list