[PATCH] kdemultimedia/kioslave/audiocd

Scott Wheeler wheeler at kde.org
Mon Dec 15 10:50:24 GMT 2003


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Monday 15 December 2003 10:15, David Faure wrote:
> >  atom.m_str = url.fileName().replace('/', QFL1("%2F"));
> This is wrong, it'll miss encoding % itself - this should use 
> KIO::encodeFileName. 

There are so many things wrong with the current code that one could go on for 
a long time about it.  ;-)  The code is a real mess...

I started a rewrite, but had to move it to the 3.3 TODO, so it's currently 
just lurking in a branch...

- -Scott

- -- 
There is more stupidity than hydrogen in the universe, and it has a longer 
shelf life.
- --Frank Zappa
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD4DBQE/3ZHwQu0ByfY5QTkRApxvAJsGNU5ExQO6amh7losJ8L6y4dzCtgCUCTxX
7aeDaN1iFnlSII1bv0zvJw==
=o2lU
-----END PGP SIGNATURE-----




More information about the kde-core-devel mailing list