dfaure requested changes to this revision.
dfaure added a comment.
This revision now requires changes to proceed.


  Wait, who's calling KIO::iconNameForUrl() with an empty URL? This should be 
an assert or an early-return at the top of that method, not nested into one 
specific if(). Even the very beginning, db.mimeTypeForUrl(url), makes no sense 
for an empty URL.

REPOSITORY
  R241 KIO

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

To: jtamate, #frameworks, dfaure
Cc: broulik, michaelh

Reply via email to