# HG changeset patch # User reimar # Date 1254343679 0 # Node ID 5462710f17ee5c6cbb3e9584d336ab192b4675b4 # Parent d4c97368f3e4cf153641f77b7117f4eec46091f4 Do not needlessly add 7 to put_buts_count before dividing by 8, flush_put_bits ensures it is divisible 8. diff -r d4c97368f3e4 -r 5462710f17ee vorbis_enc.c --- a/vorbis_enc.c Wed Sep 30 15:13:09 2009 +0000 +++ b/vorbis_enc.c Wed Sep 30 20:47:59 2009 +0000 @@ -542,7 +542,7 @@ put_bits(&pb, 1, 1); // framing flush_put_bits(&pb); - hlens[0] = (put_bits_count(&pb) + 7) / 8; + hlens[0] = put_bits_count(&pb) >> 3; buffer_len -= hlens[0]; p += hlens[0]; @@ -555,7 +555,7 @@ put_bits(&pb, 1, 1); // framing flush_put_bits(&pb); - hlens[1] = (put_bits_count(&pb) + 7) / 8; + hlens[1] = put_bits_count(&pb) >> 3; buffer_len -= hlens[1]; p += hlens[1]; @@ -628,7 +628,7 @@ put_bits(&pb, 1, 1); // framing flush_put_bits(&pb); - hlens[2] = (put_bits_count(&pb) + 7) / 8; + hlens[2] = put_bits_count(&pb) >> 3; len = hlens[0] + hlens[1] + hlens[2]; p = *out = av_mallocz(64 + len + len/255); @@ -1023,7 +1023,7 @@ avccontext->coded_frame->pts = venc->sample_count; venc->sample_count += avccontext->frame_size; flush_put_bits(&pb); - return (put_bits_count(&pb) + 7) / 8; + return put_bits_count(&pb) >> 3; }