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

List:       kde-core-devel
Subject:    Re: New Krazy check: Forbidden Qt classes
From:       "Thiago Macieira" <thiago () kde ! org>
Date:       2007-04-26 13:41:33
Message-ID: 2724.172.26.0.1.1177594893.squirrel () 172 ! 26 ! 0 ! 1
[Download RAW message or body]

Sebastian Trüg said:
> On Thursday 26 April 2007 01:19:06 David Faure wrote:
>> QUrl has dangerous/broken(imho) API, e.g. QUrl::toString() gives a
>> string
>> that cannot be parsed back as a url (like we do in many many places with
>> KUrl::url(), e.g. over DBus), because e.g. a '#' in the path will appear
>> as
>> a '#' in QUrl::toString(), and QUrl(thatstring) will see it as a
>> reference.

> Now this is really a problem. I did it the same way to pass QUrls over
> DBus as
> strings:
>
> QDBusArgument << QUrl.toString

Use QUrl::url() for that. It returns a QString that is parseable.

> QDBusArgument >> QString
> QUrl(QString)

If the string was created using QUrl::url(), this is correct.

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358


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