D8347: KIO::PreviewJob::defaultPlugins() function

Nathaniel Graham noreply at phabricator.kde.org
Wed Oct 18 16:40:40 BST 2017


ngraham marked an inline comment as done.
ngraham added inline comments.

INLINE COMMENTS

> anthonyfieroni wrote in previewjob.cpp:740
>   const QStringList blackList = { QStringLiteral("textthumbnail") };

I wrote it the way I did on purpose, to facilitate people adding new entries if and when the blacklist needs to be updated without having the diff show the whole line as having been changed.

REPOSITORY
  R241 KIO

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

To: ngraham, #frameworks, broulik, #dolphin, markg
Cc: markg, anthonyfieroni, elvisangelaccio, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171018/77802ddd/attachment.htm>


More information about the kfm-devel mailing list