From d39ceb58ac950ff512c7666b223ed613eacfc292 Mon Sep 17 00:00:00 2001
From: James Zern <jzern@google.com>
Date: Thu, 19 May 2016 23:07:10 -0700
Subject: [PATCH] VP8LDoFillBitWindow: remove stale TODO

the read size may be fixed, but the offsets into buf_ are not. forcing
an aligned read then shifting or using a temporary would be costly. this
is less important now that WebPMemToUint32() is being used.

Change-Id: I357fec8f750969cce91987abebed2f95e27a835f
---
 src/utils/bit_reader.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/utils/bit_reader.c b/src/utils/bit_reader.c
index 3c1b8815..453fe966 100644
--- a/src/utils/bit_reader.c
+++ b/src/utils/bit_reader.c
@@ -192,8 +192,6 @@ static void ShiftBytes(VP8LBitReader* const br) {
 
 void VP8LDoFillBitWindow(VP8LBitReader* const br) {
   assert(br->bit_pos_ >= VP8L_WBITS);
-  // TODO(jzern): given the fixed read size it may be possible to force
-  //              alignment in this block.
 #if defined(VP8L_USE_UNALIGNED_LOAD)
   if (br->pos_ + sizeof(br->val_) < br->len_) {
     br->val_ >>= VP8L_WBITS;