Hey there!

When using PlaylistSaver::loadM3U (passing M3U as the options argument to 
PlaylistSaver::load), I get a ton of error messages in the console, in the 
form:
[...]
Invalid entry (missing '=') at /home/perdig/mp3/all.m3u:2342
Invalid entry (missing '=') at /home/perdig/mp3/all.m3u:2343
Invalid entry (missing '=') at /home/perdig/mp3/all.m3u:2344
Invalid entry (missing '=') at /home/perdig/mp3/all.m3u:2345
Invalid entry (missing '=') at /home/perdig/mp3/all.m3u:2346
Invalid entry (missing '=') at /home/perdig/mp3/all.m3u:2347
[...]
It goes from the first to the last line of my playlist.

It seems to be due to the following code:
bool PlaylistSaver::loadM3U(const KURL &file, int)
{
  QString local;
  if(KIO::NetAccess::download(file, local))
  {
    // if it's a PLS, transfer control, again (KIO bug?)
    {
      KSimpleConfig list(local, true);
      list.setGroup("playlist");

      // some stupid Windows lusers like to be case insensitive
      QStringList groups=list.groupList().grep(QRegExp("^playlist$", false));
      if (groups.count())
      {
        KURL l;
        l.setPath(local);
        return loadPLS(l);
      }
    }
[.....]

When calling KSimpleConfig list(local, true), it tries to parse local as if it 
were a config file, which gives all these error messages if we are really a 
M3U playlist... from what I see, that part of the code is necessary to check 
if we are really a M3U file or a PLS. But is this check really necessary? And 
there isn't another way of doing it? For large playlists it kinda slows 
loading down.

Best regards,

-- 
Ismael Orenstein (UIN 1293061)
ismore@ieg.com.br
http://ksnes9x.sourceforge.net
http://ggz.sourceforge.net
http://linuxgames.com/xrally
_______________________________________________
kde-multimedia mailing list
kde-multimedia@mail.kde.org
http://mail.kde.org/mailman/listinfo/kde-multimedia