[FFmpeg-devel] [PATCH] avcodec/jpeglsenc: Remove redundant pixel format checks

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Mon Apr 12 20:07:43 EEST 2021


This encoder has AVCodec.pix_fmts set, so ff_encode_preinit() already
checks for this.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
---
Will apply tomorrow unless there are objections.

 libavcodec/jpeglsenc.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/libavcodec/jpeglsenc.c b/libavcodec/jpeglsenc.c
index 2bb6b1407a..d03ce32f41 100644
--- a/libavcodec/jpeglsenc.c
+++ b/libavcodec/jpeglsenc.c
@@ -429,14 +429,6 @@ FF_DISABLE_DEPRECATION_WARNINGS
 FF_ENABLE_DEPRECATION_WARNINGS
 #endif
 
-    if (ctx->pix_fmt != AV_PIX_FMT_GRAY8  &&
-        ctx->pix_fmt != AV_PIX_FMT_GRAY16 &&
-        ctx->pix_fmt != AV_PIX_FMT_RGB24  &&
-        ctx->pix_fmt != AV_PIX_FMT_BGR24) {
-        av_log(ctx, AV_LOG_ERROR,
-               "Only grayscale and RGB24/BGR24 images are supported\n");
-        return -1;
-    }
     return 0;
 }
 
-- 
2.27.0



More information about the ffmpeg-devel mailing list