[FFmpeg-cvslog] sunrast: Remove the useless check.

Aneesh Dogra git at videolan.org
Fri Feb 10 07:29:45 CET 2012


ffmpeg | branch: master | Aneesh Dogra <lionaneesh at gmail.com> | Fri Feb 10 11:56:07 2012 +0530| [1b7ff901078064ccd7df3640e0958a344b2c00cb] | committer: Michael Niedermayer

sunrast: Remove the useless check.

in , else (1) { if (!1) } the if conditional will never evaluate to be true.
So as making the check useless.

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=1b7ff901078064ccd7df3640e0958a344b2c00cb
---

 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;



More information about the ffmpeg-cvslog mailing list