Commit 0e37ded9 authored by jbr's avatar jbr

flacdec: cosmetics: comments and white space


git-svn-id: file:///var/local/repositories/ffmpeg/trunk@18150 9553f0bf-9b14-0410-a0b8-cfaf0461ba5b
parent c3d9ebce
...@@ -486,11 +486,15 @@ static int decode_frame(FLACContext *s, int alloc_data_size) ...@@ -486,11 +486,15 @@ static int decode_frame(FLACContext *s, int alloc_data_size)
int ch_mode, bps, blocksize, samplerate; int ch_mode, bps, blocksize, samplerate;
GetBitContext *gb = &s->gb; GetBitContext *gb = &s->gb;
bs_code = get_bits(gb, 4); /* frame sync code */
skip_bits(&s->gb, 16);
/* block size and sample rate codes */
bs_code = get_bits(gb, 4);
sr_code = get_bits(gb, 4); sr_code = get_bits(gb, 4);
ch_mode = get_bits(gb, 4); /* channel assignment */ /* channels and decorrelation */
ch_mode = get_bits(gb, 4);
if (ch_mode < FLAC_MAX_CHANNELS && s->channels == ch_mode+1) { if (ch_mode < FLAC_MAX_CHANNELS && s->channels == ch_mode+1) {
ch_mode = FLAC_CHMODE_INDEPENDENT; ch_mode = FLAC_CHMODE_INDEPENDENT;
} else if (ch_mode > FLAC_CHMODE_MID_SIDE || s->channels != 2) { } else if (ch_mode > FLAC_CHMODE_MID_SIDE || s->channels != 2) {
...@@ -499,6 +503,7 @@ static int decode_frame(FLACContext *s, int alloc_data_size) ...@@ -499,6 +503,7 @@ static int decode_frame(FLACContext *s, int alloc_data_size)
return -1; return -1;
} }
/* bits per sample */
bps_code = get_bits(gb, 3); bps_code = get_bits(gb, 3);
if (bps_code == 0) if (bps_code == 0)
bps= s->bps; bps= s->bps;
...@@ -520,16 +525,19 @@ static int decode_frame(FLACContext *s, int alloc_data_size) ...@@ -520,16 +525,19 @@ static int decode_frame(FLACContext *s, int alloc_data_size)
} }
s->bps = s->avctx->bits_per_raw_sample = bps; s->bps = s->avctx->bits_per_raw_sample = bps;
/* reserved bit */
if (get_bits1(gb)) { if (get_bits1(gb)) {
av_log(s->avctx, AV_LOG_ERROR, "broken stream, invalid padding\n"); av_log(s->avctx, AV_LOG_ERROR, "broken stream, invalid padding\n");
return -1; return -1;
} }
/* sample or frame count */
if (get_utf8(gb) < 0) { if (get_utf8(gb) < 0) {
av_log(s->avctx, AV_LOG_ERROR, "utf8 fscked\n"); av_log(s->avctx, AV_LOG_ERROR, "utf8 fscked\n");
return -1; return -1;
} }
/* blocksize */
if (bs_code == 0) { if (bs_code == 0) {
av_log(s->avctx, AV_LOG_ERROR, "reserved blocksize code: 0\n"); av_log(s->avctx, AV_LOG_ERROR, "reserved blocksize code: 0\n");
return -1; return -1;
...@@ -549,6 +557,7 @@ static int decode_frame(FLACContext *s, int alloc_data_size) ...@@ -549,6 +557,7 @@ static int decode_frame(FLACContext *s, int alloc_data_size)
if (blocksize * s->channels * (s->is32 ? 4 : 2) > alloc_data_size) if (blocksize * s->channels * (s->is32 ? 4 : 2) > alloc_data_size)
return -1; return -1;
/* sample rate */
if (sr_code == 0) if (sr_code == 0)
samplerate= s->samplerate; samplerate= s->samplerate;
else if (sr_code < 12) else if (sr_code < 12)
...@@ -565,6 +574,7 @@ static int decode_frame(FLACContext *s, int alloc_data_size) ...@@ -565,6 +574,7 @@ static int decode_frame(FLACContext *s, int alloc_data_size)
return -1; return -1;
} }
/* header CRC-8 check */
skip_bits(gb, 8); skip_bits(gb, 8);
if (av_crc(av_crc_get_table(AV_CRC_8_ATM), 0, gb->buffer, if (av_crc(av_crc_get_table(AV_CRC_8_ATM), 0, gb->buffer,
get_bits_count(gb)/8)) { get_bits_count(gb)/8)) {
...@@ -665,7 +675,6 @@ static int flac_decode_frame(AVCodecContext *avctx, ...@@ -665,7 +675,6 @@ static int flac_decode_frame(AVCodecContext *avctx,
/* decode frame */ /* decode frame */
init_get_bits(&s->gb, buf, buf_size*8); init_get_bits(&s->gb, buf, buf_size*8);
skip_bits(&s->gb, 16);
if (decode_frame(s, alloc_data_size) < 0) { if (decode_frame(s, alloc_data_size) < 0) {
av_log(s->avctx, AV_LOG_ERROR, "decode_frame() failed\n"); av_log(s->avctx, AV_LOG_ERROR, "decode_frame() failed\n");
s->bitstream_size=0; s->bitstream_size=0;
......
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