On 4/20/2016 4:30 AM, Hendrik Leppkes wrote: > On Wed, Apr 20, 2016 at 4:03 AM, James Almer <jamr...@gmail.com> wrote: >> Signed-off-by: James Almer <jamr...@gmail.com> >> --- >> tests/lavf-regression.sh | 2 +- >> tests/ref/lavf-fate/latm | 6 +++--- >> 2 files changed, 4 insertions(+), 4 deletions(-) >> > > I'm confused, this BSF is not for LATM, its for ADTS, so how exactly > are these two things related? > > - Hendrik
The LATM muxer expects an asc bitstream (Raw AAC afaik). If it gets an ADTS bitstream, it will automatically call the bsf. With this patch, we're copying an ADTS stream instead of a raw one, so the test will cover both LATM muxing and the adtstoasc bsf, the latter which is currently not covered by fate. http://coverage.ffmpeg.org/src/libavcodec/aac_adtstoasc_bsf.c.gcov.html _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel