D10045: Don't assert on empty names

Milian Wolff noreply at phabricator.kde.org
Tue Jan 30 09:28:37 UTC 2018


mwolff added inline comments.

INLINE COMMENTS

> remoteimpl.cpp:56
>  
> -        const QStringList filenames = dir.entryList(QDir::Files | QDir::Readable);
> +        const QStringList filenames = dir.entryList({QStringLiteral("*.desktop")},
> +                                                    QDir::Files | QDir::Readable);

is this OK? or are there other entries that should be shown under remote:/ ?

REPOSITORY
  R241 KIO

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

To: mwolff, dfaure
Cc: dhaumann, #frameworks, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180130/0f7a55a5/attachment.html>


More information about the Kde-frameworks-devel mailing list