D10155: Use the much faster urls() method from QMimeData

Milian Wolff noreply at phabricator.kde.org
Tue Jan 30 15:35:19 UTC 2018


mwolff added inline comments.

INLINE COMMENTS

> kurlmimedata.cpp:74
>      }
>      QByteArray ba = mimeData->data(QString::fromLatin1(firstMimeType));
>      if (ba.isEmpty()) {

if `firstMimeType == "text/uri-list"` (see above, `PreferLocalUrls`), it's still going to be slow, no? Should this be handled generally here by introducing a helper that checks the mime type and then uses either `urls()` directly or call data as before?

also, what happens if `PreferLocalUrls` is set, but no `text/uri-list` data is available, could the old way of getting the data for the `s_kdeUriListMime` have worked more reliably?

REPOSITORY
  R244 KCoreAddons

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

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


More information about the Kde-frameworks-devel mailing list