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 | [email protected] | 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 _______________________________________________ Development mailing list [email protected] http://lists.qt-project.org/mailman/listinfo/development
