[FFmpeg-devel] [PATCH] avformat/hlsenc: actual segment file size and duration in segment filenames

[Moritz Barsnick]

On Tue, Jan 03, 2017 at 09:38:18 +0100, Bodecs Bela wrote:
>  doc/muxers.texi | 19 ++++++++++++++++++-
>  1 file changed, 18 insertions(+), 1 deletion(-)

You should probably squash this with the other patch, into one commit.

> -Makes it possible to use segment indexes as %%d besides date/time values when use_localtime is on.
> +Makes it possible to use segment indexes as %%d in hls_segment_filename expression besides date/time values when use_localtime is on. To get fixed width numbers with trailing zeroes, %%0xd format is available where x is the required width.

You could break these lines (texinfo formatting will handle the real
representation later), but I'm not sure it's totally necessary.

> + at example
> +./ffmpeg -loglevel info -y -f lavfi -i color=c=red:size=640x480:r=25  \
> +   -f lavfi -i sine=f=440:b=4:r=44100 -c:v mpeg2video -g 25 -acodec aac -cutoff 20000 -ac 2 -ar 44100 -ab 192k \
> +   -f hls -hls_time 3 -hls_list_size 5 \

a) replace ./ffmpeg with ffmpeg (as most users would expect to use it),
b) please use simpler syntax for the stuff not related to what you are
trying to document; i.e.: ffmpeg -i input.mov -f hls -hls_time 3 -hls_list_size 5 [...]
> +This will produce segments like this:
> + at file{segment_20170102194334_0003_00122200_0000003000000.ts}, @file{segment_20170102194334_0004_00120072_0000003000000.ts} etc.

The rest seems fine to me!

Moritz