[PATCH] kdemultimedia/kioslave/audiocd

Allan Sandfeld Jensen kde at carewolf.com
Mon Dec 15 23:11:10 GMT 2003


On Monday 15 December 2003 22:07, David Faure wrote:
> On Monday 15 December 2003 20:05, André Wöbbeking wrote:
> > On Monday 15 December 2003 10:15, David Faure wrote:
> > > On Sunday 14 December 2003 14:44, André Wöbbeking wrote:
> > > >  atom.m_str = url.fileName().replace('/', QFL1("%2F"));
> > >
> > > This is wrong, it'll miss encoding % itself - this should use
> > > KIO::encodeFileName.
> >
> > I only replaced filename() with fileName(). Why must % be encoded, for
> > KUrl magic?
>
> No, to differenciate %2F (for slash) with %2F (as an actual part of the
> filename). KIO::encodeFileName turns % into %%.
>
Why do we need to differenciate? Can't we assume both are meant to be 
displayed as a slash?

`Allan




More information about the kde-core-devel mailing list