minor asserts
authorJosh Coalson <jcoalson@users.sourceforce.net>
Sat, 3 Feb 2007 02:54:10 +0000 (02:54 +0000)
committerJosh Coalson <jcoalson@users.sourceforce.net>
Sat, 3 Feb 2007 02:54:10 +0000 (02:54 +0000)
src/libFLAC/bitreader.c
src/libFLAC/bitwriter.c

index 6c2982e..f3a65d9 100644 (file)
@@ -374,10 +374,6 @@ FLAC__uint16 FLAC__bitreader_get_read_crc16(FLAC__BitReader *br)
        /* CRC any tail bytes in a partially-consumed word */
        if(br->consumed_bits) {
                const brword tail = br->buffer[br->consumed_words];
-#ifdef DEBUG
-if(br->crc16_align)fprintf(stderr,"@@@@@@ FLAC__bitreader_get_read_crc16() got nonzero crc align = %u\n",br->crc16_align);
-#endif
-               /* non-zero crc align here can probably never happen with FLAC but check for consistency */
                for( ; br->crc16_align < br->consumed_bits; br->crc16_align += 8)
                        br->read_crc16 = FLAC__CRC16_UPDATE((unsigned)((tail >> (FLAC__BITS_PER_WORD-8-br->crc16_align)) & 0xff), br->read_crc16);
        }
index 28d2675..e9962e9 100644 (file)
@@ -347,9 +347,6 @@ FLaC__INLINE FLAC__bool FLAC__bitwriter_write_raw_uint32(FLAC__BitWriter *bw, FL
                bw->bits += bits;
        }
        else if(bw->bits) { /* WATCHOUT: if bw->bits == 0, left==FLAC__BITS_PER_WORD and bw->accum<<=left is a NOP instead of setting to 0 */
-#ifdef DEBUG
-if(left>=FLAC__BITS_PER_WORD)fprintf(stderr,"@@@@@@ bitwriter shift error @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
-#endif
                bw->accum <<= left;
                bw->accum |= val >> (bw->bits = bits - left);
                bw->buffer[bw->words++] = SWAP_BE_WORD_TO_HOST(bw->accum);
@@ -357,7 +354,7 @@ if(left>=FLAC__BITS_PER_WORD)fprintf(stderr,"@@@@@@ bitwriter shift error @@@@@@
        }
        else {
 #ifdef DEBUG
-if(bits!=left)fprintf(stderr,"@@@@@@ bitwriter shift error2 @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
+if(bits!=left)fprintf(stderr,"@@@@@@ bitwriter error2 @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
 #endif
                bw->accum = val;
                bw->bits = 0;
@@ -596,9 +593,7 @@ break1:
                         * triggered the (lsbits<left) case above.
                         */
                        FLAC__ASSERT(bw->bits);
-#ifdef DEBUG
-if(left>=FLAC__BITS_PER_WORD)fprintf(stderr,"@@@@@@ bitwriter shift error @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
-#endif
+                       FLAC__ASSERT(left < FLAC__BITS_PER_WORD);
                        bw->accum <<= left;
                        bw->accum |= uval >> (bw->bits = lsbits - left);
                        bw->buffer[bw->words++] = SWAP_BE_WORD_TO_HOST(bw->accum);