Bug 141681

Summary: CBR encoded mp3 files with "Info" Xing header have their duration reported incorrected by taglib
Product: [Frameworks and Libraries] taglib Reporter: Xavier Duret <xaviour.maillists>
Component: generalAssignee: Scott Wheeler <wheeler>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Add support for "Info" headers

Description Xavier Duret 2007-02-14 14:17:47 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources

"Info" headers are just like "Xing" headers but for CBR files. Failure to use it result in improper calculation of the song length and the bitrate.
Comment 1 Xavier Duret 2007-02-14 14:18:41 UTC
Created attachment 19687 [details]
Add support for "Info" headers

This fix is trivial.
Comment 2 Scott Wheeler 2007-04-02 00:23:08 UTC
SVN commit 649037 by wheeler:

Check for CBR Info headers also.

BUG:141681


 M  +1 -1      xingheader.cpp  


--- trunk/kdesupport/taglib/taglib/mpeg/xingheader.cpp #649036:649037
@@ -88,7 +88,7 @@
 {
   // Check to see if a valid Xing header is available.
 
-  if(!data.startsWith("Xing"))
+  if(!data.startsWith("Xing") && !data.startsWith("Info"))
     return;
 
   // If the XingHeader doesn't contain the number of frames and the total stream