From kde-commits  Wed Aug 31 23:36:27 2005
From: Alexandre Pereira de Oliveira <aoliveira () kdemail ! net>
Date: Wed, 31 Aug 2005 23:36:27 +0000
To: kde-commits
Subject: extragear/multimedia/amarok/src
Message-Id: <1125531387.843874.20206.nullmailer () svn ! kde ! org>
X-MARC-Message: https://marc.info/?l=kde-commits&m=112553139528545

SVN commit 455607 by aoliveira:

Revert bad commit.
TagLib::FileRef deletes the pointer on its dtor.
CCMAIL: possebaer@gmx.at


 M  +1 -4      metabundle.cpp  


--- trunk/extragear/multimedia/amarok/src/metabundle.cpp #455606:455607
@@ -161,11 +161,10 @@
     const QString path = m_url.path();
     TagLib::FileRef fileref;
     TagLib::Tag *tag = 0;
-    TagLib::MPEG::File *mpeg = 0;
 
     if( AmarokConfig::recodeID3v1Tags() && path.endsWith( ".mp3", false ) )
     {
-        mpeg = new TagLib::MPEG::File( QFile::encodeName( path ), true, readStyle );
+        TagLib::MPEG::File *mpeg = new TagLib::MPEG::File( QFile::encodeName( path ), true, readStyle );
         fileref = TagLib::FileRef( mpeg );
 
         if( mpeg->isValid() )
@@ -199,8 +198,6 @@
         init( fileref.audioProperties() );
     }
 
-    delete mpeg;
-
     //FIXME disabled for beta4 as it's simpler to not got 100 bug reports
     //else if( KMimeType::findByUrl( m_url )->is( "audio" ) )
     //    init( KFileMetaInfo( m_url, QString::null, KFileMetaInfo::Everything ) );