[Bug 620323] mpegaudioparse: Add support for LAME tags and adjust segments based on the padding information from it

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=620323

--- Comment #29 from Christoph Reiter (lazka) <reiter.christoph at gmail.com> ---
(In reply to Carlos Rafael Giani from comment #28)
> Since mpegaudioparse knows the total number of frames (it is contained in
> the Xing header), it could keep track of the current frame (= the frame that
> contains the current playback position).

How would that work with seeking?

> Also, out of the number of frames,
> mpegaudioparse computes the playback length. By using the LAME tag
> information and the total number of frames, it becomes possible to figure
> out the number of the last frame. This takes care of cases where the padding
> length in the LAME tag exceeds one frame length. Also, this way, it becomes
> possible to know which one is the last frame without any buffering.

The problem I see is that we need to know when we are in the second to last
frame (we need to attach two GstAudioClippingMeta to the last two frames). With
frames having variable frame size we can't guess based on the file offset.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.