Merge commit '3865ba7b21aef5d60183719e238361ec8797ab5a'
authorMichael Niedermayer <michaelni@gmx.at>
Thu, 11 Jul 2013 09:12:48 +0000 (11:12 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Thu, 11 Jul 2013 09:12:48 +0000 (11:12 +0200)
* commit '3865ba7b21aef5d60183719e238361ec8797ab5a':
  iff: K&R formatting cosmetics

Conflicts:
libavcodec/iff.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavcodec/iff.c

@@@ -153,170 -131,16 +153,168 @@@ static int cmap_read_palette(AVCodecCon
  
      count = 1 << avctx->bits_per_coded_sample;
      // If extradata is smaller than actually needed, fill the remaining with black.
 -    count = FFMIN(avctx->extradata_size / 3, count);
 +    count = FFMIN(palette_size / 3, count);
      if (count) {
-         for (i=0; i < count; i++) {
+         for (i = 0; i < count; i++)
 -            pal[i] = 0xFF000000 | AV_RB24(avctx->extradata + i * 3);
 +            pal[i] = 0xFF000000 | AV_RB24(palette + i*3);
-         }
 +        if (s->flags && count >= 32) { // EHB
 +            for (i = 0; i < 32; i++)
 +                pal[i + 32] = 0xFF000000 | (AV_RB24(palette + i*3) & 0xFEFEFE) >> 1;
 +            count = FFMAX(count, 64);
 +        }
      } else { // Create gray-scale color palette for bps < 8
          count = 1 << avctx->bits_per_coded_sample;
  
-         for (i=0; i < count; i++) {
+         for (i = 0; i < count; i++)
              pal[i] = 0xFF000000 | gray2rgb((i * 255) >> avctx->bits_per_coded_sample);
-         }
      }
 +    if (s->masking == MASK_HAS_MASK) {
 +        memcpy(pal + (1 << avctx->bits_per_coded_sample), pal, count * 4);
 +        for (i = 0; i < count; i++)
 +            pal[i] &= 0xFFFFFF;
 +    } else if (s->masking == MASK_HAS_TRANSPARENT_COLOR &&
 +        s->transparency < 1 << avctx->bits_per_coded_sample)
 +        pal[s->transparency] &= 0xFFFFFF;
 +    return 0;
 +}
 +
 +/**
 + * Extracts the IFF extra context and updates internal
 + * decoder structures.
 + *
 + * @param avctx the AVCodecContext where to extract extra context to
 + * @param avpkt the AVPacket to extract extra context from or NULL to use avctx
 + * @return 0 in case of success, a negative error code otherwise
 + */
 +static int extract_header(AVCodecContext *const avctx,
 +                          const AVPacket *const avpkt) {
 +    const uint8_t *buf;
 +    unsigned buf_size;
 +    IffContext *s = avctx->priv_data;
 +    int i, palette_size;
 +
 +    if (avctx->extradata_size < 2) {
 +        av_log(avctx, AV_LOG_ERROR, "not enough extradata\n");
 +        return AVERROR_INVALIDDATA;
 +    }
 +    palette_size = avctx->extradata_size - AV_RB16(avctx->extradata);
 +
 +    if (avpkt) {
 +        int image_size;
 +        if (avpkt->size < 2)
 +            return AVERROR_INVALIDDATA;
 +        image_size = avpkt->size - AV_RB16(avpkt->data);
 +        buf = avpkt->data;
 +        buf_size = bytestream_get_be16(&buf);
 +        if (buf_size <= 1 || image_size <= 1) {
 +            av_log(avctx, AV_LOG_ERROR,
 +                   "Invalid image size received: %u -> image data offset: %d\n",
 +                   buf_size, image_size);
 +            return AVERROR_INVALIDDATA;
 +        }
 +    } else {
 +        buf = avctx->extradata;
 +        buf_size = bytestream_get_be16(&buf);
 +        if (buf_size <= 1 || palette_size < 0) {
 +            av_log(avctx, AV_LOG_ERROR,
 +                   "Invalid palette size received: %u -> palette data offset: %d\n",
 +                   buf_size, palette_size);
 +            return AVERROR_INVALIDDATA;
 +        }
 +    }
 +
 +    if (buf_size >= 41) {
 +        s->compression  = bytestream_get_byte(&buf);
 +        s->bpp          = bytestream_get_byte(&buf);
 +        s->ham          = bytestream_get_byte(&buf);
 +        s->flags        = bytestream_get_byte(&buf);
 +        s->transparency = bytestream_get_be16(&buf);
 +        s->masking      = bytestream_get_byte(&buf);
 +        for (i = 0; i < 16; i++)
 +            s->tvdc[i] = bytestream_get_be16(&buf);
 +
 +        if (s->masking == MASK_HAS_MASK) {
 +            if (s->bpp >= 8 && !s->ham) {
 +                avctx->pix_fmt = AV_PIX_FMT_RGB32;
 +                av_freep(&s->mask_buf);
 +                av_freep(&s->mask_palbuf);
 +                s->mask_buf = av_malloc((s->planesize * 32) + FF_INPUT_BUFFER_PADDING_SIZE);
 +                if (!s->mask_buf)
 +                    return AVERROR(ENOMEM);
 +                if (s->bpp > 16) {
 +                    av_log(avctx, AV_LOG_ERROR, "bpp %d too large for palette\n", s->bpp);
 +                    av_freep(&s->mask_buf);
 +                    return AVERROR(ENOMEM);
 +                }
 +                s->mask_palbuf = av_malloc((2 << s->bpp) * sizeof(uint32_t) + FF_INPUT_BUFFER_PADDING_SIZE);
 +                if (!s->mask_palbuf) {
 +                    av_freep(&s->mask_buf);
 +                    return AVERROR(ENOMEM);
 +                }
 +            }
 +            s->bpp++;
 +        } else if (s->masking != MASK_NONE && s->masking != MASK_HAS_TRANSPARENT_COLOR) {
 +            av_log(avctx, AV_LOG_ERROR, "Masking not supported\n");
 +            return AVERROR_PATCHWELCOME;
 +        }
 +        if (!s->bpp || s->bpp > 32) {
 +            av_log(avctx, AV_LOG_ERROR, "Invalid number of bitplanes: %u\n", s->bpp);
 +            return AVERROR_INVALIDDATA;
 +        } else if (s->ham >= 8) {
 +            av_log(avctx, AV_LOG_ERROR, "Invalid number of hold bits for HAM: %u\n", s->ham);
 +            return AVERROR_INVALIDDATA;
 +        }
 +
 +        av_freep(&s->ham_buf);
 +        av_freep(&s->ham_palbuf);
 +
 +        if (s->ham) {
 +            int i, count = FFMIN(palette_size / 3, 1 << s->ham);
 +            int ham_count;
 +            const uint8_t *const palette = avctx->extradata + AV_RB16(avctx->extradata);
 +
 +            s->ham_buf = av_malloc((s->planesize * 8) + FF_INPUT_BUFFER_PADDING_SIZE);
 +            if (!s->ham_buf)
 +                return AVERROR(ENOMEM);
 +
 +            ham_count = 8 * (1 << s->ham);
 +            s->ham_palbuf = av_malloc((ham_count << !!(s->masking == MASK_HAS_MASK)) * sizeof (uint32_t) + FF_INPUT_BUFFER_PADDING_SIZE);
 +            if (!s->ham_palbuf) {
 +                av_freep(&s->ham_buf);
 +                return AVERROR(ENOMEM);
 +            }
 +
 +            if (count) { // HAM with color palette attached
 +                // prefill with black and palette and set HAM take direct value mask to zero
 +                memset(s->ham_palbuf, 0, (1 << s->ham) * 2 * sizeof (uint32_t));
 +                for (i=0; i < count; i++) {
 +                    s->ham_palbuf[i*2+1] = 0xFF000000 | AV_RL24(palette + i*3);
 +                }
 +                count = 1 << s->ham;
 +            } else { // HAM with grayscale color palette
 +                count = 1 << s->ham;
 +                for (i=0; i < count; i++) {
 +                    s->ham_palbuf[i*2]   = 0xFF000000; // take direct color value from palette
 +                    s->ham_palbuf[i*2+1] = 0xFF000000 | av_le2ne32(gray2rgb((i * 255) >> s->ham));
 +                }
 +            }
 +            for (i=0; i < count; i++) {
 +                uint32_t tmp = i << (8 - s->ham);
 +                tmp |= tmp >> s->ham;
 +                s->ham_palbuf[(i+count)*2]     = 0xFF00FFFF; // just modify blue color component
 +                s->ham_palbuf[(i+count*2)*2]   = 0xFFFFFF00; // just modify red color component
 +                s->ham_palbuf[(i+count*3)*2]   = 0xFFFF00FF; // just modify green color component
 +                s->ham_palbuf[(i+count)*2+1]   = 0xFF000000 | tmp << 16;
 +                s->ham_palbuf[(i+count*2)*2+1] = 0xFF000000 | tmp;
 +                s->ham_palbuf[(i+count*3)*2+1] = 0xFF000000 | tmp << 8;
 +            }
 +            if (s->masking == MASK_HAS_MASK) {
 +                for (i = 0; i < ham_count; i++)
 +                    s->ham_palbuf[(1 << s->bpp) + i] = s->ham_palbuf[i] | 0xFF000000;
 +            }
 +        }
 +    }
 +
      return 0;
  }
  
@@@ -326,29 -150,11 +324,29 @@@ static av_cold int decode_init(AVCodecC
      int err;
  
      if (avctx->bits_per_coded_sample <= 8) {
 -        avctx->pix_fmt = (avctx->bits_per_coded_sample < 8 ||
 -                          avctx->extradata_size) ? AV_PIX_FMT_PAL8
 -                                                 : AV_PIX_FMT_GRAY8;
 +        int palette_size;
 +
 +        if (avctx->extradata_size >= 2)
 +            palette_size = avctx->extradata_size - AV_RB16(avctx->extradata);
 +        else
 +            palette_size = 0;
 +        avctx->pix_fmt = (avctx->bits_per_coded_sample < 8) ||
 +                         (avctx->extradata_size >= 2 && palette_size) ? AV_PIX_FMT_PAL8 : AV_PIX_FMT_GRAY8;
      } else if (avctx->bits_per_coded_sample <= 32) {
-         if (avctx->codec_tag == MKTAG('R','G','B','8')) {
 -        avctx->pix_fmt = AV_PIX_FMT_BGR32;
++        if (avctx->codec_tag == MKTAG('R', 'G', 'B', '8')) {
 +            avctx->pix_fmt = AV_PIX_FMT_RGB32;
-         } else if (avctx->codec_tag == MKTAG('R','G','B','N')) {
++        } else if (avctx->codec_tag == MKTAG('R', 'G', 'B', 'N')) {
 +            avctx->pix_fmt = AV_PIX_FMT_RGB444;
-         } else if (avctx->codec_tag != MKTAG('D','E','E','P')) {
++        } else if (avctx->codec_tag != MKTAG('D', 'E', 'E', 'P')) {
 +            if (avctx->bits_per_coded_sample == 24) {
 +                avctx->pix_fmt = AV_PIX_FMT_0BGR32;
 +            } else if (avctx->bits_per_coded_sample == 32) {
 +                avctx->pix_fmt = AV_PIX_FMT_BGR32;
 +            } else {
 +                avpriv_request_sample(avctx, "unknown bits_per_coded_sample");
 +                return AVERROR_PATCHWELCOME;
 +            }
 +        }
      } else {
          return AVERROR_INVALIDDATA;
      }
@@@ -489,368 -245,113 +488,369 @@@ static int decode_byterun(uint8_t *dst
      return buf - buf_start;
  }
  
 -static int decode_frame_ilbm(AVCodecContext *avctx,
 -                             void *data, int *got_frame,
 -                             AVPacket *avpkt)
 -{
 -    IffContext *s          = avctx->priv_data;
 -    const uint8_t *buf     = avpkt->data;
 -    int buf_size           = avpkt->size;
 -    const uint8_t *buf_end = buf + buf_size;
 -    int y, plane, res;
 +#define DECODE_RGBX_COMMON(type) \
 +    if (!length) { \
 +        length = bytestream2_get_byte(gb); \
 +        if (!length) { \
 +            length = bytestream2_get_be16(gb); \
 +            if (!length) \
 +                return; \
 +        } \
 +    } \
 +    for (i = 0; i < length; i++) { \
 +        *(type *)(dst + y*linesize + x * sizeof(type)) = pixel; \
 +        x += 1; \
 +        if (x >= width) { \
 +            y += 1; \
 +            if (y >= height) \
 +                return; \
 +            x = 0; \
 +        } \
 +    }
  
 -    if ((res = ff_reget_buffer(avctx, &s->frame)) < 0)
 -        return res;
 +/**
 + * Decode RGB8 buffer
 + * @param[out] dst Destination buffer
 + * @param width Width of destination buffer (pixels)
 + * @param height Height of destination buffer (pixels)
 + * @param linesize Line size of destination buffer (bytes)
 + */
 +static void decode_rgb8(GetByteContext *gb, uint8_t *dst, int width, int height, int linesize)
 +{
 +    int x = 0, y = 0, i, length;
 +    while (bytestream2_get_bytes_left(gb) >= 4) {
 +        uint32_t pixel = 0xFF000000 | bytestream2_get_be24(gb);
 +        length = bytestream2_get_byte(gb) & 0x7F;
 +        DECODE_RGBX_COMMON(uint32_t)
 +    }
 +}
  
 -    if (!s->init && avctx->bits_per_coded_sample <= 8 &&
 -        avctx->pix_fmt != AV_PIX_FMT_GRAY8) {
 -        if ((res = cmap_read_palette(avctx, (uint32_t *)s->frame.data[1])) < 0)
 -            return res;
 +/**
 + * Decode RGBN buffer
 + * @param[out] dst Destination buffer
 + * @param width Width of destination buffer (pixels)
 + * @param height Height of destination buffer (pixels)
 + * @param linesize Line size of destination buffer (bytes)
 + */
 +static void decode_rgbn(GetByteContext *gb, uint8_t *dst, int width, int height, int linesize)
 +{
 +    int x = 0, y = 0, i, length;
 +    while (bytestream2_get_bytes_left(gb) >= 2) {
 +        uint32_t pixel = bytestream2_get_be16u(gb);
 +        length = pixel & 0x7;
 +        pixel >>= 4;
 +        DECODE_RGBX_COMMON(uint16_t)
      }
 -    s->init = 1;
 +}
  
 -    if (avctx->codec_tag == MKTAG('I', 'L', 'B', 'M')) { // interleaved
 -        if (avctx->pix_fmt == AV_PIX_FMT_PAL8 || avctx->pix_fmt == AV_PIX_FMT_GRAY8) {
 -            for (y = 0; y < avctx->height; y++) {
 -                uint8_t *row = &s->frame.data[0][y * s->frame.linesize[0]];
 -                memset(row, 0, avctx->width);
 -                for (plane = 0; plane < avctx->bits_per_coded_sample && buf < buf_end;
 -                     plane++) {
 -                    decodeplane8(row, buf, FFMIN(s->planesize, buf_end - buf), plane);
 -                    buf += s->planesize;
 +/**
 + * Decode DEEP RLE 32-bit buffer
 + * @param[out] dst Destination buffer
 + * @param[in] src Source buffer
 + * @param src_size Source buffer size (bytes)
 + * @param width Width of destination buffer (pixels)
 + * @param height Height of destination buffer (pixels)
 + * @param linesize Line size of destination buffer (bytes)
 + */
 +static void decode_deep_rle32(uint8_t *dst, const uint8_t *src, int src_size, int width, int height, int linesize)
 +{
 +    const uint8_t *src_end = src + src_size;
 +    int x = 0, y = 0, i;
 +    while (src + 5 <= src_end) {
 +        int opcode;
 +        opcode = *(int8_t *)src++;
 +        if (opcode >= 0) {
 +            int size = opcode + 1;
 +            for (i = 0; i < size; i++) {
 +                int length = FFMIN(size - i, width);
 +                memcpy(dst + y*linesize + x * 4, src, length * 4);
 +                src += length * 4;
 +                x += length;
 +                i += length;
 +                if (x >= width) {
 +                    x = 0;
 +                    y += 1;
 +                    if (y >= height)
 +                        return;
                  }
              }
 -        } else { // AV_PIX_FMT_BGR32
 -            for (y = 0; y < avctx->height; y++) {
 -                uint8_t *row = &s->frame.data[0][y * s->frame.linesize[0]];
 -                memset(row, 0, avctx->width << 2);
 -                for (plane = 0; plane < avctx->bits_per_coded_sample && buf < buf_end;
 -                     plane++) {
 -                    decodeplane32((uint32_t *)row, buf,
 -                                  FFMIN(s->planesize, buf_end - buf), plane);
 -                    buf += s->planesize;
 +        } else {
 +            int size = -opcode + 1;
 +            uint32_t pixel = AV_RN32(src);
 +            for (i = 0; i < size; i++) {
 +                *(uint32_t *)(dst + y*linesize + x * 4) = pixel;
 +                x += 1;
 +                if (x >= width) {
 +                    x = 0;
 +                    y += 1;
 +                    if (y >= height)
 +                        return;
                  }
              }
 -        }
 -    } else if (avctx->pix_fmt == AV_PIX_FMT_PAL8 || avctx->pix_fmt == AV_PIX_FMT_GRAY8) { // IFF-PBM
 -        for (y = 0; y < avctx->height; y++) {
 -            uint8_t *row = &s->frame.data[0][y * s->frame.linesize[0]];
 -            memcpy(row, buf, FFMIN(avctx->width, buf_end - buf));
 -            buf += avctx->width + (avctx->width % 2); // padding if odd
 +            src += 4;
          }
      }
 +}
  
 -    if ((res = av_frame_ref(data, &s->frame)) < 0)
 -        return res;
 -
 -    *got_frame = 1;
 +/**
 + * Decode DEEP TVDC 32-bit buffer
 + * @param[out] dst Destination buffer
 + * @param[in] src Source buffer
 + * @param src_size Source buffer size (bytes)
 + * @param width Width of destination buffer (pixels)
 + * @param height Height of destination buffer (pixels)
 + * @param linesize Line size of destination buffer (bytes)
 + * @param[int] tvdc TVDC lookup table
 + */
 +static void decode_deep_tvdc32(uint8_t *dst, const uint8_t *src, int src_size, int width, int height, int linesize, const int16_t *tvdc)
 +{
 +    int x = 0, y = 0, plane = 0;
 +    int8_t pixel = 0;
 +    int i, j;
 +
 +    for (i = 0; i < src_size * 2;) {
 +#define GETNIBBLE ((i & 1) ?  (src[i>>1] & 0xF) : (src[i>>1] >> 4))
 +        int d = tvdc[GETNIBBLE];
 +        i++;
 +        if (d) {
 +            pixel += d;
 +            dst[y * linesize + x*4 + plane] = pixel;
 +            x++;
 +        } else {
 +            if (i >= src_size * 2)
 +                return;
 +            d = GETNIBBLE + 1;
 +            i++;
 +            d = FFMIN(d, width - x);
 +            for (j = 0; j < d; j++) {
 +                dst[y * linesize + x*4 + plane] = pixel;
 +                x++;
 +            }
 +        }
 +        if (x >= width) {
 +            plane++;
 +            if (plane >= 4) {
 +                y++;
 +                if (y >= height)
 +                    return;
 +                plane = 0;
 +            }
 +            x = 0;
 +            pixel = 0;
 +            i = (i + 1) & ~1;
 +        }
 +    }
 +}
  
 -    return buf_size;
 +static int unsupported(AVCodecContext *avctx)
 +{
 +    IffContext *s = avctx->priv_data;
 +    avpriv_request_sample(avctx, "bitmap (compression %i, bpp %i, ham %i)", s->compression, s->bpp, s->ham);
 +    return AVERROR_INVALIDDATA;
  }
  
 -static int decode_frame_byterun1(AVCodecContext *avctx,
 -                                 void *data, int *got_frame,
 -                                 AVPacket *avpkt)
 +static int decode_frame(AVCodecContext *avctx,
-                             void *data, int *got_frame,
-                             AVPacket *avpkt)
++                        void *data, int *got_frame,
++                        AVPacket *avpkt)
  {
-     IffContext *s = avctx->priv_data;
-     const uint8_t *buf = avpkt->size >= 2 ? avpkt->data + AV_RB16(avpkt->data) : NULL;
-     const int buf_size = avpkt->size >= 2 ? avpkt->size - AV_RB16(avpkt->data) : 0;
-     const uint8_t *buf_end = buf+buf_size;
+     IffContext *s          = avctx->priv_data;
 -    const uint8_t *buf     = avpkt->data;
 -    int buf_size           = avpkt->size;
++    const uint8_t *buf     = avpkt->size >= 2 ? avpkt->data + AV_RB16(avpkt->data) : NULL;
++    const int buf_size     = avpkt->size >= 2 ? avpkt->size - AV_RB16(avpkt->data) : 0;
+     const uint8_t *buf_end = buf + buf_size;
      int y, plane, res;
 +    GetByteContext gb;
  
 -    if ((res = ff_reget_buffer(avctx, &s->frame)) < 0)
 +    if ((res = extract_header(avctx, avpkt)) < 0)
 +        return res;
 +    if ((res = ff_reget_buffer(avctx, s->frame)) < 0)
          return res;
 -
      if (!s->init && avctx->bits_per_coded_sample <= 8 &&
 -        avctx->pix_fmt != AV_PIX_FMT_GRAY8) {
 -        if ((res = cmap_read_palette(avctx, (uint32_t *)s->frame.data[1])) < 0)
 +        avctx->pix_fmt == AV_PIX_FMT_PAL8) {
-         if ((res = cmap_read_palette(avctx, (uint32_t*)s->frame->data[1])) < 0)
++        if ((res = cmap_read_palette(avctx, (uint32_t *)s->frame->data[1])) < 0)
 +            return res;
 +    } else if (!s->init && avctx->bits_per_coded_sample <= 8 &&
 +               avctx->pix_fmt == AV_PIX_FMT_RGB32) {
 +        if ((res = cmap_read_palette(avctx, s->mask_palbuf)) < 0)
              return res;
      }
      s->init = 1;
  
 -    if (avctx->codec_tag == MKTAG('I', 'L', 'B', 'M')) { // interleaved
 -        if (avctx->pix_fmt == AV_PIX_FMT_PAL8 || avctx->pix_fmt == AV_PIX_FMT_GRAY8) {
 -            for (y = 0; y < avctx->height; y++) {
 -                uint8_t *row = &s->frame.data[0][y * s->frame.linesize[0]];
 -                memset(row, 0, avctx->width);
 -                for (plane = 0; plane < avctx->bits_per_coded_sample; plane++) {
 -                    buf += decode_byterun(s->planebuf, s->planesize, buf, buf_end);
 -                    decodeplane8(row, s->planebuf, s->planesize, plane);
 +    switch (s->compression) {
 +    case 0:
-         if (avctx->codec_tag == MKTAG('A','C','B','M')) {
++        if (avctx->codec_tag == MKTAG('A', 'C', 'B', 'M')) {
 +            if (avctx->pix_fmt == AV_PIX_FMT_PAL8 || avctx->pix_fmt == AV_PIX_FMT_GRAY8) {
 +                memset(s->frame->data[0], 0, avctx->height * s->frame->linesize[0]);
 +                for (plane = 0; plane < s->bpp; plane++) {
-                     for(y = 0; y < avctx->height && buf < buf_end; y++ ) {
-                         uint8_t *row = &s->frame->data[0][ y*s->frame->linesize[0] ];
++                    for (y = 0; y < avctx->height && buf < buf_end; y++) {
++                        uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                        decodeplane8(row, buf, FFMIN(s->planesize, buf_end - buf), plane);
 +                        buf += s->planesize;
 +                    }
 +                }
 +            } else if (s->ham) { // HAM to AV_PIX_FMT_BGR32
 +                memset(s->frame->data[0], 0, avctx->height * s->frame->linesize[0]);
-                 for(y = 0; y < avctx->height; y++) {
++                for (y = 0; y < avctx->height; y++) {
 +                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memset(s->ham_buf, 0, s->planesize * 8);
 +                    for (plane = 0; plane < s->bpp; plane++) {
 +                        const uint8_t * start = buf + (plane * avctx->height + y) * s->planesize;
 +                        if (start >= buf_end)
 +                            break;
 +                        decodeplane8(s->ham_buf, start, FFMIN(s->planesize, buf_end - start), plane);
 +                    }
-                     decode_ham_plane32((uint32_t *) row, s->ham_buf, s->ham_palbuf, s->planesize);
++                    decode_ham_plane32((uint32_t *)row, s->ham_buf, s->ham_palbuf, s->planesize);
 +                }
 +            } else
 +                return unsupported(avctx);
-         } else if (avctx->codec_tag == MKTAG('D','E','E','P')) {
++        } else if (avctx->codec_tag == MKTAG('D', 'E', 'E', 'P')) {
 +            const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(avctx->pix_fmt);
 +            int raw_width = avctx->width * (av_get_bits_per_pixel(desc) >> 3);
 +            int x;
-             for(y = 0; y < avctx->height && buf < buf_end; y++ ) {
++            for (y = 0; y < avctx->height && buf < buf_end; y++) {
 +                uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                memcpy(row, buf, FFMIN(raw_width, buf_end - buf));
 +                buf += raw_width;
 +                if (avctx->pix_fmt == AV_PIX_FMT_BGR32) {
-                     for(x = 0; x < avctx->width; x++)
++                    for (x = 0; x < avctx->width; x++)
 +                        row[4 * x + 3] = row[4 * x + 3] & 0xF0 | (row[4 * x + 3] >> 4);
                  }
              }
-         } else if (avctx->codec_tag == MKTAG('I','L','B','M')) { // interleaved
 -        } else { // AV_PIX_FMT_BGR32
 -            for (y = 0; y < avctx->height; y++) {
 -                uint8_t *row = &s->frame.data[0][y * s->frame.linesize[0]];
 -                memset(row, 0, avctx->width << 2);
 -                for (plane = 0; plane < avctx->bits_per_coded_sample; plane++) {
 -                    buf += decode_byterun(s->planebuf, s->planesize, buf, buf_end);
 -                    decodeplane32((uint32_t *)row, s->planebuf, s->planesize, plane);
++        } else if (avctx->codec_tag == MKTAG('I', 'L', 'B', 'M')) { // interleaved
 +            if (avctx->pix_fmt == AV_PIX_FMT_PAL8 || avctx->pix_fmt == AV_PIX_FMT_GRAY8) {
-                 for(y = 0; y < avctx->height; y++ ) {
-                     uint8_t *row = &s->frame->data[0][ y*s->frame->linesize[0] ];
++                for (y = 0; y < avctx->height; y++) {
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memset(row, 0, avctx->width);
 +                    for (plane = 0; plane < s->bpp && buf < buf_end; plane++) {
 +                        decodeplane8(row, buf, FFMIN(s->planesize, buf_end - buf), plane);
 +                        buf += s->planesize;
 +                    }
 +                }
 +            } else if (s->ham) { // HAM to AV_PIX_FMT_BGR32
 +                for (y = 0; y < avctx->height; y++) {
-                     uint8_t *row = &s->frame->data[0][ y*s->frame->linesize[0] ];
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memset(s->ham_buf, 0, s->planesize * 8);
 +                    for (plane = 0; plane < s->bpp && buf < buf_end; plane++) {
 +                        decodeplane8(s->ham_buf, buf, FFMIN(s->planesize, buf_end - buf), plane);
 +                        buf += s->planesize;
 +                    }
-                     decode_ham_plane32((uint32_t *) row, s->ham_buf, s->ham_palbuf, s->planesize);
++                    decode_ham_plane32((uint32_t *)row, s->ham_buf, s->ham_palbuf, s->planesize);
 +                }
 +            } else { // AV_PIX_FMT_BGR32
-                 for(y = 0; y < avctx->height; y++ ) {
-                     uint8_t *row = &s->frame->data[0][y*s->frame->linesize[0]];
++                for (y = 0; y < avctx->height; y++) {
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memset(row, 0, avctx->width << 2);
 +                    for (plane = 0; plane < s->bpp && buf < buf_end; plane++) {
-                         decodeplane32((uint32_t *) row, buf, FFMIN(s->planesize, buf_end - buf), plane);
++                        decodeplane32((uint32_t *)row, buf,
++                                      FFMIN(s->planesize, buf_end - buf), plane);
 +                        buf += s->planesize;
 +                    }
                  }
              }
-         } else if (avctx->codec_tag == MKTAG('P','B','M',' ')) { // IFF-PBM
++        } else if (avctx->codec_tag == MKTAG('P', 'B', 'M', ' ')) { // IFF-PBM
 +            if (avctx->pix_fmt == AV_PIX_FMT_PAL8 || avctx->pix_fmt == AV_PIX_FMT_GRAY8) {
-                 for(y = 0; y < avctx->height && buf_end > buf; y++ ) {
++                for (y = 0; y < avctx->height && buf_end > buf; y++) {
 +                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memcpy(row, buf, FFMIN(avctx->width, buf_end - buf));
 +                    buf += avctx->width + (avctx->width % 2); // padding if odd
 +                }
 +            } else if (s->ham) { // IFF-PBM: HAM to AV_PIX_FMT_BGR32
 +                for (y = 0; y < avctx->height && buf_end > buf; y++) {
-                     uint8_t *row = &s->frame->data[0][ y*s->frame->linesize[0] ];
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memcpy(s->ham_buf, buf, FFMIN(avctx->width, buf_end - buf));
 +                    buf += avctx->width + (avctx->width & 1); // padding if odd
-                     decode_ham_plane32((uint32_t *) row, s->ham_buf, s->ham_palbuf, s->planesize);
++                    decode_ham_plane32((uint32_t *)row, s->ham_buf, s->ham_palbuf, s->planesize);
 +                }
 +            } else
 +                return unsupported(avctx);
          }
 -    } else {
 -        for (y = 0; y < avctx->height; y++) {
 -            uint8_t *row = &s->frame.data[0][y * s->frame.linesize[0]];
 -            buf += decode_byterun(row, avctx->width, buf, buf_end);
 +        break;
 +    case 1:
-         if (avctx->codec_tag == MKTAG('I','L','B','M')) { //interleaved
++        if (avctx->codec_tag == MKTAG('I', 'L', 'B', 'M')) { // interleaved
 +            if (avctx->pix_fmt == AV_PIX_FMT_PAL8 || avctx->pix_fmt == AV_PIX_FMT_GRAY8) {
-                 for(y = 0; y < avctx->height ; y++ ) {
-                     uint8_t *row = &s->frame->data[0][ y*s->frame->linesize[0] ];
++                for (y = 0; y < avctx->height; y++) {
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memset(row, 0, avctx->width);
 +                    for (plane = 0; plane < s->bpp; plane++) {
 +                        buf += decode_byterun(s->planebuf, s->planesize, buf, buf_end);
 +                        decodeplane8(row, s->planebuf, s->planesize, plane);
 +                    }
 +                }
 +            } else if (avctx->bits_per_coded_sample <= 8) { //8-bit (+ mask) to AV_PIX_FMT_BGR32
-                 for (y = 0; y < avctx->height ; y++ ) {
-                     uint8_t *row = &s->frame->data[0][y*s->frame->linesize[0]];
++                for (y = 0; y < avctx->height; y++) {
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memset(s->mask_buf, 0, avctx->width * sizeof(uint32_t));
 +                    for (plane = 0; plane < s->bpp; plane++) {
 +                        buf += decode_byterun(s->planebuf, s->planesize, buf, buf_end);
 +                        decodeplane32(s->mask_buf, s->planebuf, s->planesize, plane);
 +                    }
-                     lookup_pal_indicies((uint32_t *) row, s->mask_buf, s->mask_palbuf, avctx->width);
++                    lookup_pal_indicies((uint32_t *)row, s->mask_buf, s->mask_palbuf, avctx->width);
 +                }
 +            } else if (s->ham) { // HAM to AV_PIX_FMT_BGR32
-                 for (y = 0; y < avctx->height ; y++) {
-                     uint8_t *row = &s->frame->data[0][y*s->frame->linesize[0]];
++                for (y = 0; y < avctx->height; y++) {
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memset(s->ham_buf, 0, s->planesize * 8);
 +                    for (plane = 0; plane < s->bpp; plane++) {
 +                        buf += decode_byterun(s->planebuf, s->planesize, buf, buf_end);
 +                        decodeplane8(s->ham_buf, s->planebuf, s->planesize, plane);
 +                    }
-                     decode_ham_plane32((uint32_t *) row, s->ham_buf, s->ham_palbuf, s->planesize);
++                    decode_ham_plane32((uint32_t *)row, s->ham_buf, s->ham_palbuf, s->planesize);
 +                }
-             } else { //AV_PIX_FMT_BGR32
-                 for(y = 0; y < avctx->height ; y++ ) {
-                     uint8_t *row = &s->frame->data[0][y*s->frame->linesize[0]];
++            } else { // AV_PIX_FMT_BGR32
++                for (y = 0; y < avctx->height; y++) {
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    memset(row, 0, avctx->width << 2);
 +                    for (plane = 0; plane < s->bpp; plane++) {
 +                        buf += decode_byterun(s->planebuf, s->planesize, buf, buf_end);
-                         decodeplane32((uint32_t *) row, s->planebuf, s->planesize, plane);
++                        decodeplane32((uint32_t *)row, s->planebuf, s->planesize, plane);
 +                    }
 +                }
 +            }
-         } else if (avctx->codec_tag == MKTAG('P','B','M',' ')) { // IFF-PBM
++        } else if (avctx->codec_tag == MKTAG('P', 'B', 'M', ' ')) { // IFF-PBM
 +            if (avctx->pix_fmt == AV_PIX_FMT_PAL8 || avctx->pix_fmt == AV_PIX_FMT_GRAY8) {
-                 for(y = 0; y < avctx->height ; y++ ) {
-                     uint8_t *row = &s->frame->data[0][y*s->frame->linesize[0]];
++                for (y = 0; y < avctx->height; y++) {
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    buf += decode_byterun(row, avctx->width, buf, buf_end);
 +                }
 +            } else if (s->ham) { // IFF-PBM: HAM to AV_PIX_FMT_BGR32
-                 for (y = 0; y < avctx->height ; y++) {
-                     uint8_t *row = &s->frame->data[0][y*s->frame->linesize[0]];
++                for (y = 0; y < avctx->height; y++) {
++                    uint8_t *row = &s->frame->data[0][y * s->frame->linesize[0]];
 +                    buf += decode_byterun(s->ham_buf, avctx->width, buf, buf_end);
-                     decode_ham_plane32((uint32_t *) row, s->ham_buf, s->ham_palbuf, s->planesize);
++                    decode_ham_plane32((uint32_t *)row, s->ham_buf, s->ham_palbuf, s->planesize);
 +                }
 +            } else
 +                return unsupported(avctx);
-         } else if (avctx->codec_tag == MKTAG('D','E','E','P')) { // IFF-DEEP
++        } else if (avctx->codec_tag == MKTAG('D', 'E', 'E', 'P')) { // IFF-DEEP
 +            const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(avctx->pix_fmt);
 +            if (av_get_bits_per_pixel(desc) == 32)
 +                decode_deep_rle32(s->frame->data[0], buf, buf_size, avctx->width, avctx->height, s->frame->linesize[0]);
 +            else
 +                return unsupported(avctx);
          }
-         if (avctx->codec_tag == MKTAG('R','G','B','8'))
 +        break;
 +    case 4:
 +        bytestream2_init(&gb, buf, buf_size);
-         else if (avctx->codec_tag == MKTAG('R','G','B','N'))
++        if (avctx->codec_tag == MKTAG('R', 'G', 'B', '8'))
 +            decode_rgb8(&gb, s->frame->data[0], avctx->width, avctx->height, s->frame->linesize[0]);
-         if (avctx->codec_tag == MKTAG('D','E','E','P')) {
++        else if (avctx->codec_tag == MKTAG('R', 'G', 'B', 'N'))
 +            decode_rgbn(&gb, s->frame->data[0], avctx->width, avctx->height, s->frame->linesize[0]);
 +        else
 +            return unsupported(avctx);
 +        break;
 +    case 5:
++        if (avctx->codec_tag == MKTAG('D', 'E', 'E', 'P')) {
 +            const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(avctx->pix_fmt);
 +            if (av_get_bits_per_pixel(desc) == 32)
 +                decode_deep_tvdc32(s->frame->data[0], buf, buf_size, avctx->width, avctx->height, s->frame->linesize[0], s->tvdc);
 +            else
 +                return unsupported(avctx);
 +        } else
 +            return unsupported(avctx);
 +        break;
 +    default:
 +        return unsupported(avctx);
      }
  
 -    if ((res = av_frame_ref(data, &s->frame)) < 0)
 +    if ((res = av_frame_ref(data, s->frame)) < 0)
          return res;
  
      *got_frame = 1;