> 2020年4月29日 下午3:07,myp...@gmail.com 写道:
>
> On Wed, Apr 29, 2020 at 12:44 PM Steven Liu wrote:
>>
>> segment duration is using vs duration which compute by frame per second,
>> that can not fix problem of VFR video stream, so compute the duration
>> when split the segment, set the segment target duration use
>> current packet pts minus the prev segment end pts and plus current
>> packet's duration.
>>
>> Reported-by: Zhao Jun
>> Signed-off-by: Steven Liu
>> ---
>> libavformat/hlsenc.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
>> index d75684741f..9e8b34d83c 100644
>> --- a/libavformat/hlsenc.c
>> +++ b/libavformat/hlsenc.c
>> @@ -2460,7 +2460,8 @@ static int hls_write_packet(AVFormatContext *s,
>> AVPacket *pkt)
>> }
>>
>> if (vs->start_pos || hls->segment_type != SEGMENT_TYPE_FMP4) {
>> -ret = hls_append_segment(s, hls, vs, vs->duration,
>> vs->start_pos, vs->size);
>> +double cur_duration = (double)(pkt->pts - vs->end_pts) *
>> st->time_base.num / st->time_base.den + vs->dpp;
>> +ret = hls_append_segment(s, hls, vs, cur_duration,
>> vs->start_pos, vs->size);
>> vs->end_pts = pkt->pts;
>> vs->duration = 0;
>> if (ret < 0) {
>> --
>> 2.25.0
>>
>>
> LGTM, Tested and verified, tks the quick fix.
Sorry I make a mistake, the pkt should not in current segment, it should in the
next segment, so just remove the + vs->dpp
I will submit version 2 patch
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Thanks
Steven Liu
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".