[FFmpeg-devel] [PATCH] sunrast: Remove the useless check.

Aneesh Dogra lionaneesh at gmail.com
Fri Feb 10 07:08:07 CET 2012


in , else (1) { if (!1) } the if conditional will never evaluate to be true.
So as making the check useless.
---
 libavcodec/sunrast.c |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/libavcodec/sunrast.c b/libavcodec/sunrast.c
index aa12947..8562e11 100644
--- a/libavcodec/sunrast.c
+++ b/libavcodec/sunrast.c
@@ -153,10 +153,6 @@ static int sunrast_decode_frame(AVCodecContext *avctx, void *data,
     } else if (maplength) {
         unsigned int len = maplength / 3;
 
-        if (!maplength) {
-            av_log(avctx, AV_LOG_ERROR, "colormap expected\n");
-            return -1;
-        }
         if (maplength % 3 || maplength > 768) {
             av_log(avctx, AV_LOG_WARNING, "invalid colormap length\n");
             return -1;
-- 
1.7.1



More information about the ffmpeg-devel mailing list