On Wed, 30 Sep 2015 15:41:04 +0200 Michael Niedermayer <michae...@gmx.at> wrote:
> On Wed, Sep 30, 2015 at 02:54:11PM +0200, wm4 wrote: > > Commits 43bc5cf9 and c5371f77 add code for skipping initial zeros in mp3 > > packets. (There are no explanations, no samples.) > > > > This code forgot to report to the user that data was skipped at all. IT > > should do this by adding the skipped amount of bytes to the amount of > > consumed bytes returned by the decode call. > > > > Since audio codecs allow partial packet decoding, the user application > > has to rely on the return value. It will remove the data reported as > > consumed by the decoder, and feed it to the decoder again. This resulted > > in the mp3 frame after the zero region to be decoded over and over > > again, until the zero region was finally skipped by the application. > > > > Fixes trac ticket #4890. > > --- > > libavcodec/mpegaudiodec_template.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > LGTM > > thanks > > [...] Pushed. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
