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