[Patch] KUrl::prettyUrl and xmpp: url

Olivier Goffart ogoffart at kde.org
Mon Oct 8 13:39:09 BST 2007


Le lundi 8 octobre 2007, Thiago Macieira a écrit :
> Olivier Goffart wrote:
> >So do you suggest me to fix pathOrUrl()  (and his documentation)
> >Or do I commit the previous patch ?
>
> Yes, fix pathOrUrl() if it's being used for machine-to-machine.

Well according to pathOrUrl() is mostly used for display to the user, as the 
documentation say it.

I don't think KRun should use pathOrUrl.


but either url() as in my patch, 
or
(url.isLocalFile() && url.fragment().isNull() && url.encodedQuery().isNull())? 
    url.toLocalFile()  : url.url();
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071008/1f80e235/attachment.sig>


More information about the kde-core-devel mailing list