Message ID | 20200121231243.26529-1-cus@passwd.hu |
---|---|
State | Accepted |
Headers | show |
Series | [FFmpeg-devel] avformat/hlsenc: fix hls_ts_options with mpegts | expand |
Context | Check | Description |
---|---|---|
andriy/ffmpeg-patchwork | success | Make fate finished |
On Wed, 22 Jan 2020, Marton Balint wrote: > Was broken since cdbf8847ea97a985dfd55432e1384bb7fe5d2d3b. Ping, will apply soon. Thanks, Marton > > Signed-off-by: Marton Balint <cus@passwd.hu> > --- > libavformat/hlsenc.c | 36 +++++++++++++++++------------------- > 1 file changed, 17 insertions(+), 19 deletions(-) > > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c > index 2b3d3742d9..87b861d437 100644 > --- a/libavformat/hlsenc.c > +++ b/libavformat/hlsenc.c > @@ -764,6 +764,7 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs) > AVFormatContext *oc; > AVFormatContext *vtt_oc = NULL; > int byterange_mode = (hls->flags & HLS_SINGLE_FILE) || (hls->max_seg_size > 0); > + int remaining_options; > int i, ret; > > ret = avformat_alloc_output_context2(&vs->avf, vs->oformat, NULL, NULL); > @@ -852,21 +853,25 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs) > return ret; > } > > + av_dict_copy(&options, hls->format_options, 0); > if (hls->segment_type == SEGMENT_TYPE_FMP4) { > - int remaining_options; > - > - av_dict_copy(&options, hls->format_options, 0); > av_dict_set(&options, "fflags", "-autobsf", 0); > av_dict_set(&options, "movflags", "+frag_custom+dash+delay_moov", AV_DICT_APPEND); > - ret = avformat_init_output(oc, &options); > - remaining_options = av_dict_count(options); > - av_dict_free(&options); > - if (ret < 0) > - return ret; > - if (remaining_options) { > - av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n"); > - return AVERROR(EINVAL); > - } > + } else { > + /* We only require one PAT/PMT per segment. */ > + char period[21]; > + snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1); > + av_dict_set(&options, "sdt_period", period, 0); > + av_dict_set(&options, "pat_period", period, 0); > + } > + ret = avformat_init_output(oc, &options); > + remaining_options = av_dict_count(options); > + av_dict_free(&options); > + if (ret < 0) > + return ret; > + if (remaining_options) { > + av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n"); > + return AVERROR(EINVAL); > } > avio_flush(oc->pb); > return 0; > @@ -1683,15 +1688,8 @@ static int hls_start(AVFormatContext *s, VariantStream *vs) > } > } > if (c->segment_type != SEGMENT_TYPE_FMP4) { > - /* We only require one PAT/PMT per segment. */ > if (oc->oformat->priv_class && oc->priv_data) { > - char period[21]; > - > - snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1); > - > av_opt_set(oc->priv_data, "mpegts_flags", "resend_headers", 0); > - av_opt_set(oc->priv_data, "sdt_period", period, 0); > - av_opt_set(oc->priv_data, "pat_period", period, 0); > } > if (c->flags & HLS_SINGLE_FILE) { > set_http_options(s, &options, c); > -- > 2.16.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-request@ffmpeg.org with subject "unsubscribe".
Marton Balint <cus@passwd.hu> 于2020年1月31日周五 上午4:40写道: > > > > On Wed, 22 Jan 2020, Marton Balint wrote: > > > Was broken since cdbf8847ea97a985dfd55432e1384bb7fe5d2d3b. > > Ping, will apply soon. > > Thanks, > Marton > > > > > Signed-off-by: Marton Balint <cus@passwd.hu> > > --- > > libavformat/hlsenc.c | 36 +++++++++++++++++------------------- > > 1 file changed, 17 insertions(+), 19 deletions(-) > > > > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c > > index 2b3d3742d9..87b861d437 100644 > > --- a/libavformat/hlsenc.c > > +++ b/libavformat/hlsenc.c > > @@ -764,6 +764,7 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs) > > AVFormatContext *oc; > > AVFormatContext *vtt_oc = NULL; > > int byterange_mode = (hls->flags & HLS_SINGLE_FILE) || (hls->max_seg_size > 0); > > + int remaining_options; > > int i, ret; > > > > ret = avformat_alloc_output_context2(&vs->avf, vs->oformat, NULL, NULL); > > @@ -852,21 +853,25 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs) > > return ret; > > } > > > > + av_dict_copy(&options, hls->format_options, 0); > > if (hls->segment_type == SEGMENT_TYPE_FMP4) { > > - int remaining_options; > > - > > - av_dict_copy(&options, hls->format_options, 0); > > av_dict_set(&options, "fflags", "-autobsf", 0); > > av_dict_set(&options, "movflags", "+frag_custom+dash+delay_moov", AV_DICT_APPEND); > > - ret = avformat_init_output(oc, &options); > > - remaining_options = av_dict_count(options); > > - av_dict_free(&options); > > - if (ret < 0) > > - return ret; > > - if (remaining_options) { > > - av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n"); > > - return AVERROR(EINVAL); > > - } > > + } else { > > + /* We only require one PAT/PMT per segment. */ > > + char period[21]; > > + snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1); > > + av_dict_set(&options, "sdt_period", period, 0); > > + av_dict_set(&options, "pat_period", period, 0); > > + } > > + ret = avformat_init_output(oc, &options); > > + remaining_options = av_dict_count(options); > > + av_dict_free(&options); > > + if (ret < 0) > > + return ret; > > + if (remaining_options) { > > + av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n"); > > + return AVERROR(EINVAL); > > } > > avio_flush(oc->pb); > > return 0; > > @@ -1683,15 +1688,8 @@ static int hls_start(AVFormatContext *s, VariantStream *vs) > > } > > } > > if (c->segment_type != SEGMENT_TYPE_FMP4) { > > - /* We only require one PAT/PMT per segment. */ > > if (oc->oformat->priv_class && oc->priv_data) { > > - char period[21]; > > - > > - snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1); > > - > > av_opt_set(oc->priv_data, "mpegts_flags", "resend_headers", 0); > > - av_opt_set(oc->priv_data, "sdt_period", period, 0); > > - av_opt_set(oc->priv_data, "pat_period", period, 0); > > } > > if (c->flags & HLS_SINGLE_FILE) { > > set_http_options(s, &options, c); > > -- > > 2.16.4 > > LGTM > > _______________________________________________ > > ffmpeg-devel mailing list > > ffmpeg-devel@ffmpeg.org > > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > To unsubscribe, visit link above, or email > > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
On Fri, 31 Jan 2020, Steven Liu wrote: > Marton Balint <cus@passwd.hu> 于2020年1月31日周五 上午4:40写道: >> >> >> >> On Wed, 22 Jan 2020, Marton Balint wrote: >> >> > Was broken since cdbf8847ea97a985dfd55432e1384bb7fe5d2d3b. >> >> Ping, will apply soon. >> >> Thanks, >> Marton >> >> > >> > Signed-off-by: Marton Balint <cus@passwd.hu> >> > --- >> > libavformat/hlsenc.c | 36 +++++++++++++++++------------------- >> > 1 file changed, 17 insertions(+), 19 deletions(-) >> > >> > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c >> > index 2b3d3742d9..87b861d437 100644 >> > --- a/libavformat/hlsenc.c >> > +++ b/libavformat/hlsenc.c >> > @@ -764,6 +764,7 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs) >> > AVFormatContext *oc; >> > AVFormatContext *vtt_oc = NULL; >> > int byterange_mode = (hls->flags & HLS_SINGLE_FILE) || (hls->max_seg_size > 0); >> > + int remaining_options; >> > int i, ret; >> > >> > ret = avformat_alloc_output_context2(&vs->avf, vs->oformat, NULL, NULL); >> > @@ -852,21 +853,25 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs) >> > return ret; >> > } >> > >> > + av_dict_copy(&options, hls->format_options, 0); >> > if (hls->segment_type == SEGMENT_TYPE_FMP4) { >> > - int remaining_options; >> > - >> > - av_dict_copy(&options, hls->format_options, 0); >> > av_dict_set(&options, "fflags", "-autobsf", 0); >> > av_dict_set(&options, "movflags", "+frag_custom+dash+delay_moov", AV_DICT_APPEND); >> > - ret = avformat_init_output(oc, &options); >> > - remaining_options = av_dict_count(options); >> > - av_dict_free(&options); >> > - if (ret < 0) >> > - return ret; >> > - if (remaining_options) { >> > - av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n"); >> > - return AVERROR(EINVAL); >> > - } >> > + } else { >> > + /* We only require one PAT/PMT per segment. */ >> > + char period[21]; >> > + snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1); >> > + av_dict_set(&options, "sdt_period", period, 0); >> > + av_dict_set(&options, "pat_period", period, 0); >> > + } >> > + ret = avformat_init_output(oc, &options); >> > + remaining_options = av_dict_count(options); >> > + av_dict_free(&options); >> > + if (ret < 0) >> > + return ret; >> > + if (remaining_options) { >> > + av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n"); >> > + return AVERROR(EINVAL); >> > } >> > avio_flush(oc->pb); >> > return 0; >> > @@ -1683,15 +1688,8 @@ static int hls_start(AVFormatContext *s, VariantStream *vs) >> > } >> > } >> > if (c->segment_type != SEGMENT_TYPE_FMP4) { >> > - /* We only require one PAT/PMT per segment. */ >> > if (oc->oformat->priv_class && oc->priv_data) { >> > - char period[21]; >> > - >> > - snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1); >> > - >> > av_opt_set(oc->priv_data, "mpegts_flags", "resend_headers", 0); >> > - av_opt_set(oc->priv_data, "sdt_period", period, 0); >> > - av_opt_set(oc->priv_data, "pat_period", period, 0); >> > } >> > if (c->flags & HLS_SINGLE_FILE) { >> > set_http_options(s, &options, c); >> > -- >> > 2.16.4 >> > > LGTM Thanks, applied. Regards, Marton
diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 2b3d3742d9..87b861d437 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -764,6 +764,7 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs) AVFormatContext *oc; AVFormatContext *vtt_oc = NULL; int byterange_mode = (hls->flags & HLS_SINGLE_FILE) || (hls->max_seg_size > 0); + int remaining_options; int i, ret; ret = avformat_alloc_output_context2(&vs->avf, vs->oformat, NULL, NULL); @@ -852,21 +853,25 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs) return ret; } + av_dict_copy(&options, hls->format_options, 0); if (hls->segment_type == SEGMENT_TYPE_FMP4) { - int remaining_options; - - av_dict_copy(&options, hls->format_options, 0); av_dict_set(&options, "fflags", "-autobsf", 0); av_dict_set(&options, "movflags", "+frag_custom+dash+delay_moov", AV_DICT_APPEND); - ret = avformat_init_output(oc, &options); - remaining_options = av_dict_count(options); - av_dict_free(&options); - if (ret < 0) - return ret; - if (remaining_options) { - av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n"); - return AVERROR(EINVAL); - } + } else { + /* We only require one PAT/PMT per segment. */ + char period[21]; + snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1); + av_dict_set(&options, "sdt_period", period, 0); + av_dict_set(&options, "pat_period", period, 0); + } + ret = avformat_init_output(oc, &options); + remaining_options = av_dict_count(options); + av_dict_free(&options); + if (ret < 0) + return ret; + if (remaining_options) { + av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n"); + return AVERROR(EINVAL); } avio_flush(oc->pb); return 0; @@ -1683,15 +1688,8 @@ static int hls_start(AVFormatContext *s, VariantStream *vs) } } if (c->segment_type != SEGMENT_TYPE_FMP4) { - /* We only require one PAT/PMT per segment. */ if (oc->oformat->priv_class && oc->priv_data) { - char period[21]; - - snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1); - av_opt_set(oc->priv_data, "mpegts_flags", "resend_headers", 0); - av_opt_set(oc->priv_data, "sdt_period", period, 0); - av_opt_set(oc->priv_data, "pat_period", period, 0); } if (c->flags & HLS_SINGLE_FILE) { set_http_options(s, &options, c);
Was broken since cdbf8847ea97a985dfd55432e1384bb7fe5d2d3b. Signed-off-by: Marton Balint <cus@passwd.hu> --- libavformat/hlsenc.c | 36 +++++++++++++++++------------------- 1 file changed, 17 insertions(+), 19 deletions(-)