[FFmpeg-devel] [PATCH 4/5] avformat/mxfenc: Remove dead code

Tomas Härdin git at haerdin.se
Tue Jun 18 17:32:18 EEST 2024


fre 2024-06-07 klockan 02:32 +0200 skrev Michael Niedermayer:
> Fixes: CID1524681 Logically dead code
> 
> Sponsored-by: Sovereign Tech Fund
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/mxfenc.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c
> index f424858fc4e..b8e7bfe3018 100644
> --- a/libavformat/mxfenc.c
> +++ b/libavformat/mxfenc.c
> @@ -2606,9 +2606,6 @@ static int mxf_parse_ffv1_frame(AVFormatContext
> *s, AVStream *st, AVPacket *pkt)
>          ff_build_rac_states(&c, 0.05 * (1LL << 32), 256 - 8);
>          v = get_ffv1_unsigned_symbol(&c, state);
>          av_assert0(v >= 2);
> -        if (v > 4) {
> -            return 0;
> -        }
>          if (v > 4) {
>              av_log(s, AV_LOG_ERROR, "unsupported ffv1 version %d\n",
> v);
>              return 0;

Commit message isn't quite accurate - this rather resurrects the error
print

/Tomas



More information about the ffmpeg-devel mailing list