----- On Mar 6, 2020, at 11:08 AM, Andreas Rheinhardt
andreas.rheinha...@gmail.com wrote:
> Hongcheng Zhong:
>> ----- On Mar 6, 2020, at 12:55 AM, Andreas Rheinhardt
>> andreas.rheinha...@gmail.com wrote:
>>
>>> Hongcheng Zhong:
>>>> ----- On Mar 5, 2020, at 11:38 PM, lq l...@chinaffmpeg.org wrote:
>>>>
>>>>>> 2020年3月5日 下午9:41，Hongcheng Zhong <sj.hc_zh...@sjtu.edu.cn> 写道：
>>>>>>
>>>>>> This patch fixes Bug #8469
>>>>>> If x264 baseline profile is used with other profiles,
>>>>>> start_pts will be initialized to audio stream's first pts,
>>>>>> while the duration is calculated based on video stream's pts.
>>>>>> In this patch the start_pts is initialized with the correct stream's
>>>>>> first pts.
>>>>>>
>>>>>> Signed-off-by: Hongcheng Zhong <sj.hc_zh...@sjtu.edu.cn>
>>>>>> ---
>>>>>> libavformat/hlsenc.c | 10 +++++++++-
>>>>>> 1 file changed, 9 insertions(+), 1 deletion(-)
>>>>>>
>>>>>> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
>>>>>> index f6dd894..3b2434f 100644
>>>>>> --- a/libavformat/hlsenc.c
>>>>>> +++ b/libavformat/hlsenc.c
>>>>>> @@ -126,6 +126,7 @@ typedef struct VariantStream {
>>>>>> int has_video;
>>>>>> int has_subtitle;
>>>>>> int new_start;
>>>>>> + int start_pts_from_audio;
>>>>>> double dpp; // duration per packet
>>>>>> int64_t start_pts;
>>>>>> int64_t end_pts;
>>>>>> @@ -2274,9 +2275,15 @@ static int hls_write_packet(AVFormatContext *s,
>>>>>> AVPacket
>>>>>> *pkt)
>>>>>>
>>>>>> if (vs->start_pts == AV_NOPTS_VALUE) {
>>>>>> vs->start_pts = pkt->pts;
>>>>>> + if (st->codecpar->codec_type == AVMEDIA_TYPE_AUDIO)
>>>>>> + vs->start_pts_from_audio = 1;
>>>>>> + }
>>>>>> + if (vs->start_pts_from_audio && st->codecpar->codec_type ==
>>>>>> AVMEDIA_TYPE_VIDEO && vs->start_pts > pkt->pts) {
>>>>>> + vs->start_pts = pkt->pts;
>>>>>> + vs->start_pts_from_audio = 0;
>>>>>> }
>>>>>>
>>>>>> - if (vs->has_video) {
>>>>>> + if (vs->has_video) {
>>>>> What’s happen with this line?
>>>>>> can_split = st->codecpar->codec_type == AVMEDIA_TYPE_VIDEO &&
>>>>>> ((pkt->flags & AV_PKT_FLAG_KEY) || (hls->flags &
>>>>>> HLS_SPLIT_BY_TIME));
>>>>>> is_ref_pkt = (st->codecpar->codec_type == AVMEDIA_TYPE_VIDEO) &&
>>>>>> (pkt->stream_index == vs->reference_stream_index);
>>>>>> @@ -2751,6 +2758,7 @@ static int hls_init(AVFormatContext *s)
>>>>>> vs->sequence = hls->start_sequence;
>>>>>> vs->start_pts = AV_NOPTS_VALUE;
>>>>>> vs->end_pts = AV_NOPTS_VALUE;
>>>>>> + vs->start_pts_from_audio = 0;
>>>>>> vs->current_segment_final_filename_fmt[0] = '\0';
>>>>>>
>>>>>> if (hls->flags & HLS_SPLIT_BY_TIME && hls->flags &
>>>>>> HLS_INDEPENDENT_SEGMENTS) {
>>>>>> --
>>>>>> 2.7.4
>>>>>>
>>>>>> _______________________________________________
>>>>>> 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".
>>>>
>>>> Actually, this line is useless because that vs is initialized as all zero.
>>>> Should I remove this line and submit this patch as v2 again? Thanks for
>>>> your
>>>> reply.
>>>
>>> But vs->has_video is initialized to a potentially nonzero value in
>>> hls_init().
>>>
>>> - Andreas
>>> _______________________________________________
>>> 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".
>>
>> I write that line to make sure that vs->start_pts_from_audio is initialized
>> as
>> 0.
>> Since `hls->var_streams = av_mallocz(sizeof(*hls->var_streams) *
>> nb_varstreams);`
>> in hlsenc.c: 1897, var_streams is allocated with av_mallocz(), that line is
>> not
>> necessary.
>
> There seems to be a misunderstanding: The line that Steven asked about
> is "if (vs->has_video) {". By changing it you mixed cosmetic changes
> and functional ones; this should not be done.
> Steven did not ask about "vs->start_pts_from_audio = 0;". But you are
> right: It is redundant.
>
> - Andreas
> _______________________________________________
> 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".
Sorry, my mistake. "Cosmetic changes should be kept in separate patches."
Thanks a lot for pointing this out.
_______________________________________________
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".
