From c065f4a0c652cc569c927c95148111b3aa2962dc Mon Sep 17 00:00:00 2001 From: Diego Biurrun Date: Sun, 3 Aug 2014 10:49:40 -0700 Subject: huffyuv: K&R formatting cosmetics --- libavcodec/huffyuvdec.c | 334 ++++++++++++++++++++++++++---------------------- 1 file changed, 178 insertions(+), 156 deletions(-) (limited to 'libavcodec/huffyuvdec.c') diff --git a/libavcodec/huffyuvdec.c b/libavcodec/huffyuvdec.c index e0d8a06093..e57b461e1e 100644 --- a/libavcodec/huffyuvdec.c +++ b/libavcodec/huffyuvdec.c @@ -36,54 +36,55 @@ #define classic_shift_luma_table_size 42 static const unsigned char classic_shift_luma[classic_shift_luma_table_size + FF_INPUT_BUFFER_PADDING_SIZE] = { - 34,36,35,69,135,232,9,16,10,24,11,23,12,16,13,10,14,8,15,8, - 16,8,17,20,16,10,207,206,205,236,11,8,10,21,9,23,8,8,199,70, - 69,68, 0 + 34, 36, 35, 69, 135, 232, 9, 16, 10, 24, 11, 23, 12, 16, 13, 10, + 14, 8, 15, 8, 16, 8, 17, 20, 16, 10, 207, 206, 205, 236, 11, 8, + 10, 21, 9, 23, 8, 8, 199, 70, 69, 68, 0 }; #define classic_shift_chroma_table_size 59 static const unsigned char classic_shift_chroma[classic_shift_chroma_table_size + FF_INPUT_BUFFER_PADDING_SIZE] = { - 66,36,37,38,39,40,41,75,76,77,110,239,144,81,82,83,84,85,118,183, - 56,57,88,89,56,89,154,57,58,57,26,141,57,56,58,57,58,57,184,119, - 214,245,116,83,82,49,80,79,78,77,44,75,41,40,39,38,37,36,34, 0 + 66, 36, 37, 38, 39, 40, 41, 75, 76, 77, 110, 239, 144, 81, 82, 83, + 84, 85, 118, 183, 56, 57, 88, 89, 56, 89, 154, 57, 58, 57, 26, 141, + 57, 56, 58, 57, 58, 57, 184, 119, 214, 245, 116, 83, 82, 49, 80, 79, + 78, 77, 44, 75, 41, 40, 39, 38, 37, 36, 34, 0 }; static const unsigned char classic_add_luma[256] = { - 3, 9, 5, 12, 10, 35, 32, 29, 27, 50, 48, 45, 44, 41, 39, 37, - 73, 70, 68, 65, 64, 61, 58, 56, 53, 50, 49, 46, 44, 41, 38, 36, - 68, 65, 63, 61, 58, 55, 53, 51, 48, 46, 45, 43, 41, 39, 38, 36, - 35, 33, 32, 30, 29, 27, 26, 25, 48, 47, 46, 44, 43, 41, 40, 39, - 37, 36, 35, 34, 32, 31, 30, 28, 27, 26, 24, 23, 22, 20, 19, 37, - 35, 34, 33, 31, 30, 29, 27, 26, 24, 23, 21, 20, 18, 17, 15, 29, - 27, 26, 24, 22, 21, 19, 17, 16, 14, 26, 25, 23, 21, 19, 18, 16, - 15, 27, 25, 23, 21, 19, 17, 16, 14, 26, 25, 23, 21, 18, 17, 14, - 12, 17, 19, 13, 4, 9, 2, 11, 1, 7, 8, 0, 16, 3, 14, 6, - 12, 10, 5, 15, 18, 11, 10, 13, 15, 16, 19, 20, 22, 24, 27, 15, - 18, 20, 22, 24, 26, 14, 17, 20, 22, 24, 27, 15, 18, 20, 23, 25, - 28, 16, 19, 22, 25, 28, 32, 36, 21, 25, 29, 33, 38, 42, 45, 49, - 28, 31, 34, 37, 40, 42, 44, 47, 49, 50, 52, 54, 56, 57, 59, 60, - 62, 64, 66, 67, 69, 35, 37, 39, 40, 42, 43, 45, 47, 48, 51, 52, - 54, 55, 57, 59, 60, 62, 63, 66, 67, 69, 71, 72, 38, 40, 42, 43, - 46, 47, 49, 51, 26, 28, 30, 31, 33, 34, 18, 19, 11, 13, 7, 8, + 3, 9, 5, 12, 10, 35, 32, 29, 27, 50, 48, 45, 44, 41, 39, 37, + 73, 70, 68, 65, 64, 61, 58, 56, 53, 50, 49, 46, 44, 41, 38, 36, + 68, 65, 63, 61, 58, 55, 53, 51, 48, 46, 45, 43, 41, 39, 38, 36, + 35, 33, 32, 30, 29, 27, 26, 25, 48, 47, 46, 44, 43, 41, 40, 39, + 37, 36, 35, 34, 32, 31, 30, 28, 27, 26, 24, 23, 22, 20, 19, 37, + 35, 34, 33, 31, 30, 29, 27, 26, 24, 23, 21, 20, 18, 17, 15, 29, + 27, 26, 24, 22, 21, 19, 17, 16, 14, 26, 25, 23, 21, 19, 18, 16, + 15, 27, 25, 23, 21, 19, 17, 16, 14, 26, 25, 23, 21, 18, 17, 14, + 12, 17, 19, 13, 4, 9, 2, 11, 1, 7, 8, 0, 16, 3, 14, 6, + 12, 10, 5, 15, 18, 11, 10, 13, 15, 16, 19, 20, 22, 24, 27, 15, + 18, 20, 22, 24, 26, 14, 17, 20, 22, 24, 27, 15, 18, 20, 23, 25, + 28, 16, 19, 22, 25, 28, 32, 36, 21, 25, 29, 33, 38, 42, 45, 49, + 28, 31, 34, 37, 40, 42, 44, 47, 49, 50, 52, 54, 56, 57, 59, 60, + 62, 64, 66, 67, 69, 35, 37, 39, 40, 42, 43, 45, 47, 48, 51, 52, + 54, 55, 57, 59, 60, 62, 63, 66, 67, 69, 71, 72, 38, 40, 42, 43, + 46, 47, 49, 51, 26, 28, 30, 31, 33, 34, 18, 19, 11, 13, 7, 8, }; static const unsigned char classic_add_chroma[256] = { - 3, 1, 2, 2, 2, 2, 3, 3, 7, 5, 7, 5, 8, 6, 11, 9, - 7, 13, 11, 10, 9, 8, 7, 5, 9, 7, 6, 4, 7, 5, 8, 7, - 11, 8, 13, 11, 19, 15, 22, 23, 20, 33, 32, 28, 27, 29, 51, 77, - 43, 45, 76, 81, 46, 82, 75, 55, 56,144, 58, 80, 60, 74,147, 63, - 143, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, - 80, 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 27, 30, 21, 22, - 17, 14, 5, 6,100, 54, 47, 50, 51, 53,106,107,108,109,110,111, - 112,113,114,115, 4,117,118, 92, 94,121,122, 3,124,103, 2, 1, - 0,129,130,131,120,119,126,125,136,137,138,139,140,141,142,134, - 135,132,133,104, 64,101, 62, 57,102, 95, 93, 59, 61, 28, 97, 96, - 52, 49, 48, 29, 32, 25, 24, 46, 23, 98, 45, 44, 43, 20, 42, 41, - 19, 18, 99, 40, 15, 39, 38, 16, 13, 12, 11, 37, 10, 9, 8, 36, - 7,128,127,105,123,116, 35, 34, 33,145, 31, 79, 42,146, 78, 26, - 83, 48, 49, 50, 44, 47, 26, 31, 30, 18, 17, 19, 21, 24, 25, 13, - 14, 16, 17, 18, 20, 21, 12, 14, 15, 9, 10, 6, 9, 6, 5, 8, - 6, 12, 8, 10, 7, 9, 6, 4, 6, 2, 2, 3, 3, 3, 3, 2, + 3, 1, 2, 2, 2, 2, 3, 3, 7, 5, 7, 5, 8, 6, 11, 9, + 7, 13, 11, 10, 9, 8, 7, 5, 9, 7, 6, 4, 7, 5, 8, 7, + 11, 8, 13, 11, 19, 15, 22, 23, 20, 33, 32, 28, 27, 29, 51, 77, + 43, 45, 76, 81, 46, 82, 75, 55, 56, 144, 58, 80, 60, 74, 147, 63, + 143, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, + 80, 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 27, 30, 21, 22, + 17, 14, 5, 6, 100, 54, 47, 50, 51, 53, 106, 107, 108, 109, 110, 111, + 112, 113, 114, 115, 4, 117, 118, 92, 94, 121, 122, 3, 124, 103, 2, 1, + 0, 129, 130, 131, 120, 119, 126, 125, 136, 137, 138, 139, 140, 141, 142, 134, + 135, 132, 133, 104, 64, 101, 62, 57, 102, 95, 93, 59, 61, 28, 97, 96, + 52, 49, 48, 29, 32, 25, 24, 46, 23, 98, 45, 44, 43, 20, 42, 41, + 19, 18, 99, 40, 15, 39, 38, 16, 13, 12, 11, 37, 10, 9, 8, 36, + 7, 128, 127, 105, 123, 116, 35, 34, 33, 145, 31, 79, 42, 146, 78, 26, + 83, 48, 49, 50, 44, 47, 26, 31, 30, 18, 17, 19, 21, 24, 25, 13, + 14, 16, 17, 18, 20, 21, 12, 14, 15, 9, 10, 6, 9, 6, 5, 8, + 6, 12, 8, 10, 7, 9, 6, 4, 6, 2, 2, 3, 3, 3, 3, 2, }; static int read_len_table(uint8_t *dst, GetBitContext *gb) @@ -114,18 +115,18 @@ static void generate_joint_tables(HYuvContext *s) int p, i, y, u; for (p = 0; p < 3; p++) { for (i = y = 0; y < 256; y++) { - int len0 = s->len[0][y]; + int len0 = s->len[0][y]; int limit = VLC_BITS - len0; - if(limit <= 0) + if (limit <= 0) continue; for (u = 0; u < 256; u++) { int len1 = s->len[p][u]; if (len1 > limit) continue; - len[i] = len0 + len1; - bits[i] = (s->bits[0][y] << len1) + s->bits[p][u]; + len[i] = len0 + len1; + bits[i] = (s->bits[0][y] << len1) + s->bits[p][u]; symbols[i] = (y << 8) + u; - if(symbols[i] != 0xffff) // reserved to mean "invalid" + if (symbols[i] != 0xffff) // reserved to mean "invalid" i++; } } @@ -134,20 +135,20 @@ static void generate_joint_tables(HYuvContext *s) bits, 2, 2, symbols, 2, 2, 0); } } else { - uint8_t (*map)[4] = (uint8_t(*)[4])s->pix_bgr_map; + uint8_t (*map)[4] = (uint8_t(*)[4]) s->pix_bgr_map; int i, b, g, r, code; int p0 = s->decorrelate; int p1 = !s->decorrelate; - // restrict the range to +/-16 because that's pretty much guaranteed to - // cover all the combinations that fit in 11 bits total, and it doesn't - // matter if we miss a few rare codes. + /* Restrict the range to +/-16 because that's pretty much guaranteed + * to cover all the combinations that fit in 11 bits total, and it + * does not matter if we miss a few rare codes. */ for (i = 0, g = -16; g < 16; g++) { - int len0 = s->len[p0][g & 255]; + int len0 = s->len[p0][g & 255]; int limit0 = VLC_BITS - len0; if (limit0 < 2) continue; for (b = -16; b < 16; b++) { - int len1 = s->len[p1][b & 255]; + int len1 = s->len[p1][b & 255]; int limit1 = limit0 - len1; if (limit1 < 1) continue; @@ -156,7 +157,7 @@ static void generate_joint_tables(HYuvContext *s) int len2 = s->len[2][r & 255]; if (len2 > limit1) continue; - len[i] = len0 + len1 + len2; + len[i] = len0 + len1 + len2; bits[i] = (code << len2) + s->bits[2][r & 255]; if (s->decorrelate) { map[i][G] = g; @@ -186,9 +187,8 @@ static int read_huffman_tables(HYuvContext *s, const uint8_t *src, int length) for (i = 0; i < 3; i++) { if (read_len_table(s->len[i], &gb) < 0) return -1; - if (ff_huffyuv_generate_bits_table(s->bits[i], s->len[i]) < 0) { + if (ff_huffyuv_generate_bits_table(s->bits[i], s->len[i]) < 0) return -1; - } ff_free_vlc(&s->vlc[i]); init_vlc(&s->vlc[i], VLC_BITS, 256, s->len[i], 1, 1, s->bits[i], 4, 4, 0); @@ -214,15 +214,17 @@ static int read_old_huffman_tables(HYuvContext *s) if (read_len_table(s->len[1], &gb) < 0) return -1; - for(i=0; i<256; i++) s->bits[0][i] = classic_add_luma [i]; - for(i=0; i<256; i++) s->bits[1][i] = classic_add_chroma[i]; + for (i = 0; i < 256; i++) + s->bits[0][i] = classic_add_luma[i]; + for (i = 0; i < 256; i++) + s->bits[1][i] = classic_add_chroma[i]; if (s->bitstream_bpp >= 24) { memcpy(s->bits[1], s->bits[0], 256 * sizeof(uint32_t)); - memcpy(s->len[1] , s->len [0], 256 * sizeof(uint8_t)); + memcpy(s->len[1], s->len[0], 256 * sizeof(uint8_t)); } memcpy(s->bits[2], s->bits[1], 256 * sizeof(uint32_t)); - memcpy(s->len[2] , s->len [1], 256 * sizeof(uint8_t)); + memcpy(s->len[2], s->len[1], 256 * sizeof(uint8_t)); for (i = 0; i < 3; i++) { ff_free_vlc(&s->vlc[i]); @@ -244,8 +246,7 @@ static av_cold int decode_init(AVCodecContext *avctx) memset(s->vlc, 0, 3 * sizeof(VLC)); s->interlaced = s->height > 288; - - s->bgr32 = 1; + s->bgr32 = 1; if (avctx->extradata_size) { if ((avctx->bits_per_coded_sample & 7) && @@ -262,44 +263,44 @@ static av_cold int decode_init(AVCodecContext *avctx) if (avctx->extradata_size < 4) return -1; - method = ((uint8_t*)avctx->extradata)[0]; - s->decorrelate = method & 64 ? 1 : 0; - s->predictor = method & 63; - s->bitstream_bpp = ((uint8_t*)avctx->extradata)[1]; + method = ((uint8_t *) avctx->extradata)[0]; + s->decorrelate = method & 64 ? 1 : 0; + s->predictor = method & 63; + s->bitstream_bpp = ((uint8_t *) avctx->extradata)[1]; if (s->bitstream_bpp == 0) s->bitstream_bpp = avctx->bits_per_coded_sample & ~7; - interlace = (((uint8_t*)avctx->extradata)[2] & 0x30) >> 4; + interlace = (((uint8_t *) avctx->extradata)[2] & 0x30) >> 4; s->interlaced = (interlace == 1) ? 1 : (interlace == 2) ? 0 : s->interlaced; - s->context = ((uint8_t*)avctx->extradata)[2] & 0x40 ? 1 : 0; + s->context = ((uint8_t *) avctx->extradata)[2] & 0x40 ? 1 : 0; - if ( read_huffman_tables(s, ((uint8_t*)avctx->extradata) + 4, - avctx->extradata_size - 4) < 0) + if (read_huffman_tables(s, ((uint8_t *) avctx->extradata) + 4, + avctx->extradata_size - 4) < 0) return -1; - }else{ + } else { switch (avctx->bits_per_coded_sample & 7) { case 1: - s->predictor = LEFT; + s->predictor = LEFT; s->decorrelate = 0; break; case 2: - s->predictor = LEFT; + s->predictor = LEFT; s->decorrelate = 1; break; case 3: - s->predictor = PLANE; + s->predictor = PLANE; s->decorrelate = avctx->bits_per_coded_sample >= 24; break; case 4: - s->predictor = MEDIAN; + s->predictor = MEDIAN; s->decorrelate = 0; break; default: - s->predictor = LEFT; //OLD + s->predictor = LEFT; // OLD s->decorrelate = 0; break; } s->bitstream_bpp = avctx->bits_per_coded_sample & ~7; - s->context = 0; + s->context = 0; if (read_old_huffman_tables(s) < 0) return -1; @@ -310,19 +311,17 @@ static av_cold int decode_init(AVCodecContext *avctx) avctx->pix_fmt = AV_PIX_FMT_YUV420P; break; case 16: - if (s->yuy2) { + if (s->yuy2) avctx->pix_fmt = AV_PIX_FMT_YUYV422; - } else { + else avctx->pix_fmt = AV_PIX_FMT_YUV422P; - } break; case 24: case 32: - if (s->bgr32) { + if (s->bgr32) avctx->pix_fmt = AV_PIX_FMT_RGB32; - } else { + else avctx->pix_fmt = AV_PIX_FMT_BGR24; - } break; default: return AVERROR_INVALIDDATA; @@ -344,7 +343,7 @@ static av_cold int decode_init_thread_copy(AVCodecContext *avctx) s->vlc[i].table = NULL; if (s->version == 2) { - if (read_huffman_tables(s, ((uint8_t*)avctx->extradata) + 4, + if (read_huffman_tables(s, ((uint8_t *) avctx->extradata) + 4, avctx->extradata_size) < 0) return -1; } else { @@ -357,16 +356,18 @@ static av_cold int decode_init_thread_copy(AVCodecContext *avctx) /* TODO instead of restarting the read when the code isn't in the first level * of the joint table, jump into the 2nd level of the individual table. */ -#define READ_2PIX(dst0, dst1, plane1){\ - uint16_t code = get_vlc2(&s->gb, s->vlc[3+plane1].table, VLC_BITS, 1);\ - if(code != 0xffff){\ - dst0 = code>>8;\ - dst1 = code;\ - }else{\ - dst0 = get_vlc2(&s->gb, s->vlc[0].table, VLC_BITS, 3);\ - dst1 = get_vlc2(&s->gb, s->vlc[plane1].table, VLC_BITS, 3);\ - }\ -} +#define READ_2PIX(dst0, dst1, plane1) \ + { \ + uint16_t code = get_vlc2(&s->gb, s->vlc[3 + plane1].table, \ + VLC_BITS, 1); \ + if (code != 0xffff) { \ + dst0 = code >> 8; \ + dst1 = code; \ + } else { \ + dst0 = get_vlc2(&s->gb, s->vlc[0].table, VLC_BITS, 3); \ + dst1 = get_vlc2(&s->gb, s->vlc[plane1].table, VLC_BITS, 3); \ + } \ + } static void decode_422_bitstream(HYuvContext *s, int count) { @@ -376,12 +377,12 @@ static void decode_422_bitstream(HYuvContext *s, int count) if (count >= (get_bits_left(&s->gb)) / (31 * 4)) { for (i = 0; i < count && get_bits_left(&s->gb) > 0; i++) { - READ_2PIX(s->temp[0][2 * i ], s->temp[1][i], 1); + READ_2PIX(s->temp[0][2 * i], s->temp[1][i], 1); READ_2PIX(s->temp[0][2 * i + 1], s->temp[2][i], 2); } } else { for (i = 0; i < count; i++) { - READ_2PIX(s->temp[0][2 * i ], s->temp[1][i], 1); + READ_2PIX(s->temp[0][2 * i], s->temp[1][i], 1); READ_2PIX(s->temp[0][2 * i + 1], s->temp[2][i], 2); } } @@ -391,14 +392,14 @@ static void decode_gray_bitstream(HYuvContext *s, int count) { int i; - count/=2; + count /= 2; if (count >= (get_bits_left(&s->gb)) / (31 * 2)) { for (i = 0; i < count && get_bits_left(&s->gb) > 0; i++) { READ_2PIX(s->temp[0][2 * i], s->temp[0][2 * i + 1], 0); } } else { - for(i=0; itemp[0][2 * i], s->temp[0][2 * i + 1], 0); } } @@ -411,8 +412,8 @@ static av_always_inline void decode_bgr_1(HYuvContext *s, int count, for (i = 0; i < count; i++) { int code = get_vlc2(&s->gb, s->vlc[3].table, VLC_BITS, 1); if (code != -1) { - *(uint32_t*)&s->temp[0][4 * i] = s->pix_bgr_map[code]; - } else if(decorrelate) { + *(uint32_t *) &s->temp[0][4 * i] = s->pix_bgr_map[code]; + } else if (decorrelate) { s->temp[0][4 * i + G] = get_vlc2(&s->gb, s->vlc[1].table, VLC_BITS, 3); s->temp[0][4 * i + B] = get_vlc2(&s->gb, s->vlc[0].table, VLC_BITS, 3) + s->temp[0][4 * i + G]; @@ -431,12 +432,12 @@ static av_always_inline void decode_bgr_1(HYuvContext *s, int count, static void decode_bgr_bitstream(HYuvContext *s, int count) { if (s->decorrelate) { - if (s->bitstream_bpp==24) + if (s->bitstream_bpp == 24) decode_bgr_1(s, count, 1, 0); else decode_bgr_1(s, count, 1, 1); } else { - if (s->bitstream_bpp==24) + if (s->bitstream_bpp == 24) decode_bgr_1(s, count, 0, 0); else decode_bgr_1(s, count, 0, 1); @@ -448,17 +449,16 @@ static void draw_slice(HYuvContext *s, AVFrame *frame, int y) int h, cy, i; int offset[AV_NUM_DATA_POINTERS]; - if (s->avctx->draw_horiz_band==NULL) + if (s->avctx->draw_horiz_band == NULL) return; - h = y - s->last_slice_end; + h = y - s->last_slice_end; y -= h; - if (s->bitstream_bpp == 12) { - cy = y>>1; - } else { + if (s->bitstream_bpp == 12) + cy = y >> 1; + else cy = y; - } offset[0] = frame->linesize[0] * y; offset[1] = frame->linesize[1] * cy; @@ -476,14 +476,14 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, AVPacket *avpkt) { const uint8_t *buf = avpkt->data; - int buf_size = avpkt->size; + int buf_size = avpkt->size; HYuvContext *s = avctx->priv_data; - const int width = s->width; - const int width2 = s->width>>1; + const int width = s->width; + const int width2 = s->width >> 1; const int height = s->height; int fake_ystride, fake_ustride, fake_vstride; ThreadFrame frame = { .f = data }; - AVFrame * const p = data; + AVFrame *const p = data; int table_size = 0; av_fast_malloc(&s->bitstream_buffer, @@ -507,15 +507,15 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, return -1; } - if ((unsigned)(buf_size-table_size) >= INT_MAX / 8) + if ((unsigned) (buf_size - table_size) >= INT_MAX / 8) return -1; - init_get_bits(&s->gb, s->bitstream_buffer+table_size, - (buf_size-table_size) * 8); + init_get_bits(&s->gb, s->bitstream_buffer + table_size, + (buf_size - table_size) * 8); - fake_ystride = s->interlaced ? p->linesize[0] * 2 : p->linesize[0]; - fake_ustride = s->interlaced ? p->linesize[1] * 2 : p->linesize[1]; - fake_vstride = s->interlaced ? p->linesize[2] * 2 : p->linesize[2]; + fake_ystride = s->interlaced ? p->linesize[0] * 2 : p->linesize[0]; + fake_ustride = s->interlaced ? p->linesize[1] * 2 : p->linesize[1]; + fake_vstride = s->interlaced ? p->linesize[2] * 2 : p->linesize[2]; s->last_slice_end = 0; @@ -534,18 +534,21 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, "YUY2 output is not implemented yet\n"); return -1; } else { - - leftv = p->data[2][0] = get_bits(&s->gb, 8); - lefty = p->data[0][1] = get_bits(&s->gb, 8); - leftu = p->data[1][0] = get_bits(&s->gb, 8); - p->data[0][0] = get_bits(&s->gb, 8); + leftv = + p->data[2][0] = get_bits(&s->gb, 8); + lefty = + p->data[0][1] = get_bits(&s->gb, 8); + leftu = + p->data[1][0] = get_bits(&s->gb, 8); + p->data[0][0] = get_bits(&s->gb, 8); switch (s->predictor) { case LEFT: case PLANE: - decode_422_bitstream(s, width-2); - lefty = s->hdsp.add_hfyu_left_pred(p->data[0] + 2, s->temp[0], width - 2, lefty); - if (!(s->flags&CODEC_FLAG_GRAY)) { + decode_422_bitstream(s, width - 2); + lefty = s->hdsp.add_hfyu_left_pred(p->data[0] + 2, s->temp[0], + width - 2, lefty); + if (!(s->flags & CODEC_FLAG_GRAY)) { leftu = s->hdsp.add_hfyu_left_pred(p->data[1] + 1, s->temp[1], width2 - 1, leftu); leftv = s->hdsp.add_hfyu_left_pred(p->data[2] + 1, s->temp[2], width2 - 1, leftv); } @@ -558,23 +561,26 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, ydst = p->data[0] + p->linesize[0] * y; - lefty = s->hdsp.add_hfyu_left_pred(ydst, s->temp[0], width, lefty); + lefty = s->hdsp.add_hfyu_left_pred(ydst, s->temp[0], + width, lefty); if (s->predictor == PLANE) { if (y > s->interlaced) s->hdsp.add_bytes(ydst, ydst - fake_ystride, width); } y++; - if (y >= s->height) break; + if (y >= s->height) + break; } draw_slice(s, p, y); - ydst = p->data[0] + p->linesize[0]*y; - udst = p->data[1] + p->linesize[1]*cy; - vdst = p->data[2] + p->linesize[2]*cy; + ydst = p->data[0] + p->linesize[0] * y; + udst = p->data[1] + p->linesize[1] * cy; + vdst = p->data[2] + p->linesize[2] * cy; decode_422_bitstream(s, width); - lefty = s->hdsp.add_hfyu_left_pred(ydst, s->temp[0], width, lefty); + lefty = s->hdsp.add_hfyu_left_pred(ydst, s->temp[0], + width, lefty); if (!(s->flags & CODEC_FLAG_GRAY)) { leftu = s->hdsp.add_hfyu_left_pred(udst, s->temp[1], width2, leftu); leftv = s->hdsp.add_hfyu_left_pred(vdst, s->temp[2], width2, leftv); @@ -595,7 +601,8 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, case MEDIAN: /* first line except first 2 pixels is left predicted */ decode_422_bitstream(s, width - 2); - lefty = s->hdsp.add_hfyu_left_pred(p->data[0] + 2, s->temp[0], width - 2, lefty); + lefty = s->hdsp.add_hfyu_left_pred(p->data[0] + 2, s->temp[0], + width - 2, lefty); if (!(s->flags & CODEC_FLAG_GRAY)) { leftu = s->hdsp.add_hfyu_left_pred(p->data[1] + 1, s->temp[1], width2 - 1, leftu); leftv = s->hdsp.add_hfyu_left_pred(p->data[2] + 1, s->temp[2], width2 - 1, leftv); @@ -606,18 +613,21 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, /* second line is left predicted for interlaced case */ if (s->interlaced) { decode_422_bitstream(s, width); - lefty = s->hdsp.add_hfyu_left_pred(p->data[0] + p->linesize[0], s->temp[0], width, lefty); + lefty = s->hdsp.add_hfyu_left_pred(p->data[0] + p->linesize[0], + s->temp[0], width, lefty); if (!(s->flags & CODEC_FLAG_GRAY)) { leftu = s->hdsp.add_hfyu_left_pred(p->data[1] + p->linesize[2], s->temp[1], width2, leftu); leftv = s->hdsp.add_hfyu_left_pred(p->data[2] + p->linesize[1], s->temp[2], width2, leftv); } - y++; cy++; + y++; + cy++; } /* next 4 pixels are left predicted too */ decode_422_bitstream(s, 4); - lefty = s->hdsp.add_hfyu_left_pred(p->data[0] + fake_ystride, s->temp[0], 4, lefty); - if (!(s->flags&CODEC_FLAG_GRAY)) { + lefty = s->hdsp.add_hfyu_left_pred(p->data[0] + fake_ystride, + s->temp[0], 4, lefty); + if (!(s->flags & CODEC_FLAG_GRAY)) { leftu = s->hdsp.add_hfyu_left_pred(p->data[1] + fake_ustride, s->temp[1], 2, leftu); leftv = s->hdsp.add_hfyu_left_pred(p->data[2] + fake_vstride, s->temp[2], 2, leftv); } @@ -625,26 +635,32 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, /* next line except the first 4 pixels is median predicted */ lefttopy = p->data[0][3]; decode_422_bitstream(s, width - 4); - s->hdsp.add_hfyu_median_pred(p->data[0] + fake_ystride + 4, p->data[0] + 4, s->temp[0], width - 4, &lefty, &lefttopy); - if (!(s->flags&CODEC_FLAG_GRAY)) { + s->hdsp.add_hfyu_median_pred(p->data[0] + fake_ystride + 4, + p->data[0] + 4, s->temp[0], + width - 4, &lefty, &lefttopy); + if (!(s->flags & CODEC_FLAG_GRAY)) { lefttopu = p->data[1][1]; lefttopv = p->data[2][1]; s->hdsp.add_hfyu_median_pred(p->data[1] + fake_ustride + 2, p->data[1] + 2, s->temp[1], width2 - 2, &leftu, &lefttopu); s->hdsp.add_hfyu_median_pred(p->data[2] + fake_vstride + 2, p->data[2] + 2, s->temp[2], width2 - 2, &leftv, &lefttopv); } - y++; cy++; + y++; + cy++; - for (; ybitstream_bpp == 12) { while (2 * cy > y) { decode_gray_bitstream(s, width); ydst = p->data[0] + p->linesize[0] * y; - s->hdsp.add_hfyu_median_pred(ydst, ydst - fake_ystride, s->temp[0], width, &lefty, &lefttopy); + s->hdsp.add_hfyu_median_pred(ydst, ydst - fake_ystride, + s->temp[0], width, + &lefty, &lefttopy); y++; } - if (y >= height) break; + if (y >= height) + break; } draw_slice(s, p, y); @@ -654,7 +670,9 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, udst = p->data[1] + p->linesize[1] * cy; vdst = p->data[2] + p->linesize[2] * cy; - s->hdsp.add_hfyu_median_pred(ydst, ydst - fake_ystride, s->temp[0], width, &lefty, &lefttopy); + s->hdsp.add_hfyu_median_pred(ydst, ydst - fake_ystride, + s->temp[0], width, + &lefty, &lefttopy); if (!(s->flags & CODEC_FLAG_GRAY)) { s->hdsp.add_hfyu_median_pred(udst, udst - fake_ustride, s->temp[1], width2, &leftu, &lefttopu); s->hdsp.add_hfyu_median_pred(vdst, vdst - fake_vstride, s->temp[2], width2, &leftv, &lefttopv); @@ -671,15 +689,15 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, const int last_line = (height - 1) * p->linesize[0]; if (s->bitstream_bpp == 32) { - lefta = p->data[0][last_line+A] = get_bits(&s->gb, 8); - leftr = p->data[0][last_line+R] = get_bits(&s->gb, 8); - leftg = p->data[0][last_line+G] = get_bits(&s->gb, 8); - leftb = p->data[0][last_line+B] = get_bits(&s->gb, 8); + lefta = p->data[0][last_line + A] = get_bits(&s->gb, 8); + leftr = p->data[0][last_line + R] = get_bits(&s->gb, 8); + leftg = p->data[0][last_line + G] = get_bits(&s->gb, 8); + leftb = p->data[0][last_line + B] = get_bits(&s->gb, 8); } else { - leftr = p->data[0][last_line+R] = get_bits(&s->gb, 8); - leftg = p->data[0][last_line+G] = get_bits(&s->gb, 8); - leftb = p->data[0][last_line+B] = get_bits(&s->gb, 8); - lefta = p->data[0][last_line+A] = 255; + leftr = p->data[0][last_line + R] = get_bits(&s->gb, 8); + leftg = p->data[0][last_line + G] = get_bits(&s->gb, 8); + leftb = p->data[0][last_line + B] = get_bits(&s->gb, 8); + lefta = p->data[0][last_line + A] = 255; skip_bits(&s->gb, 8); } @@ -688,14 +706,19 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, case LEFT: case PLANE: decode_bgr_bitstream(s, width - 1); - s->hdsp.add_hfyu_left_pred_bgr32(p->data[0] + last_line + 4, s->temp[0], width - 1, &leftr, &leftg, &leftb, &lefta); + s->hdsp.add_hfyu_left_pred_bgr32(p->data[0] + last_line + 4, + s->temp[0], width - 1, &leftr, + &leftg, &leftb, &lefta); - for (y = s->height - 2; y >= 0; y--) { //Yes it is stored upside down. + for (y = s->height - 2; y >= 0; y--) { // Yes it is stored upside down. decode_bgr_bitstream(s, width); - s->hdsp.add_hfyu_left_pred_bgr32(p->data[0] + p->linesize[0] * y, s->temp[0], width, &leftr, &leftg, &leftb, &lefta); + s->hdsp.add_hfyu_left_pred_bgr32(p->data[0] + p->linesize[0] * y, + s->temp[0], width, &leftr, + &leftg, &leftb, &lefta); if (s->predictor == PLANE) { - if (s->bitstream_bpp != 32) lefta = 0; + if (s->bitstream_bpp != 32) + lefta = 0; if ((y & s->interlaced) == 0 && y < s->height - 1 - s->interlaced) { s->hdsp.add_bytes(p->data[0] + p->linesize[0] * y, @@ -711,7 +734,7 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, av_log(avctx, AV_LOG_ERROR, "prediction type not supported!\n"); } - }else{ + } else { av_log(avctx, AV_LOG_ERROR, "BGR24 output is not implemented yet\n"); return -1; @@ -732,9 +755,8 @@ static av_cold int decode_end(AVCodecContext *avctx) ff_huffyuv_common_end(s); av_freep(&s->bitstream_buffer); - for (i = 0; i < 6; i++) { + for (i = 0; i < 6; i++) ff_free_vlc(&s->vlc[i]); - } return 0; } @@ -767,4 +789,4 @@ AVCodec ff_ffvhuff_decoder = { CODEC_CAP_FRAME_THREADS, .init_thread_copy = ONLY_IF_THREADS_ENABLED(decode_init_thread_copy), }; -#endif +#endif /* CONFIG_FFVHUFF_DECODER */ -- cgit v1.2.3