Commit 0bd44664 authored by scherkus's avatar scherkus Committed by Commit bot

Increase DVLOG level of BitReader::SkipBits() from 0 to 1 and add TODO.

DVLOG(0) is way too spammy for such a low-level function.

BUG=376450
TBR=dalecurtis

Review URL: https://codereview.chromium.org/537653002

Cr-Commit-Position: refs/heads/master@{#293192}
parent 2d298bab
...@@ -52,8 +52,9 @@ int BitReaderCore::PeekBitsMsbAligned(int num_bits, uint64* out) { ...@@ -52,8 +52,9 @@ int BitReaderCore::PeekBitsMsbAligned(int num_bits, uint64* out) {
} }
bool BitReaderCore::SkipBits(int num_bits) { bool BitReaderCore::SkipBits(int num_bits) {
// TODO(dalecurtis): Rewrite to be efficient, see http://crbug.com/376450
DCHECK_GE(num_bits, 0); DCHECK_GE(num_bits, 0);
DVLOG_IF(0, num_bits > 100) DVLOG_IF(1, num_bits > 100)
<< "BitReader::SkipBits inefficient for large skips"; << "BitReader::SkipBits inefficient for large skips";
uint64 dummy; uint64 dummy;
......
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