commit c407123a2629863873c6285ee79129d5c803be0a
author Yaowu Xu <yaowu@google.com> Fri Mar 30 12:53:22 2018 -0700
committer Yaowu Xu <yaowu@google.com> Mon Apr 02 16:16:23 2018 +0000
tree 4a77e682b27300608a7e54f25b35e54634f63519
parent aaed33a9961e8c4d1d754e3c7712cf1568f4280f

Remove CONFIG_TRAILING_BITS

This tool is adopted.

Change-Id: I2eac0a5bcab50f8179f95b9d8fe7dca3d0102ddf

av1/decoder/decodeframe.c

diff --git a/av1/decoder/decodeframe.c b/av1/decoder/decodeframe.c
index 0c86e6d..cfb87d3 100644
--- a/av1/decoder/decodeframe.c
+++ b/av1/decoder/decodeframe.c
@@ -62,7 +62,6 @@
 #define MAX_AV1_HEADER_SIZE 80
 #define ACCT_STR __func__
 
-#if CONFIG_TRAILING_BITS
 // Checks that the remaining bits start with a 1 and ends with 0s.
 // It consumes an additional byte, if already byte aligned before the check.
 int av1_check_trailing_bits(AV1Decoder *pbi, struct aom_read_bit_buffer *rb) {
@@ -76,7 +75,6 @@
   }
   return 0;
 }
-#endif
 
 // Use only_chroma = 1 to only set the chroma planes
 static void set_planes_to_neutral_grey(AV1_COMMON *const cm,
@@ -1838,7 +1836,6 @@
     decode_tile_sb_row(pbi, td, tile_info, mi_row);
   }
 
-#if CONFIG_TRAILING_BITS
   uint32_t nb_bits = aom_reader_tell(&td->bit_reader);
   uint32_t nb_bytes = (nb_bits + 7) >> 3;
 
@@ -1857,7 +1854,6 @@
     if (*p != 0) cm->error.error_code = AOM_CODEC_CORRUPT_FRAME;
     p++;
   }
-#endif
 }
 
 static const uint8_t *decode_tiles(AV1Decoder *pbi, const uint8_t *data,
@@ -3129,13 +3125,8 @@
 }
 
 int av1_decode_frame_headers_and_setup(AV1Decoder *pbi,
-#if CONFIG_TRAILING_BITS
                                        struct aom_read_bit_buffer *rb,
-#endif
                                        const uint8_t *data,
-#if !CONFIG_TRAILING_BITS
-                                       const uint8_t *data_end,
-#endif
                                        const uint8_t **p_data_end) {
   AV1_COMMON *const cm = &pbi->common;
   const int num_planes = av1_num_planes(cm);
@@ -3154,19 +3145,9 @@
   }
   xd->global_motion = cm->global_motion;
 
-#if !CONFIG_TRAILING_BITS
-  struct aom_read_bit_buffer rb;
-#endif
-  read_uncompressed_header(pbi,
-#if CONFIG_TRAILING_BITS
-                           rb);
-#else
-                           av1_init_read_bit_buffer(pbi, &rb, data, data_end));
-#endif
+  read_uncompressed_header(pbi, rb);
 
-#if CONFIG_TRAILING_BITS
   av1_check_trailing_bits(pbi, rb);
-#endif
 
   // If cm->single_tile_decoding = 0, the independent decoding of a single tile
   // or a section of a frame is not allowed.
@@ -3176,11 +3157,7 @@
     pbi->dec_tile_col = -1;
   }
 
-#if CONFIG_TRAILING_BITS
   pbi->uncomp_hdr_size = aom_rb_bytes_read(rb);
-#else
-  pbi->uncomp_hdr_size = aom_rb_bytes_read(&rb);
-#endif
   YV12_BUFFER_CONFIG *new_fb = get_frame_new_buffer(cm);
   xd->cur_buf = new_fb;
   if (av1_allow_intrabc(cm)) {
@@ -3190,12 +3167,8 @@
   }
 
   if (cm->show_existing_frame) {
-  // showing a frame directly
-#if CONFIG_TRAILING_BITS
+    // showing a frame directly
     *p_data_end = data + aom_rb_bytes_read(rb);
-#else
-    *p_data_end = data + aom_rb_bytes_read(&rb);
-#endif
     if (cm->reset_decoder_state) {
       // Use the default frame context values.
       *cm->fc = cm->frame_contexts[FRAME_CONTEXT_DEFAULTS];