[Development] New QUrl implementation
David Faure
david.faure at kdab.com
Tue Dec 27 13:42:10 CET 2011
On Friday 23 December 2011 17:22:38 Thiago Macieira wrote:
> QUrl: Always lowercase the scheme
> Change-Id: I8d467014d22384f1be15fdd746e20b1153a82a4e
>
> Do we want this? I think so.
I would say yes, too. This simplifies the checks in application code.
PS: great work on QUrl!
Isn't this going to go to gerrit for review before merging? I'm surprised that
this is in gitorious branches for now, but I suppose that's because topic
branches don't really work there iirc.
What I would like to do with your new QUrl (after this week's vacations) is to
import it into kdelibs-frameworks, to port kurltest to it and see what breaks.
Or did you do that already locally? You have my permission to import any tests
from kurltests that I wrote, into the qurl auto tests, of course.
--
David Faure | david.faure at kdab.com | KDE/Qt Senior Software Engineer
KDAB (France) S.A.S., a KDAB Group company
Tel. France +33 (0)4 90 84 08 53, Sweden (HQ) +46-563-540090
KDAB - Qt Experts - Platform-independent software solutions
More information about the Development
mailing list