[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: KDE/kdelibs/phonon
From:       David Faure <faure () kde ! org>
Date:       2008-03-11 13:19:53
Message-ID: 200803111419.54312.faure () kde ! org
[Download RAW message or body]

On Tuesday 11 March 2008, Thierry Bastian wrote:
> +    } else if (QUrl(filename).isValid()) {
> d->url = filename;

This does the QUrl parsing twice though; if performance matters, checking isValid on \
d->url would be faster.

-- 
David Faure, faure@kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).


[prev in list] [next in list] [prev in thread] [next in thread]