KURL problem

Michael Brade brade at kde.org
Tue Sep 27 11:39:16 BST 2005


On Tuesday 27 September 2005 12:28, Andras Mantia wrote:
> On Tuesday 20 September 2005 14:02, Michael Brade wrote:
> > What do you suggest? Can we please fix KURL? If adjustPath() removes
> > *one* slash, then cleanPath() should get rid of all of them but one.
> > Or neither of them should change the url.
>
> Isn't the attached patch a solution (adjustPath(-1) removes all trailing
> slashes)?. Hopefully it doesn't break existing applications (it
> shouldn't).
Yes, that is why I asked. I have no idea about the consequences of such 
changes. Waldo, David,...? 

And what happens with your patch in case of ftp://ya\@kde.org//? You don't 
want to get ftp://ya\@kde.org.

> And it fixes the crash. 
That's for sure :-)

-- 
Michael Brade;                 KDE Developer, Student of Computer Science
  |-mail: echo brade !#|tr -d "c oh"|s\e\d 's/e/\@/2;s/$/.org/;s/bra/k/2'
  °--web: http://www.kde.org/people/michaelb.html

KDE 3: The Next Generation in Desktop Experience
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20050927/5a96ad48/attachment.sig>


More information about the kde-core-devel mailing list