diff mbox

[FFmpeg-devel,02/10] avcodec/mjpegdec: Unify switch statements

Message ID 20190918032607.11774-2-andreas.rheinhardt@gmail.com
State Accepted
Commit 9442b0de1bf607ef9577623780897c0380736dd7
Headers show

Commit Message

Andreas Rheinhardt Sept. 18, 2019, 3:25 a.m. UTC 
This has been forgotten in d5a3a20d.

Found via PVS-Studio (see ticket #8156).

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---
 libavcodec/mjpegdec.c | 5 -----
 1 file changed, 5 deletions(-)

Comments

Michael Niedermayer Sept. 18, 2019, 9 p.m. UTC | #1 
On Wed, Sep 18, 2019 at 05:25:59AM +0200, Andreas Rheinhardt wrote:
> This has been forgotten in d5a3a20d.
> 
> Found via PVS-Studio (see ticket #8156).
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
> ---
>  libavcodec/mjpegdec.c | 5 -----
>  1 file changed, 5 deletions(-)

will apply

thx

[...]
diff mbox

Patch

diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c
index f095afaed8..8b053613fc 100644
--- a/libavcodec/mjpegdec.c
+++ b/libavcodec/mjpegdec.c
@@ -555,11 +555,6 @@  int ff_mjpeg_decode_sof(MJpegDecodeContext *s)
         case 0x22122100:
         case 0x21211100:
         case 0x22211200:
-            if (s->bits <= 8) s->avctx->pix_fmt = s->cs_itu601 ? AV_PIX_FMT_YUV444P : AV_PIX_FMT_YUVJ444P;
-            else
-                goto unk_pixfmt;
-            s->avctx->color_range = s->cs_itu601 ? AVCOL_RANGE_MPEG : AVCOL_RANGE_JPEG;
-            break;
         case 0x22221100:
         case 0x22112200:
         case 0x11222200: