Re: [FFmpeg-devel] [PATCH] avcodec/cbs_vp8: Remove empty flush callback

[Andreas Rheinhardt <andreas.rheinhardt@outlook.com>]

Andreas Rheinhardt:
> This callback is optional and should therefore only be set
> if it actually does something.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
>  libavcodec/cbs_vp8.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/libavcodec/cbs_vp8.c b/libavcodec/cbs_vp8.c
> index 065156c248..eabdef358f 100644
> --- a/libavcodec/cbs_vp8.c
> +++ b/libavcodec/cbs_vp8.c
> @@ -353,11 +353,6 @@ static int cbs_vp8_assemble_fragment(CodedBitstreamContext *ctx,
>      return AVERROR_PATCHWELCOME;
>  }
>  
> -static void cbs_vp8_flush(CodedBitstreamContext *ctx)
> -{
> -    // Do nothing.
> -}
> -
>  static const CodedBitstreamUnitTypeDescriptor cbs_vp8_unit_types[] = {
>      CBS_UNIT_TYPE_INTERNAL_REF(0, VP8RawFrame, data),
>      CBS_UNIT_TYPE_END_OF_LIST,
> @@ -374,7 +369,5 @@ const CodedBitstreamType ff_cbs_type_vp8 = {
>      .read_unit         = &cbs_vp8_read_unit,
>      .write_unit        = &cbs_vp8_write_unit,
>  
> -    .flush             = &cbs_vp8_flush,
> -
>      .assemble_fragment = &cbs_vp8_assemble_fragment,
>  };

Will apply this patch tomorrow unless there are objections.

- 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-request@ffmpeg.org with subject "unsubscribe".