[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