Commit 9a23b599 authored by Alexandra Hájková's avatar Alexandra Hájková Committed by Anton Khirnov

bink: Convert to the new bitstream reader

Signed-off-by: 's avatarAnton Khirnov <anton@khirnov.net>
parent dae9b0b9
...@@ -28,8 +28,8 @@ ...@@ -28,8 +28,8 @@
#include "avcodec.h" #include "avcodec.h"
#include "binkdata.h" #include "binkdata.h"
#include "binkdsp.h" #include "binkdsp.h"
#include "bitstream.h"
#include "blockdsp.h" #include "blockdsp.h"
#include "get_bits.h"
#include "hpeldsp.h" #include "hpeldsp.h"
#include "internal.h" #include "internal.h"
#include "mathops.h" #include "mathops.h"
...@@ -93,8 +93,9 @@ typedef struct Tree { ...@@ -93,8 +93,9 @@ typedef struct Tree {
uint8_t syms[16]; ///< leaf value to symbol mapping uint8_t syms[16]; ///< leaf value to symbol mapping
} Tree; } Tree;
#define GET_HUFF(gb, tree) (tree).syms[get_vlc2(gb, bink_trees[(tree).vlc_num].table,\ #define GET_HUFF(bc, tree) \
bink_trees[(tree).vlc_num].bits, 1)] (tree).syms[bitstream_read_vlc(bc, bink_trees[(tree).vlc_num].table, \
bink_trees[(tree).vlc_num].bits, 1)]
/** /**
* data structure used for decoding single Bink data type * data structure used for decoding single Bink data type
...@@ -204,18 +205,18 @@ static av_cold void free_bundles(BinkContext *c) ...@@ -204,18 +205,18 @@ static av_cold void free_bundles(BinkContext *c)
/** /**
* Merge two consequent lists of equal size depending on bits read. * Merge two consequent lists of equal size depending on bits read.
* *
* @param gb context for reading bits * @param bc context for reading bits
* @param dst buffer where merged list will be written to * @param dst buffer where merged list will be written to
* @param src pointer to the head of the first list (the second lists starts at src+size) * @param src pointer to the head of the first list (the second lists starts at src+size)
* @param size input lists size * @param size input lists size
*/ */
static void merge(GetBitContext *gb, uint8_t *dst, uint8_t *src, int size) static void merge(BitstreamContext *bc, uint8_t *dst, uint8_t *src, int size)
{ {
uint8_t *src2 = src + size; uint8_t *src2 = src + size;
int size2 = size; int size2 = size;
do { do {
if (!get_bits1(gb)) { if (!bitstream_read_bit(bc)) {
*dst++ = *src++; *dst++ = *src++;
size--; size--;
} else { } else {
...@@ -233,37 +234,37 @@ static void merge(GetBitContext *gb, uint8_t *dst, uint8_t *src, int size) ...@@ -233,37 +234,37 @@ static void merge(GetBitContext *gb, uint8_t *dst, uint8_t *src, int size)
/** /**
* Read information about Huffman tree used to decode data. * Read information about Huffman tree used to decode data.
* *
* @param gb context for reading bits * @param bc context for reading bits
* @param tree pointer for storing tree data * @param tree pointer for storing tree data
*/ */
static void read_tree(GetBitContext *gb, Tree *tree) static void read_tree(BitstreamContext *bc, Tree *tree)
{ {
uint8_t tmp1[16] = { 0 }, tmp2[16], *in = tmp1, *out = tmp2; uint8_t tmp1[16] = { 0 }, tmp2[16], *in = tmp1, *out = tmp2;
int i, t, len; int i, t, len;
tree->vlc_num = get_bits(gb, 4); tree->vlc_num = bitstream_read(bc, 4);
if (!tree->vlc_num) { if (!tree->vlc_num) {
for (i = 0; i < 16; i++) for (i = 0; i < 16; i++)
tree->syms[i] = i; tree->syms[i] = i;
return; return;
} }
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
len = get_bits(gb, 3); len = bitstream_read(bc, 3);
for (i = 0; i <= len; i++) { for (i = 0; i <= len; i++) {
tree->syms[i] = get_bits(gb, 4); tree->syms[i] = bitstream_read(bc, 4);
tmp1[tree->syms[i]] = 1; tmp1[tree->syms[i]] = 1;
} }
for (i = 0; i < 16 && len < 16 - 1; i++) for (i = 0; i < 16 && len < 16 - 1; i++)
if (!tmp1[i]) if (!tmp1[i])
tree->syms[++len] = i; tree->syms[++len] = i;
} else { } else {
len = get_bits(gb, 2); len = bitstream_read(bc, 2);
for (i = 0; i < 16; i++) for (i = 0; i < 16; i++)
in[i] = i; in[i] = i;
for (i = 0; i <= len; i++) { for (i = 0; i <= len; i++) {
int size = 1 << i; int size = 1 << i;
for (t = 0; t < 16; t += size << 1) for (t = 0; t < 16; t += size << 1)
merge(gb, out + t, in + t, size); merge(bc, out + t, in + t, size);
FFSWAP(uint8_t*, in, out); FFSWAP(uint8_t*, in, out);
} }
memcpy(tree->syms, in, 16); memcpy(tree->syms, in, 16);
...@@ -273,21 +274,21 @@ static void read_tree(GetBitContext *gb, Tree *tree) ...@@ -273,21 +274,21 @@ static void read_tree(GetBitContext *gb, Tree *tree)
/** /**
* Prepare bundle for decoding data. * Prepare bundle for decoding data.
* *
* @param gb context for reading bits * @param bc context for reading bits
* @param c decoder context * @param c decoder context
* @param bundle_num number of the bundle to initialize * @param bundle_num number of the bundle to initialize
*/ */
static void read_bundle(GetBitContext *gb, BinkContext *c, int bundle_num) static void read_bundle(BitstreamContext *bc, BinkContext *c, int bundle_num)
{ {
int i; int i;
if (bundle_num == BINK_SRC_COLORS) { if (bundle_num == BINK_SRC_COLORS) {
for (i = 0; i < 16; i++) for (i = 0; i < 16; i++)
read_tree(gb, &c->col_high[i]); read_tree(bc, &c->col_high[i]);
c->col_lastval = 0; c->col_lastval = 0;
} }
if (bundle_num != BINK_SRC_INTRA_DC && bundle_num != BINK_SRC_INTER_DC) if (bundle_num != BINK_SRC_INTRA_DC && bundle_num != BINK_SRC_INTER_DC)
read_tree(gb, &c->bundle[bundle_num].tree); read_tree(bc, &c->bundle[bundle_num].tree);
c->bundle[bundle_num].cur_dec = c->bundle[bundle_num].cur_dec =
c->bundle[bundle_num].cur_ptr = c->bundle[bundle_num].data; c->bundle[bundle_num].cur_ptr = c->bundle[bundle_num].data;
} }
...@@ -295,66 +296,64 @@ static void read_bundle(GetBitContext *gb, BinkContext *c, int bundle_num) ...@@ -295,66 +296,64 @@ static void read_bundle(GetBitContext *gb, BinkContext *c, int bundle_num)
/** /**
* common check before starting decoding bundle data * common check before starting decoding bundle data
* *
* @param gb context for reading bits * @param bc context for reading bits
* @param b bundle * @param b bundle
* @param t variable where number of elements to decode will be stored * @param t variable where number of elements to decode will be stored
*/ */
#define CHECK_READ_VAL(gb, b, t) \ #define CHECK_READ_VAL(bc, b, t) \
if (!b->cur_dec || (b->cur_dec > b->cur_ptr)) \ if (!b->cur_dec || (b->cur_dec > b->cur_ptr)) \
return 0; \ return 0; \
t = get_bits(gb, b->len); \ t = bitstream_read(bc, b->len); \
if (!t) { \ if (!t) { \
b->cur_dec = NULL; \ b->cur_dec = NULL; \
return 0; \ return 0; \
} \ } \
static int read_runs(AVCodecContext *avctx, GetBitContext *gb, Bundle *b) static int read_runs(AVCodecContext *avctx, BitstreamContext *bc, Bundle *b)
{ {
int t, v; int t, v;
const uint8_t *dec_end; const uint8_t *dec_end;
CHECK_READ_VAL(gb, b, t); CHECK_READ_VAL(bc, b, t);
dec_end = b->cur_dec + t; dec_end = b->cur_dec + t;
if (dec_end > b->data_end) { if (dec_end > b->data_end) {
av_log(avctx, AV_LOG_ERROR, "Run value went out of bounds\n"); av_log(avctx, AV_LOG_ERROR, "Run value went out of bounds\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
v = get_bits(gb, 4); v = bitstream_read(bc, 4);
memset(b->cur_dec, v, t); memset(b->cur_dec, v, t);
b->cur_dec += t; b->cur_dec += t;
} else { } else {
while (b->cur_dec < dec_end) while (b->cur_dec < dec_end)
*b->cur_dec++ = GET_HUFF(gb, b->tree); *b->cur_dec++ = GET_HUFF(bc, b->tree);
} }
return 0; return 0;
} }
static int read_motion_values(AVCodecContext *avctx, GetBitContext *gb, Bundle *b) static int read_motion_values(AVCodecContext *avctx, BitstreamContext *bc, Bundle *b)
{ {
int t, sign, v; int t, v;
const uint8_t *dec_end; const uint8_t *dec_end;
CHECK_READ_VAL(gb, b, t); CHECK_READ_VAL(bc, b, t);
dec_end = b->cur_dec + t; dec_end = b->cur_dec + t;
if (dec_end > b->data_end) { if (dec_end > b->data_end) {
av_log(avctx, AV_LOG_ERROR, "Too many motion values\n"); av_log(avctx, AV_LOG_ERROR, "Too many motion values\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
v = get_bits(gb, 4); v = bitstream_read(bc, 4);
if (v) { if (v) {
sign = -get_bits1(gb); v = bitstream_apply_sign(bc, v);
v = (v ^ sign) - sign;
} }
memset(b->cur_dec, v, t); memset(b->cur_dec, v, t);
b->cur_dec += t; b->cur_dec += t;
} else { } else {
while (b->cur_dec < dec_end) { while (b->cur_dec < dec_end) {
v = GET_HUFF(gb, b->tree); v = GET_HUFF(bc, b->tree);
if (v) { if (v) {
sign = -get_bits1(gb); v = bitstream_apply_sign(bc, v);
v = (v ^ sign) - sign;
} }
*b->cur_dec++ = v; *b->cur_dec++ = v;
} }
...@@ -364,25 +363,25 @@ static int read_motion_values(AVCodecContext *avctx, GetBitContext *gb, Bundle * ...@@ -364,25 +363,25 @@ static int read_motion_values(AVCodecContext *avctx, GetBitContext *gb, Bundle *
static const uint8_t bink_rlelens[4] = { 4, 8, 12, 32 }; static const uint8_t bink_rlelens[4] = { 4, 8, 12, 32 };
static int read_block_types(AVCodecContext *avctx, GetBitContext *gb, Bundle *b) static int read_block_types(AVCodecContext *avctx, BitstreamContext *bc, Bundle *b)
{ {
int t, v; int t, v;
int last = 0; int last = 0;
const uint8_t *dec_end; const uint8_t *dec_end;
CHECK_READ_VAL(gb, b, t); CHECK_READ_VAL(bc, b, t);
dec_end = b->cur_dec + t; dec_end = b->cur_dec + t;
if (dec_end > b->data_end) { if (dec_end > b->data_end) {
av_log(avctx, AV_LOG_ERROR, "Too many block type values\n"); av_log(avctx, AV_LOG_ERROR, "Too many block type values\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
v = get_bits(gb, 4); v = bitstream_read(bc, 4);
memset(b->cur_dec, v, t); memset(b->cur_dec, v, t);
b->cur_dec += t; b->cur_dec += t;
} else { } else {
while (b->cur_dec < dec_end) { while (b->cur_dec < dec_end) {
v = GET_HUFF(gb, b->tree); v = GET_HUFF(bc, b->tree);
if (v < 12) { if (v < 12) {
last = v; last = v;
*b->cur_dec++ = v; *b->cur_dec++ = v;
...@@ -399,40 +398,40 @@ static int read_block_types(AVCodecContext *avctx, GetBitContext *gb, Bundle *b) ...@@ -399,40 +398,40 @@ static int read_block_types(AVCodecContext *avctx, GetBitContext *gb, Bundle *b)
return 0; return 0;
} }
static int read_patterns(AVCodecContext *avctx, GetBitContext *gb, Bundle *b) static int read_patterns(AVCodecContext *avctx, BitstreamContext *bc, Bundle *b)
{ {
int t, v; int t, v;
const uint8_t *dec_end; const uint8_t *dec_end;
CHECK_READ_VAL(gb, b, t); CHECK_READ_VAL(bc, b, t);
dec_end = b->cur_dec + t; dec_end = b->cur_dec + t;
if (dec_end > b->data_end) { if (dec_end > b->data_end) {
av_log(avctx, AV_LOG_ERROR, "Too many pattern values\n"); av_log(avctx, AV_LOG_ERROR, "Too many pattern values\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
while (b->cur_dec < dec_end) { while (b->cur_dec < dec_end) {
v = GET_HUFF(gb, b->tree); v = GET_HUFF(bc, b->tree);
v |= GET_HUFF(gb, b->tree) << 4; v |= GET_HUFF(bc, b->tree) << 4;
*b->cur_dec++ = v; *b->cur_dec++ = v;
} }
return 0; return 0;
} }
static int read_colors(GetBitContext *gb, Bundle *b, BinkContext *c) static int read_colors(BitstreamContext *bc, Bundle *b, BinkContext *c)
{ {
int t, sign, v; int t, sign, v;
const uint8_t *dec_end; const uint8_t *dec_end;
CHECK_READ_VAL(gb, b, t); CHECK_READ_VAL(bc, b, t);
dec_end = b->cur_dec + t; dec_end = b->cur_dec + t;
if (dec_end > b->data_end) { if (dec_end > b->data_end) {
av_log(c->avctx, AV_LOG_ERROR, "Too many color values\n"); av_log(c->avctx, AV_LOG_ERROR, "Too many color values\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
c->col_lastval = GET_HUFF(gb, c->col_high[c->col_lastval]); c->col_lastval = GET_HUFF(bc, c->col_high[c->col_lastval]);
v = GET_HUFF(gb, b->tree); v = GET_HUFF(bc, b->tree);
v = (c->col_lastval << 4) | v; v = (c->col_lastval << 4) | v;
if (c->version < 'i') { if (c->version < 'i') {
sign = ((int8_t) v) >> 7; sign = ((int8_t) v) >> 7;
...@@ -443,8 +442,8 @@ static int read_colors(GetBitContext *gb, Bundle *b, BinkContext *c) ...@@ -443,8 +442,8 @@ static int read_colors(GetBitContext *gb, Bundle *b, BinkContext *c)
b->cur_dec += t; b->cur_dec += t;
} else { } else {
while (b->cur_dec < dec_end) { while (b->cur_dec < dec_end) {
c->col_lastval = GET_HUFF(gb, c->col_high[c->col_lastval]); c->col_lastval = GET_HUFF(bc, c->col_high[c->col_lastval]);
v = GET_HUFF(gb, b->tree); v = GET_HUFF(bc, b->tree);
v = (c->col_lastval << 4) | v; v = (c->col_lastval << 4) | v;
if (c->version < 'i') { if (c->version < 'i') {
sign = ((int8_t) v) >> 7; sign = ((int8_t) v) >> 7;
...@@ -460,18 +459,17 @@ static int read_colors(GetBitContext *gb, Bundle *b, BinkContext *c) ...@@ -460,18 +459,17 @@ static int read_colors(GetBitContext *gb, Bundle *b, BinkContext *c)
/** number of bits used to store first DC value in bundle */ /** number of bits used to store first DC value in bundle */
#define DC_START_BITS 11 #define DC_START_BITS 11
static int read_dcs(AVCodecContext *avctx, GetBitContext *gb, Bundle *b, static int read_dcs(AVCodecContext *avctx, BitstreamContext *bc, Bundle *b,
int start_bits, int has_sign) int start_bits, int has_sign)
{ {
int i, j, len, len2, bsize, sign, v, v2; int i, j, len, len2, bsize, v, v2;
int16_t *dst = (int16_t*)b->cur_dec; int16_t *dst = (int16_t*)b->cur_dec;
int16_t *dst_end = (int16_t*)b->data_end; int16_t *dst_end = (int16_t*)b->data_end;
CHECK_READ_VAL(gb, b, len); CHECK_READ_VAL(bc, b, len);
v = get_bits(gb, start_bits - has_sign); v = bitstream_read(bc, start_bits - has_sign);
if (v && has_sign) { if (v && has_sign) {
sign = -get_bits1(gb); v = bitstream_apply_sign(bc, v);
v = (v ^ sign) - sign;
} }
if (dst_end - dst < 1) if (dst_end - dst < 1)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
...@@ -481,13 +479,12 @@ static int read_dcs(AVCodecContext *avctx, GetBitContext *gb, Bundle *b, ...@@ -481,13 +479,12 @@ static int read_dcs(AVCodecContext *avctx, GetBitContext *gb, Bundle *b,
len2 = FFMIN(len - i, 8); len2 = FFMIN(len - i, 8);
if (dst_end - dst < len2) if (dst_end - dst < len2)
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
bsize = get_bits(gb, 4); bsize = bitstream_read(bc, 4);
if (bsize) { if (bsize) {
for (j = 0; j < len2; j++) { for (j = 0; j < len2; j++) {
v2 = get_bits(gb, bsize); v2 = bitstream_read(bc, bsize);
if (v2) { if (v2) {
sign = -get_bits1(gb); v2 = bitstream_apply_sign(bc, v2);
v2 = (v2 ^ sign) - sign;
} }
v += v2; v += v2;
*dst++ = v; *dst++ = v;
...@@ -539,7 +536,7 @@ static av_cold void binkb_init_bundles(BinkContext *c) ...@@ -539,7 +536,7 @@ static av_cold void binkb_init_bundles(BinkContext *c)
binkb_init_bundle(c, i); binkb_init_bundle(c, i);
} }
static int binkb_read_bundle(BinkContext *c, GetBitContext *gb, int bundle_num) static int binkb_read_bundle(BinkContext *c, BitstreamContext *bc, int bundle_num)
{ {
const int bits = binkb_bundle_sizes[bundle_num]; const int bits = binkb_bundle_sizes[bundle_num];
const int mask = 1 << (bits - 1); const int mask = 1 << (bits - 1);
...@@ -547,26 +544,26 @@ static int binkb_read_bundle(BinkContext *c, GetBitContext *gb, int bundle_num) ...@@ -547,26 +544,26 @@ static int binkb_read_bundle(BinkContext *c, GetBitContext *gb, int bundle_num)
Bundle *b = &c->bundle[bundle_num]; Bundle *b = &c->bundle[bundle_num];
int i, len; int i, len;
CHECK_READ_VAL(gb, b, len); CHECK_READ_VAL(bc, b, len);
if (b->data_end - b->cur_dec < len * (1 + (bits > 8))) if (b->data_end - b->cur_dec < len * (1 + (bits > 8)))
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
if (bits <= 8) { if (bits <= 8) {
if (!issigned) { if (!issigned) {
for (i = 0; i < len; i++) for (i = 0; i < len; i++)
*b->cur_dec++ = get_bits(gb, bits); *b->cur_dec++ = bitstream_read(bc, bits);
} else { } else {
for (i = 0; i < len; i++) for (i = 0; i < len; i++)
*b->cur_dec++ = get_bits(gb, bits) - mask; *b->cur_dec++ = bitstream_read(bc, bits) - mask;
} }
} else { } else {
int16_t *dst = (int16_t*)b->cur_dec; int16_t *dst = (int16_t*)b->cur_dec;
if (!issigned) { if (!issigned) {
for (i = 0; i < len; i++) for (i = 0; i < len; i++)
*dst++ = get_bits(gb, bits); *dst++ = bitstream_read(bc, bits);
} else { } else {
for (i = 0; i < len; i++) for (i = 0; i < len; i++)
*dst++ = get_bits(gb, bits) - mask; *dst++ = bitstream_read(bc, bits) - mask;
} }
b->cur_dec = (uint8_t*)dst; b->cur_dec = (uint8_t*)dst;
} }
...@@ -590,18 +587,19 @@ static inline int binkb_get_value(BinkContext *c, int bundle_num) ...@@ -590,18 +587,19 @@ static inline int binkb_get_value(BinkContext *c, int bundle_num)
/** /**
* Read 8x8 block of DCT coefficients. * Read 8x8 block of DCT coefficients.
* *
* @param gb context for reading bits * @param bc context for reading bits
* @param block place for storing coefficients * @param block place for storing coefficients
* @param scan scan order table * @param scan scan order table
* @param quant_matrices quantization matrices * @param quant_matrices quantization matrices
* @return 0 for success, negative value in other cases * @return 0 for success, negative value in other cases
*/ */
static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t *scan, static int read_dct_coeffs(BitstreamContext *bc, int32_t block[64],
const uint8_t *scan,
const int32_t quant_matrices[16][64], int q) const int32_t quant_matrices[16][64], int q)
{ {
int coef_list[128]; int coef_list[128];
int mode_list[128]; int mode_list[128];
int i, t, bits, ccoef, mode, sign; int i, t, bits, ccoef, mode;
int list_start = 64, list_end = 64, list_pos; int list_start = 64, list_end = 64, list_pos;
int coef_count = 0; int coef_count = 0;
int coef_idx[64]; int coef_idx[64];
...@@ -615,10 +613,10 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t * ...@@ -615,10 +613,10 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t *
coef_list[list_end] = 2; mode_list[list_end++] = 3; coef_list[list_end] = 2; mode_list[list_end++] = 3;
coef_list[list_end] = 3; mode_list[list_end++] = 3; coef_list[list_end] = 3; mode_list[list_end++] = 3;
for (bits = get_bits(gb, 4) - 1; bits >= 0; bits--) { for (bits = bitstream_read(bc, 4) - 1; bits >= 0; bits--) {
list_pos = list_start; list_pos = list_start;
while (list_pos < list_end) { while (list_pos < list_end) {
if (!(mode_list[list_pos] | coef_list[list_pos]) || !get_bits1(gb)) { if (!(mode_list[list_pos] | coef_list[list_pos]) || !bitstream_read_bit(bc)) {
list_pos++; list_pos++;
continue; continue;
} }
...@@ -634,16 +632,15 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t * ...@@ -634,16 +632,15 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t *
mode_list[list_pos++] = 0; mode_list[list_pos++] = 0;
} }
for (i = 0; i < 4; i++, ccoef++) { for (i = 0; i < 4; i++, ccoef++) {
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
coef_list[--list_start] = ccoef; coef_list[--list_start] = ccoef;
mode_list[ list_start] = 3; mode_list[ list_start] = 3;
} else { } else {
if (!bits) { if (!bits) {
t = 1 - (get_bits1(gb) << 1); t = 1 - (bitstream_read_bit(bc) << 1);
} else { } else {
t = get_bits(gb, bits) | 1 << bits; t = bitstream_read(bc, bits) | 1 << bits;
sign = -get_bits1(gb); t = bitstream_apply_sign(bc, t);
t = (t ^ sign) - sign;
} }
block[scan[ccoef]] = t; block[scan[ccoef]] = t;
coef_idx[coef_count++] = ccoef; coef_idx[coef_count++] = ccoef;
...@@ -660,11 +657,10 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t * ...@@ -660,11 +657,10 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t *
break; break;
case 3: case 3:
if (!bits) { if (!bits) {
t = 1 - (get_bits1(gb) << 1); t = 1 - (bitstream_read_bit(bc) << 1);
} else { } else {
t = get_bits(gb, bits) | 1 << bits; t = bitstream_read(bc, bits) | 1 << bits;
sign = -get_bits1(gb); t = bitstream_apply_sign(bc, t);
t = (t ^ sign) - sign;
} }
block[scan[ccoef]] = t; block[scan[ccoef]] = t;
coef_idx[coef_count++] = ccoef; coef_idx[coef_count++] = ccoef;
...@@ -676,7 +672,7 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t * ...@@ -676,7 +672,7 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t *
} }
if (q == -1) { if (q == -1) {
quant_idx = get_bits(gb, 4); quant_idx = bitstream_read(bc, 4);
} else { } else {
quant_idx = q; quant_idx = q;
} }
...@@ -698,16 +694,16 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t * ...@@ -698,16 +694,16 @@ static int read_dct_coeffs(GetBitContext *gb, int32_t block[64], const uint8_t *
/** /**
* Read 8x8 block with residue after motion compensation. * Read 8x8 block with residue after motion compensation.
* *
* @param gb context for reading bits * @param bc context for reading bits
* @param block place to store read data * @param block place to store read data
* @param masks_count number of masks to decode * @param masks_count number of masks to decode
* @return 0 on success, negative value in other cases * @return 0 on success, negative value in other cases
*/ */
static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count) static int read_residue(BitstreamContext *bc, int16_t block[64], int masks_count)
{ {
int coef_list[128]; int coef_list[128];
int mode_list[128]; int mode_list[128];
int i, sign, mask, ccoef, mode; int i, mask, ccoef, mode;
int list_start = 64, list_end = 64, list_pos; int list_start = 64, list_end = 64, list_pos;
int nz_coeff[64]; int nz_coeff[64];
int nz_coeff_count = 0; int nz_coeff_count = 0;
...@@ -717,9 +713,9 @@ static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count) ...@@ -717,9 +713,9 @@ static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count)
coef_list[list_end] = 44; mode_list[list_end++] = 0; coef_list[list_end] = 44; mode_list[list_end++] = 0;
coef_list[list_end] = 0; mode_list[list_end++] = 2; coef_list[list_end] = 0; mode_list[list_end++] = 2;
for (mask = 1 << get_bits(gb, 3); mask; mask >>= 1) { for (mask = 1 << bitstream_read(bc, 3); mask; mask >>= 1) {
for (i = 0; i < nz_coeff_count; i++) { for (i = 0; i < nz_coeff_count; i++) {
if (!get_bits1(gb)) if (!bitstream_read_bit(bc))
continue; continue;
if (block[nz_coeff[i]] < 0) if (block[nz_coeff[i]] < 0)
block[nz_coeff[i]] -= mask; block[nz_coeff[i]] -= mask;
...@@ -731,7 +727,7 @@ static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count) ...@@ -731,7 +727,7 @@ static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count)
} }
list_pos = list_start; list_pos = list_start;
while (list_pos < list_end) { while (list_pos < list_end) {
if (!(coef_list[list_pos] | mode_list[list_pos]) || !get_bits1(gb)) { if (!(coef_list[list_pos] | mode_list[list_pos]) || !bitstream_read_bit(bc)) {
list_pos++; list_pos++;
continue; continue;
} }
...@@ -747,13 +743,12 @@ static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count) ...@@ -747,13 +743,12 @@ static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count)
mode_list[list_pos++] = 0; mode_list[list_pos++] = 0;
} }
for (i = 0; i < 4; i++, ccoef++) { for (i = 0; i < 4; i++, ccoef++) {
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
coef_list[--list_start] = ccoef; coef_list[--list_start] = ccoef;
mode_list[ list_start] = 3; mode_list[ list_start] = 3;
} else { } else {
nz_coeff[nz_coeff_count++] = bink_scan[ccoef]; nz_coeff[nz_coeff_count++] = bink_scan[ccoef];
sign = -get_bits1(gb); block[bink_scan[ccoef]] = bitstream_apply_sign(bc, mask);
block[bink_scan[ccoef]] = (mask ^ sign) - sign;
masks_count--; masks_count--;
if (masks_count < 0) if (masks_count < 0)
return 0; return 0;
...@@ -770,8 +765,7 @@ static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count) ...@@ -770,8 +765,7 @@ static int read_residue(GetBitContext *gb, int16_t block[64], int masks_count)
break; break;
case 3: case 3:
nz_coeff[nz_coeff_count++] = bink_scan[ccoef]; nz_coeff[nz_coeff_count++] = bink_scan[ccoef];
sign = -get_bits1(gb); block[bink_scan[ccoef]] = bitstream_apply_sign(bc, mask);
block[bink_scan[ccoef]] = (mask ^ sign) - sign;
coef_list[list_pos] = 0; coef_list[list_pos] = 0;
mode_list[list_pos++] = 0; mode_list[list_pos++] = 0;
masks_count--; masks_count--;
...@@ -798,7 +792,7 @@ static inline void put_pixels8x8_overlapped(uint8_t *dst, uint8_t *src, int stri ...@@ -798,7 +792,7 @@ static inline void put_pixels8x8_overlapped(uint8_t *dst, uint8_t *src, int stri
memcpy(dst + i*stride, tmp + i*8, 8); memcpy(dst + i*stride, tmp + i*8, 8);
} }
static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, static int binkb_decode_plane(BinkContext *c, AVFrame *frame, BitstreamContext *bc,
int plane_idx, int is_key, int is_chroma) int plane_idx, int is_key, int is_chroma)
{ {
int blk, ret; int blk, ret;
...@@ -826,7 +820,7 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -826,7 +820,7 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
for (by = 0; by < bh; by++) { for (by = 0; by < bh; by++) {
for (i = 0; i < BINKB_NB_SRC; i++) { for (i = 0; i < BINKB_NB_SRC; i++) {
if ((ret = binkb_read_bundle(c, gb, i)) < 0) if ((ret = binkb_read_bundle(c, bc, i)) < 0)
return ret; return ret;
} }
...@@ -837,13 +831,11 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -837,13 +831,11 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
case 0: case 0:
break; break;
case 1: case 1:
scan = bink_patterns[get_bits(gb, 4)]; scan = bink_patterns[bitstream_read(bc, 4)];
i = 0; i = 0;
do { do {
int mode, run; int mode = bitstream_read_bit(bc);
int run = bitstream_read(bc, binkb_runbits[i]) + 1;
mode = get_bits1(gb);
run = get_bits(gb, binkb_runbits[i]) + 1;
i += run; i += run;
if (i > 64) { if (i > 64) {
...@@ -866,7 +858,7 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -866,7 +858,7 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
memset(dctblock, 0, sizeof(*dctblock) * 64); memset(dctblock, 0, sizeof(*dctblock) * 64);
dctblock[0] = binkb_get_value(c, BINKB_SRC_INTRA_DC); dctblock[0] = binkb_get_value(c, BINKB_SRC_INTRA_DC);
qp = binkb_get_value(c, BINKB_SRC_INTRA_Q); qp = binkb_get_value(c, BINKB_SRC_INTRA_Q);
read_dct_coeffs(gb, dctblock, bink_scan, binkb_intra_quant, qp); read_dct_coeffs(bc, dctblock, bink_scan, binkb_intra_quant, qp);
c->binkdsp.idct_put(dst, stride, dctblock); c->binkdsp.idct_put(dst, stride, dctblock);
break; break;
case 3: case 3:
...@@ -882,7 +874,7 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -882,7 +874,7 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
} }
c->bdsp.clear_block(block); c->bdsp.clear_block(block);
v = binkb_get_value(c, BINKB_SRC_INTER_COEFS); v = binkb_get_value(c, BINKB_SRC_INTER_COEFS);
read_residue(gb, block, v); read_residue(bc, block, v);
c->binkdsp.add_pixels8(dst, block, stride); c->binkdsp.add_pixels8(dst, block, stride);
break; break;
case 4: case 4:
...@@ -899,7 +891,7 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -899,7 +891,7 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
memset(dctblock, 0, sizeof(*dctblock) * 64); memset(dctblock, 0, sizeof(*dctblock) * 64);
dctblock[0] = binkb_get_value(c, BINKB_SRC_INTER_DC); dctblock[0] = binkb_get_value(c, BINKB_SRC_INTER_DC);
qp = binkb_get_value(c, BINKB_SRC_INTER_Q); qp = binkb_get_value(c, BINKB_SRC_INTER_Q);
read_dct_coeffs(gb, dctblock, bink_scan, binkb_inter_quant, qp); read_dct_coeffs(bc, dctblock, bink_scan, binkb_inter_quant, qp);
c->binkdsp.idct_add(dst, stride, dctblock); c->binkdsp.idct_add(dst, stride, dctblock);
break; break;
case 5: case 5:
...@@ -938,8 +930,8 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -938,8 +930,8 @@ static int binkb_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
} }
} }
} }
if (get_bits_count(gb) & 0x1F) //next plane data starts at 32-bit boundary if (bitstream_tell(bc) & 0x1F) // next plane data starts at 32-bit boundary
skip_bits_long(gb, 32 - (get_bits_count(gb) & 0x1F)); bitstream_skip(bc, 32 - (bitstream_tell(bc) & 0x1F));
return 0; return 0;
} }
...@@ -962,7 +954,7 @@ static int bink_put_pixels(BinkContext *c, ...@@ -962,7 +954,7 @@ static int bink_put_pixels(BinkContext *c,
return 0; return 0;
} }
static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, static int bink_decode_plane(BinkContext *c, AVFrame *frame, BitstreamContext *bc,
int plane_idx, int is_chroma) int plane_idx, int is_chroma)
{ {
int blk, ret; int blk, ret;
...@@ -982,7 +974,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -982,7 +974,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
init_lengths(c, FFMAX(width, 8), bw); init_lengths(c, FFMAX(width, 8), bw);
for (i = 0; i < BINK_NB_SRC; i++) for (i = 0; i < BINK_NB_SRC; i++)
read_bundle(gb, c, i); read_bundle(bc, c, i);
ref_start = c->last->data[plane_idx] ? c->last->data[plane_idx] ref_start = c->last->data[plane_idx] ? c->last->data[plane_idx]
: frame->data[plane_idx]; : frame->data[plane_idx];
...@@ -993,23 +985,23 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -993,23 +985,23 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
coordmap[i] = (i & 7) + (i >> 3) * stride; coordmap[i] = (i & 7) + (i >> 3) * stride;
for (by = 0; by < bh; by++) { for (by = 0; by < bh; by++) {
if ((ret = read_block_types(c->avctx, gb, &c->bundle[BINK_SRC_BLOCK_TYPES])) < 0) if ((ret = read_block_types(c->avctx, bc, &c->bundle[BINK_SRC_BLOCK_TYPES])) < 0)
return ret; return ret;
if ((ret = read_block_types(c->avctx, gb, &c->bundle[BINK_SRC_SUB_BLOCK_TYPES])) < 0) if ((ret = read_block_types(c->avctx, bc, &c->bundle[BINK_SRC_SUB_BLOCK_TYPES])) < 0)
return ret; return ret;
if ((ret = read_colors(gb, &c->bundle[BINK_SRC_COLORS], c)) < 0) if ((ret = read_colors(bc, &c->bundle[BINK_SRC_COLORS], c)) < 0)
return ret; return ret;
if ((ret = read_patterns(c->avctx, gb, &c->bundle[BINK_SRC_PATTERN])) < 0) if ((ret = read_patterns(c->avctx, bc, &c->bundle[BINK_SRC_PATTERN])) < 0)
return ret; return ret;
if ((ret = read_motion_values(c->avctx, gb, &c->bundle[BINK_SRC_X_OFF])) < 0) if ((ret = read_motion_values(c->avctx, bc, &c->bundle[BINK_SRC_X_OFF])) < 0)
return ret; return ret;
if ((ret = read_motion_values(c->avctx, gb, &c->bundle[BINK_SRC_Y_OFF])) < 0) if ((ret = read_motion_values(c->avctx, bc, &c->bundle[BINK_SRC_Y_OFF])) < 0)
return ret; return ret;
if ((ret = read_dcs(c->avctx, gb, &c->bundle[BINK_SRC_INTRA_DC], DC_START_BITS, 0)) < 0) if ((ret = read_dcs(c->avctx, bc, &c->bundle[BINK_SRC_INTRA_DC], DC_START_BITS, 0)) < 0)
return ret; return ret;
if ((ret = read_dcs(c->avctx, gb, &c->bundle[BINK_SRC_INTER_DC], DC_START_BITS, 1)) < 0) if ((ret = read_dcs(c->avctx, bc, &c->bundle[BINK_SRC_INTER_DC], DC_START_BITS, 1)) < 0)
return ret; return ret;
if ((ret = read_runs(c->avctx, gb, &c->bundle[BINK_SRC_RUN])) < 0) if ((ret = read_runs(c->avctx, bc, &c->bundle[BINK_SRC_RUN])) < 0)
return ret; return ret;
if (by == bh) if (by == bh)
...@@ -1034,7 +1026,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -1034,7 +1026,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
blk = get_value(c, BINK_SRC_SUB_BLOCK_TYPES); blk = get_value(c, BINK_SRC_SUB_BLOCK_TYPES);
switch (blk) { switch (blk) {
case RUN_BLOCK: case RUN_BLOCK:
scan = bink_patterns[get_bits(gb, 4)]; scan = bink_patterns[bitstream_read(bc, 4)];
i = 0; i = 0;
do { do {
int run = get_value(c, BINK_SRC_RUN) + 1; int run = get_value(c, BINK_SRC_RUN) + 1;
...@@ -1044,7 +1036,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -1044,7 +1036,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
av_log(c->avctx, AV_LOG_ERROR, "Run went out of bounds\n"); av_log(c->avctx, AV_LOG_ERROR, "Run went out of bounds\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
v = get_value(c, BINK_SRC_COLORS); v = get_value(c, BINK_SRC_COLORS);
for (j = 0; j < run; j++) for (j = 0; j < run; j++)
ublock[*scan++] = v; ublock[*scan++] = v;
...@@ -1059,7 +1051,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -1059,7 +1051,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
case INTRA_BLOCK: case INTRA_BLOCK:
memset(dctblock, 0, sizeof(*dctblock) * 64); memset(dctblock, 0, sizeof(*dctblock) * 64);
dctblock[0] = get_value(c, BINK_SRC_INTRA_DC); dctblock[0] = get_value(c, BINK_SRC_INTRA_DC);
read_dct_coeffs(gb, dctblock, bink_scan, bink_intra_quant, -1); read_dct_coeffs(bc, dctblock, bink_scan, bink_intra_quant, -1);
c->binkdsp.idct_put(ublock, 8, dctblock); c->binkdsp.idct_put(ublock, 8, dctblock);
break; break;
case FILL_BLOCK: case FILL_BLOCK:
...@@ -1097,7 +1089,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -1097,7 +1089,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
return ret; return ret;
break; break;
case RUN_BLOCK: case RUN_BLOCK:
scan = bink_patterns[get_bits(gb, 4)]; scan = bink_patterns[bitstream_read(bc, 4)];
i = 0; i = 0;
do { do {
int run = get_value(c, BINK_SRC_RUN) + 1; int run = get_value(c, BINK_SRC_RUN) + 1;
...@@ -1107,7 +1099,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -1107,7 +1099,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
av_log(c->avctx, AV_LOG_ERROR, "Run went out of bounds\n"); av_log(c->avctx, AV_LOG_ERROR, "Run went out of bounds\n");
return AVERROR_INVALIDDATA; return AVERROR_INVALIDDATA;
} }
if (get_bits1(gb)) { if (bitstream_read_bit(bc)) {
v = get_value(c, BINK_SRC_COLORS); v = get_value(c, BINK_SRC_COLORS);
for (j = 0; j < run; j++) for (j = 0; j < run; j++)
dst[coordmap[*scan++]] = v; dst[coordmap[*scan++]] = v;
...@@ -1125,14 +1117,14 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -1125,14 +1117,14 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
if (ret < 0) if (ret < 0)
return ret; return ret;
c->bdsp.clear_block(block); c->bdsp.clear_block(block);
v = get_bits(gb, 7); v = bitstream_read(bc, 7);
read_residue(gb, block, v); read_residue(bc, block, v);
c->binkdsp.add_pixels8(dst, block, stride); c->binkdsp.add_pixels8(dst, block, stride);
break; break;
case INTRA_BLOCK: case INTRA_BLOCK:
memset(dctblock, 0, sizeof(*dctblock) * 64); memset(dctblock, 0, sizeof(*dctblock) * 64);
dctblock[0] = get_value(c, BINK_SRC_INTRA_DC); dctblock[0] = get_value(c, BINK_SRC_INTRA_DC);
read_dct_coeffs(gb, dctblock, bink_scan, bink_intra_quant, -1); read_dct_coeffs(bc, dctblock, bink_scan, bink_intra_quant, -1);
c->binkdsp.idct_put(dst, stride, dctblock); c->binkdsp.idct_put(dst, stride, dctblock);
break; break;
case FILL_BLOCK: case FILL_BLOCK:
...@@ -1146,7 +1138,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -1146,7 +1138,7 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
return ret; return ret;
memset(dctblock, 0, sizeof(*dctblock) * 64); memset(dctblock, 0, sizeof(*dctblock) * 64);
dctblock[0] = get_value(c, BINK_SRC_INTER_DC); dctblock[0] = get_value(c, BINK_SRC_INTER_DC);
read_dct_coeffs(gb, dctblock, bink_scan, bink_inter_quant, -1); read_dct_coeffs(bc, dctblock, bink_scan, bink_inter_quant, -1);
c->binkdsp.idct_add(dst, stride, dctblock); c->binkdsp.idct_add(dst, stride, dctblock);
break; break;
case PATTERN_BLOCK: case PATTERN_BLOCK:
...@@ -1169,8 +1161,8 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb, ...@@ -1169,8 +1161,8 @@ static int bink_decode_plane(BinkContext *c, AVFrame *frame, GetBitContext *gb,
} }
} }
} }
if (get_bits_count(gb) & 0x1F) //next plane data starts at 32-bit boundary if (bitstream_tell(bc) & 0x1F) // next plane data starts at 32-bit boundary
skip_bits_long(gb, 32 - (get_bits_count(gb) & 0x1F)); bitstream_skip(bc, 32 - (bitstream_tell(bc) & 0x1F));
return 0; return 0;
} }
...@@ -1179,7 +1171,7 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, AVPac ...@@ -1179,7 +1171,7 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, AVPac
{ {
BinkContext * const c = avctx->priv_data; BinkContext * const c = avctx->priv_data;
AVFrame *frame = data; AVFrame *frame = data;
GetBitContext gb; BitstreamContext bc;
int plane, plane_idx, ret; int plane, plane_idx, ret;
int bits_count = pkt->size << 3; int bits_count = pkt->size << 3;
...@@ -1197,28 +1189,28 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, AVPac ...@@ -1197,28 +1189,28 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *got_frame, AVPac
return ret; return ret;
} }
init_get_bits(&gb, pkt->data, bits_count); bitstream_init(&bc, pkt->data, bits_count);
if (c->has_alpha) { if (c->has_alpha) {
if (c->version >= 'i') if (c->version >= 'i')
skip_bits_long(&gb, 32); bitstream_skip(&bc, 32);
if ((ret = bink_decode_plane(c, frame, &gb, 3, 0)) < 0) if ((ret = bink_decode_plane(c, frame, &bc, 3, 0)) < 0)
return ret; return ret;
} }
if (c->version >= 'i') if (c->version >= 'i')
skip_bits_long(&gb, 32); bitstream_skip(&bc, 32);
for (plane = 0; plane < 3; plane++) { for (plane = 0; plane < 3; plane++) {
plane_idx = (!plane || !c->swap_planes) ? plane : (plane ^ 3); plane_idx = (!plane || !c->swap_planes) ? plane : (plane ^ 3);
if (c->version > 'b') { if (c->version > 'b') {
if ((ret = bink_decode_plane(c, frame, &gb, plane_idx, !!plane)) < 0) if ((ret = bink_decode_plane(c, frame, &bc, plane_idx, !!plane)) < 0)
return ret; return ret;
} else { } else {
if ((ret = binkb_decode_plane(c, frame, &gb, plane_idx, if ((ret = binkb_decode_plane(c, frame, &bc, plane_idx,
!avctx->frame_number, !!plane)) < 0) !avctx->frame_number, !!plane)) < 0)
return ret; return ret;
} }
if (get_bits_count(&gb) >= bits_count) if (bitstream_tell(&bc) >= bits_count)
break; break;
} }
emms_c(); emms_c();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment