2017-01-03 16:47 GMT+08:00 Bodecs Bela <bode...@vivanet.hu>: > > > 2017.01.03. 9:23 keltezéssel, Steven Liu írta: > >> 2017-01-03 16:11 GMT+08:00 Bodecs Bela <bode...@vivanet.hu>: >> >> >>> 2017.01.03. 2:34 keltezéssel, Steven Liu írta: >>> >>> 2017-01-03 8:22 GMT+08:00 Steven Liu <lingjiujia...@gmail.com>: >>>> >>>> >>>> 2017-01-03 2:58 GMT+08:00 Bodecs Bela <bode...@vivanet.hu>: >>>>> >>>>> Dear All, >>>>> >>>>>> this patch makes it possible to put actual segment file size (measured >>>>>> in bytes) and/or duration (calculated in microseconds) into segment >>>>>> filenames. This feature is useful when post-processing live streaming >>>>>> access log files. New behaviour works only when -use_localtime option >>>>>> is set and second_level_segment_size or/and >>>>>> second_level_segment_duration new hls_flags are specified. %%s is the >>>>>> placeholder for size and %%t for duration in hls_segment_filename >>>>>> option. Fix sized trailing zeropadding also works eg. %%09s or %%023t. >>>>>> >>>>>> A command to test new features: >>>>>> ./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 -hls_flags >>>>>> second_level_segment_index+second_level_segment_size+second_ >>>>>> level_segment_duration >>>>>> -use_localtime 1 -use_localtime_mkdir 1 -hls_segment_filename >>>>>> "segment_%Y%m%d%H%M%S_%%04d_%%08s_%%013t.ts" stream.m3u8 >>>>>> >>>>>> this will produce segments like this: >>>>>> segment_20170102194334_0003_00122200_0000003000000.ts >>>>>> segment_20170102194334_0004_00120072_0000003000000.ts >>>>>> etc. >>>>>> >>>>>> >>>>>> thank you in advance, >>>>>> >>>>>> Bela Bodecs >>>>>> >>>>>> _______________________________________________ >>>>>> ffmpeg-devel mailing list >>>>>> ffmpeg-devel@ffmpeg.org >>>>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >>>>>> >>>>>> >>>>>> you should add document for the new option. >>>>>> >>>>> in +static int replace_int_data_in_filename(char *buf, int buf_size, >>>>> >>>> const >>>> char *filename, char placeholder, int64_t number) >>>> >>>> + c = *p; // >>>> will add comment? >>>> >>>> in @@ -388,6 +443,38 @@ static int hls_append_segment(struct >>>> AVFormatContext *s, HLSContext *hls, double >>>> >>>> + if ((hls->flags & (HLS_SECOND_LEVEL_SEGMENT_SIZE | >>>> HLS_SECOND_LEVEL_SEGMENT_DURATION)) && >>>> strlen(hls->current_segment_final_filename_fmt)) { >>>> This is one line and the line is too long, maybe split to or three line >>>> be >>>> better, The other too long line should be split. >>>> >>>> I have shortened and splitted the lines and removed empty comment sign. >>> >>> _______________________________________________ >>> >>>> ffmpeg-devel mailing list >>>> ffmpeg-devel@ffmpeg.org >>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >>>> >>>> >>> _______________________________________________ >>> ffmpeg-devel mailing list >>> ffmpeg-devel@ffmpeg.org >>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >>> >>> for example: >>> >> + av_log(c, AV_LOG_ERROR, >> + "Invalid second level segment filename >> template >> '%s', you can try to remove second_level_segment_index flag\n", >> >> >> + av_log(c, AV_LOG_ERROR,"Invalid second level segment >> filename template '%s', " >> + "you can try to remove >> second_level_segment_index flag\n", >> >> What about this? >> > ok, I have done. > > Ok, Waiting Moritz Barsnick review doc, and merge two patch to one.
> _______________________________________________ >> ffmpeg-devel mailing list >> ffmpeg-devel@ffmpeg.org >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> > > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
