Home » Bug #2671

[2004-11-02 09:52 UTC] cmoehrke at netviser dot de

Description:
------------
When using study() on an MP3 which is sampled with VBR the Tags bitrate and length are not calculated correctly.
bitrate contains the Bitrate of the first frame which seem to be used for the calcutaion of length.
Would it be possible to change this behaviour, so that length ist empty and bitrate contains vbr or so?

[2004-11-24 23:57 UTC] alexmerz at php dot net

Currently i'm working on MP3_Idv2, where the study feature is dropped, because it has nothing to do with the Id standard.

This also means i'm a little bit unwilling to spend time for a dead horse, except critical bugs in it.

It is better to have the MPEG related stuff in a separate package. phpMPlayer ;-)

[2005-08-20 10:29 UTC] xpindex at hompus dot nl

http://www.hompus.nl/pear/

I've added VBR support to the MP3_Id module.

Changes:
- Added functions to check for the VBRI or XING header inside the audio frame and read it.
- Added property to store encoding type (CBR/VBR).
- Added property to store the real MP3 filelength (without the id3tag overhead).
- Added properties to store number of samples, samples_per_frame and number of frames.
- Added property to store quality indicator (VBR only).
- Added property with the length stored in hh:mm:ss format.

There are to versions of the diff file:
1. MP3_Id.orig-layout.diff
Contains the changes without changing the structure of the original file.
2. MP3_Id.diff
Contains the same changes. But:
- Stripped all trailing spaces.
- Changed all tabs into spaces
- Changed encoding to ANSI
This according to the PEAR coding standards. Since the diff file was almost useless for
people to read I included the other file.

I've tested it in my application with several files (VBR and CBR).