On Tue, Sep 13, 2016 at 11:25:39 +0800, Steven Liu wrote:
> + int byterange_mode = hls->flags & (HLS_SINGLE_FILE) || hls->max_seg_size
> > 0;
^ ^
Why the bracket?
> + av_log(oc, AV_LOG_ERROR, "Invalid segment filename template
> '%s' you can try use -use_localtime 1 with it\n", c->basename);
Rather:
av_log(oc, AV_LOG_ERROR, "Invalid segment filename template
'%s', you can try to use -use_localtime 1 with it\n", c->basename);
> + {"hls_segment_size", "set maximum size per segment file, (Byte)",
> OFFSET(max_seg_size), AV_OPT_TYPE_INT, {.i64 = 0}, 0, INT_MAX, E},
{"hls_segment_size", "set maximum size per segment file (in bytes)",
OFFSET(max_seg_size), AV_OPT_TYPE_INT, {.i64 = 0}, 0, INT_MAX, E},
This isn't a flag, right? Perhaps you shouldn't be putting the option
between the flags' lines (but probably still as the final option).
The value of 0 seems to have a special meaning. Is this documented
anywhere? In other words: You should also update doc/muxers.texi. (I
can help with the grammar of the wording.)
Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
