Pairs going to KDE Edu
faure at kde.org
Sun Apr 29 20:51:04 BST 2012
On Thursday 19 April 2012 23:32:51 Aleix Pol wrote:
> On Thu, Apr 19, 2012 at 4:45 PM, Andrius da Costa Ribas
> <andriusmao at gmail.com> wrote:
> > Hello,
> > Please find attached suggested patches to fix/improve Windows build.
The URL handling code was broken indeed, but the suggested fix is still a bit
buggy. Don't use QUrl::toString in Qt4, it will fail when a filename contains a
Use KUrl::url() instead.
This issue is fixed in Qt5.
QString PlayersModel::iconsDir(const QString& path)
- return "file://"+KGlobal::dirs()->findResource("appdata", path);
+ return QUrl::fromLocalFile(KGlobal::dirs()->findResource("appdata",
This method should really return a KUrl, rather than an ambiguous QString (is
it a path? is it a URL?) (obviously the implementation returns a URL, but who
knows what the callers do...)
David Faure, faure at kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5
More information about the kde-core-devel