On 5/25/2016 1:56 PM, Jon Toohill wrote: > --- > libavformat/mp3dec.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/mp3dec.c b/libavformat/mp3dec.c > index 3725d67..192f5ef 100644 > --- a/libavformat/mp3dec.c > +++ b/libavformat/mp3dec.c > @@ -234,6 +234,8 @@ static void mp3_parse_info_tag(AVFormatContext *s, > AVStream *st, > > mp3->start_pad = v>>12; > mp3-> end_pad = v&4095; > + st->codecpar->initial_padding = mp3->start_pad; > + st->codecpar->trailing_padding = mp3->end_pad;
Every other format so far is using the AV_PKT_DATA_SKIP_SAMPLES side data to discard samples from the last packet/frame. See matroska and ogg demuxers, currently used for Opus only. The trailing_padding AVCodecParameters element was added after the above was designed. To be honest i can't say if we should replace one with the other or find a way to keep both, and seeing how AVCodecParameters hasn't made it to a release yet, we're still on time to choose. > st->start_skip_samples = mp3->start_pad + 528 + 1; > if (mp3->frames) { > st->first_discard_sample = -mp3->end_pad + 528 + 1 + mp3->frames > * (int64_t)spf; > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel