After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 127887 - [patch] Variable bitrate mp3s displays wrong duration.
[patch] Variable bitrate mp3s displays wrong duration.
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: User Interface
unspecified
Other Linux
: Normal minor
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
: 127904 128157 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2003-11-25 12:48 UTC by Bo Bendtsen
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Variable bitrate mp3 displays too _small_ time (562.25 KB, audio/mpeg)
2003-11-25 13:41 UTC, Bo Bendtsen
  Details
Fixed bitrate displays the correct running time. (398.67 KB, audio/mpeg)
2003-11-25 13:42 UTC, Bo Bendtsen
  Details
Screenshot of the error. (23.29 KB, image/png)
2003-11-25 13:45 UTC, Bo Bendtsen
  Details
fix Xing header parsing with mpeg 2 streams (664 bytes, patch)
2003-11-26 22:54 UTC, Bastien Nocera
none Details | Review

Description Bo Bendtsen 2003-11-25 12:48:21 UTC
Description of Problem:
Variable bitrate mp3s reports duration much longer than they are. A 4
minutes file is reported as a 4150 minutes file.

Steps to reproduce the problem:
1. Make variable bitrate mp3: ("$ lame -v test.wav")
2. Add "test.wav.mp3" to the colection.

Actual Results:
Wrong (much longer) duration displayed. Plays ok.

Expected Results:
Correct duration displayed.

How often does this happen?
Always

Additional Information:
Using: Rhythmbox 0.6.1
       Gstreamer 0.6.3
Comment 1 Bastien Nocera 2003-11-25 12:50:23 UTC
Please make a test file available. This is supposed to be fixed already.
Comment 2 Bo Bendtsen 2003-11-25 13:41:07 UTC
Created attachment 21793 [details]
Variable bitrate mp3 displays too _small_ time
Comment 3 Bo Bendtsen 2003-11-25 13:42:19 UTC
Created attachment 21794 [details]
Fixed bitrate displays the correct running time.
Comment 4 Bo Bendtsen 2003-11-25 13:42:57 UTC
The error does not seems occur in small files. And sometimes the
displayed time is too _small_. Like in these test files.
Comment 5 Bo Bendtsen 2003-11-25 13:45:17 UTC
Created attachment 21795 [details]
Screenshot of the error.
Comment 6 Christophe Fergeau 2003-11-26 10:21:27 UTC
*** Bug 127904 has been marked as a duplicate of this bug. ***
Comment 7 Christophe Fergeau 2003-11-26 10:36:13 UTC
I can confirm the problem with latest cvs and the 2 attached files (I
deleted ~/.gnome2/rhythmbox before the test). xmms detects the correct
length.
Comment 8 Bastien Nocera 2003-11-26 22:54:09 UTC
Created attachment 21848 [details] [review]
fix Xing header parsing with mpeg 2 streams
Comment 9 Bastien Nocera 2003-11-26 22:55:32 UTC
Merge fix from xine-lib. We can all thank Thibaut Mattern :)
Comment 10 Colin Walters 2003-11-27 04:40:34 UTC
* commited
walters@rhythmbox.org--2003b/rhythmbox--mainline--0.6--patch-345

Thanks!
Comment 11 Bastien Nocera 2003-11-28 12:36:29 UTC
Reopening, wasn't merged on HEAD (0.7).
Comment 12 Colin Walters 2003-11-28 16:24:00 UTC
Sorry, I'd forgotten to run tla-cvs-sync for the 0.7 branch; done now!
Comment 13 Bastien Nocera 2003-11-29 14:37:11 UTC
*** Bug 128157 has been marked as a duplicate of this bug. ***
Comment 14 Ian King 2003-12-20 03:29:09 UTC
This one's still alive in 0.6.2.
Comment 15 Peter Snyder 2003-12-23 20:55:30 UTC
I am still seeing this on .6.3, using both gstreamer and xine.  For
me, the Mp3s that have problems were encoded with lame with the
following arguments:
-h -V 2 --add-id3v2 --tl %d --tt %n --ta %a --ty %y --tn %t --tg %g %w %m
Comment 16 Sebastien Bacher 2004-01-17 10:14:31 UTC
Bug still here in 0.6.4 according to the debian bug submitter (bug
#224456). I've added it to the Cc of bugzilla bug report because it's
more an upstream problem than a debian one.

"Even with gstreamer 0.6.4 and rhythmbox 0.6.4, my entire library of
12,000+ MP3s, all of which were encoded with 'lame --alt-preset' (i.e.
they're all VBR-encoded), report playing times in the thousands of
minutes. I rebuild my library with each new release of Rhythmbox
hoping this problem has been fixed, but as of yet it hasn't. Clearly,
this bug is still extant.

More information available upon request."
Comment 17 Christophe Fergeau 2004-01-21 13:49:54 UTC
Ok, so this bug report is quite confusing, it's mixing mp3s whose
length is too short with mp3s whose length is 1000 times too long.
These are different issues imo.
Worse, the latest comments say "this is not fixed" without saying if
they are talking about the length being reported as too short or far
too long.
I believe #1 was actually fixed by hadess's patch, and #2 is a dupe of
130819.
I'm closing this bug now, things should work properly in next
rhythmbox release. If you reopen please:
* describe exactly how your mp3 length is miscalculated
* tell which backend you are using (gstreamer/xine)
* tell which version of rhythmbox and gstreamer you are using