D28745: Skip caching thumbnails on encrypted filesystems

Stefan BrĂ¼ns noreply at phabricator.kde.org
Thu Apr 16 13:57:28 BST 2020


bruns added inline comments.

INLINE COMMENTS

> thumbnail.cpp:726
>  
> +            const auto mountsList = KMountPoint::currentMountPoints();
> +            const auto mount =  mountsList.findByPath(filePath);

Calling currentMountPoints() for every created file is definitely a bad idea ...

https://github.com/KDE/kio/blob/master/src/core/kmountpoint.cpp#L272

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, fprice, LeGast00n, cblack, fbampaloukas, alexde, Codezela, feverfew, michaelh, spoorun, navarromorales, firef, andrebarros, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200416/84431c9a/attachment.htm>


More information about the kfm-devel mailing list