[Development] New QUrl implementation

Thiago Macieira thiago.macieira at intel.com
Tue Dec 27 15:24:23 CET 2011


On Tuesday, 27 de December de 2011 13.42.10, David Faure wrote:
> 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.

I'm not allowed to submit to Gerrit. I can only publish as open source.

> 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.

No, I haven't. I've only run the QUrl-related tests in qtbase, so not even the 
QtDeclarative tests.

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center
     Intel Sweden AB - Registration Number: 556189-6027
     Knarrarnäsgatan 15, 164 40 Kista, Stockholm, Sweden
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.qt-project.org/pipermail/development/attachments/20111227/464bb443/attachment.sig>


More information about the Development mailing list