[FFmpeg-devel] Fwd: [PATCH] Automatically inserted bitstream filter 'h264_mp4toannexb' into hlsenc

[Steven Liu]

2016-06-29 7:07 GMT+08:00 Hendrik Leppkes <h.leppkes at gmail.com>:

> On Wed, Jun 29, 2016 at 12:30 AM, Steven Liu <lingjiujianke at gmail.com>
> wrote:
> >
> >
> > 2016-06-29 0:16 GMT+08:00 Hendrik Leppkes <h.leppkes at gmail.com>:
> >>
> >> On Tue, Jun 28, 2016 at 3:29 PM, Steven Liu <lingjiujianke at gmail.com>
> >> wrote:
> >> > before patched:
> >> > [root at localhost ffmpeg]# ./ffmpeg_g -re -i /root/facebook.mp4 -c
> copy -f
> >> > hls -v verbose -y aaaa.m3u8
> >> > ffmpeg version N-80779-gb18d6c5 Copyright (c) 2000-2016 the FFmpeg
> >> > developers
> >>
> >>
> >> The way I see it, the auto bsf from the mpegts muxer should already
> >> cover this without changes?
> >
> >
> > add the hls_check_bitstream into hls format and call the hls->avf to call
> > the mpegtsenc's check_bitstream,
> > don't do this operation,  it won't call the mpegtsenc's check_bitstream,
> > look at the message bellow:
> >
>
> This is the wrong approach to fixing this however, it should be
> understood first why its not doing this transparently right now.
> hlsenc uses the mpegts muxer like one would use any other muxer, so
> why does autobsf not get used?
>
> If we understand that, maybe it can be fixed in a more generic way,
> and not end up in ugly hacks in hlsenc.
>
> PS: now to the list as well, please just mail to the list, no need to
> directly mail anyone.
>
> - Hendrik
>

Because the m3u8 content the mpegts list, and the list is write by "hls
muxer", and the ffmpeg have the work flow call *check_bitstream*, and
the AVOutputFormat have the check_bitstream method, so I think add the
hls_check_bitstream
is the right way.