From: Misty De Meo <mistyde...@gmail.com> > the initialization to 0 is unneeded
OK - updated in the included patch. I've also included a second patch which adds an explicit goto and break in aiffdec, on top of the patch you already applied, and the Changelog entry that I forgot to include in that patch. I realize it's a niche format, but how do you feel about including the fate test for this? It's in this mail: http://ffmpeg.org/pipermail/ffmpeg-devel/2018-January/223413.html Misty De Meo (2): adpcm: consume remainder after consuming XA chunks aiff: add explicit goto got_sound Changelog | 1 + libavcodec/adpcm.c | 7 +++++++ libavformat/aiffdec.c | 3 +++ 3 files changed, 11 insertions(+) -- 2.15.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
