[FFmpeg-devel] [PATCH] avformat/hlsenc: program_date_time and append_list flags conflict

Steven Liu lq at chinaffmpeg.org
Sun Jan 19 11:20:17 EET 2020



> 在 2020年1月19日,03:31,Bodecs Bela <bodecsb at vivanet.hu> 写道:
> 
> sorry for missing patch file, here it is.
> 
> Bela
> 
> 
> 2020.01.18. 18:56 keltezéssel, Bodecs Bela írta:
>> Dear All,
>> 
>> 
>> When program_date_time flag is present, in m3u8 playlist file each
>> segment has a corresponding EXT-X-PROGRAM-DATE-TIME value. The initial
>> program-date-time value is the actual current time at init and
>> each new segment increments this value by its duration. When append_list
>> flags is also present, existing playlist parsing by hls_append_segment
>> treats existing segments as new segments regarding the program-date-time
>> calculation. But it should not do that, because this way all  the real
>> new segments' EXT-X-PROGRAM-DATE-TIME values will be shifted erroneously
>> by the sum duration of existing segments. Instead it should have
>> decremented the initial program-date-time value by its duration. This
>> would ensure that the first new segment's program-date-time value had
>> the current time as it is expected.
>> This patch corrects this behaviour and prevent existing segments to
>> increment the value of initial_prog_date_time variable but decrements
>> it.
>> 
>> 
>> please review this patch!
>> 
>> thank you in advance,
LGTM
>> 
>> Bela
>> 
>> 
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> 
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> <0001-avformat-hlsenc-program_date_time-and-append_list-fl.patch>_______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".

Thanks
Steven








More information about the ffmpeg-devel mailing list