[FFmpeg-devel] [PATCH 2/6] avformat/hlsenc: Remove redundant initializations

Steven Liu lq at chinaffmpeg.org
Sun May 10 05:02:04 EEST 2020



> 2020年5月10日 上午3:15,Andreas Rheinhardt <andreas.rheinhardt at gmail.com> 写道:
> 
> For every variantstream vs, vs->packets_written is set to one, only to be
> set to zero a few lines below. Given that the relevant structure has
> been zeroed during the allocation, this commit removes both assignments.
> A redundant initialization for vs->init_range_length has been removed as
> well a few lines below. Given that the relevant structure has been
> zeroed during the allocation, this commit removes both assignments. A
> redundant initialization for vs->init_range_length has been removed as
> well.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
> libavformat/hlsenc.c | 4 ----
> 1 file changed, 4 deletions(-)
> 
> diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
> index a796c124dd..afb4d2a0c0 100644
> --- a/libavformat/hlsenc.c
> +++ b/libavformat/hlsenc.c
> @@ -833,7 +833,6 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs)
>         av_dict_copy(&st->metadata, vs->streams[i]->metadata, 0);
>     }
> 
> -    vs->packets_written = 1;
>     vs->start_pos = 0;
>     vs->new_start = 1;
> 
> @@ -848,9 +847,6 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs)
>         }
>     }
> 
> -    vs->packets_written = 0;
> -    vs->init_range_length = 0;
> -
>     if ((ret = avio_open_dyn_buf(&oc->pb)) < 0)
>         return ret;
> 
> -- 
> 2.20.1
> 
> _______________________________________________
> 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".

LGTM

Thanks

Steven Liu



More information about the ffmpeg-devel mailing list