D4937: Add KFileWidget::setSelectedUrl()

Fabian Vogt noreply at phabricator.kde.org
Tue Mar 7 11:51:43 UTC 2017


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


  Yes, but constructing a relative QUrl isn't really that easy to do (definitely not obvious at that) and so future users will either pass a filename with QUrl(filename) (broken) or store the absolute path themselves, which is unnecessary memory use and more complicated.
  
  I just noticed that with KIOFILEWIDGETS_NO_DEPRECATED defined the header file will not include the declaration but the cpp file still has the definition of the method, so it won't compile...

REPOSITORY
  R241 KIO

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

To: dfaure, fvogt
Cc: fvogt, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170307/eb355f7f/attachment.html>


More information about the Kde-frameworks-devel mailing list