[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-core-devel
Subject: Re: KURL problem
From: Michael Brade <brade () kde ! org>
Date: 2005-09-27 10:39:16
Message-ID: 200509271239.19290.brade () kde ! org
[Download RAW message or body]
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
[Attachment #3 (application/pgp-signature)]
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic