Pairs going to KDE Edu

David Faure faure at
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> 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,
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5

More information about the kde-core-devel mailing list