From 170fb593c617cd797d00cf05bfb8d8c21d612893 Mon Sep 17 00:00:00 2001 From: Luca Barbato Date: Sat, 29 Jun 2013 06:25:31 +0200 Subject: pcx: K&R formatting cosmetics --- libavcodec/pcx.c | 116 ++++++++++++++++++++++++++++--------------------------- 1 file changed, 59 insertions(+), 57 deletions(-) (limited to 'libavcodec/pcx.c') diff --git a/libavcodec/pcx.c b/libavcodec/pcx.c index 949ea1ca70..53f8c1992a 100644 --- a/libavcodec/pcx.c +++ b/libavcodec/pcx.c @@ -32,19 +32,21 @@ * @return advanced src pointer */ static const uint8_t *pcx_rle_decode(const uint8_t *src, uint8_t *dst, - unsigned int bytes_per_scanline, int compressed) { + unsigned int bytes_per_scanline, + int compressed) +{ unsigned int i = 0; unsigned char run, value; if (compressed) { - while (i= 0xc0) { - run = value & 0x3f; + run = value & 0x3f; value = *src++; } - while (idata; - int buf_size = avpkt->size; - AVFrame * const p = data; + int buf_size = avpkt->size; + AVFrame *const p = data; int compressed, xmin, ymin, xmax, ymax; unsigned int w, h, bits_per_pixel, bytes_per_line, nplanes, stride, y, x, bytes_per_scanline; @@ -83,10 +88,10 @@ static int pcx_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, } compressed = buf[2]; - xmin = AV_RL16(buf+ 4); - ymin = AV_RL16(buf+ 6); - xmax = AV_RL16(buf+ 8); - ymax = AV_RL16(buf+10); + xmin = AV_RL16(buf + 4); + ymin = AV_RL16(buf + 6); + xmax = AV_RL16(buf + 8); + ymax = AV_RL16(buf + 10); if (xmax < xmin || ymax < ymin) { av_log(avctx, AV_LOG_ERROR, "invalid image dimensions\n"); @@ -97,7 +102,7 @@ static int pcx_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, h = ymax - ymin + 1; bits_per_pixel = buf[3]; - bytes_per_line = AV_RL16(buf+66); + bytes_per_line = AV_RL16(buf + 66); nplanes = buf[65]; bytes_per_scanline = nplanes * bytes_per_line; @@ -106,22 +111,22 @@ static int pcx_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, return AVERROR_INVALIDDATA; } - switch ((nplanes<<8) + bits_per_pixel) { - case 0x0308: - avctx->pix_fmt = AV_PIX_FMT_RGB24; - break; - case 0x0108: - case 0x0104: - case 0x0102: - case 0x0101: - case 0x0401: - case 0x0301: - case 0x0201: - avctx->pix_fmt = AV_PIX_FMT_PAL8; - break; - default: - av_log(avctx, AV_LOG_ERROR, "invalid PCX file\n"); - return AVERROR_INVALIDDATA; + switch ((nplanes << 8) + bits_per_pixel) { + case 0x0308: + avctx->pix_fmt = AV_PIX_FMT_RGB24; + break; + case 0x0108: + case 0x0104: + case 0x0102: + case 0x0101: + case 0x0401: + case 0x0301: + case 0x0201: + avctx->pix_fmt = AV_PIX_FMT_PAL8; + break; + default: + av_log(avctx, AV_LOG_ERROR, "invalid PCX file\n"); + return AVERROR_INVALIDDATA; } buf += 128; @@ -145,22 +150,21 @@ static int pcx_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, return AVERROR(ENOMEM); if (nplanes == 3 && bits_per_pixel == 8) { - for (y=0; y> (x&7), v = 0; - for (i=nplanes - 1; i>=0; i--) { + for (x = 0; x < w; x++) { + int m = 0x80 >> (x & 7), v = 0; + for (i = nplanes - 1; i >= 0; i--) { v <<= 1; - v += !!(scanline[i*bytes_per_line + (x>>3)] & m); + v += !!(scanline[i * bytes_per_line + (x >> 3)] & m); } ptr[x] = v; } @@ -206,10 +208,10 @@ static int pcx_decode_frame(AVCodecContext *avctx, void *data, int *got_frame, } if (nplanes == 1 && bits_per_pixel == 8) { - pcx_palette(&buf, (uint32_t *) p->data[1], 256); + pcx_palette(&buf, (uint32_t *)p->data[1], 256); } else if (bits_per_pixel < 8) { - const uint8_t *palette = bufstart+16; - pcx_palette(&palette, (uint32_t *) p->data[1], 16); + const uint8_t *palette = bufstart + 16; + pcx_palette(&palette, (uint32_t *)p->data[1], 16); } *got_frame = 1; @@ -221,10 +223,10 @@ end: } AVCodec ff_pcx_decoder = { - .name = "pcx", - .type = AVMEDIA_TYPE_VIDEO, - .id = AV_CODEC_ID_PCX, - .decode = pcx_decode_frame, - .capabilities = CODEC_CAP_DR1, - .long_name = NULL_IF_CONFIG_SMALL("PC Paintbrush PCX image"), + .name = "pcx", + .type = AVMEDIA_TYPE_VIDEO, + .id = AV_CODEC_ID_PCX, + .decode = pcx_decode_frame, + .capabilities = CODEC_CAP_DR1, + .long_name = NULL_IF_CONFIG_SMALL("PC Paintbrush PCX image"), }; -- cgit v1.2.3