Commit d526d0a8 authored by Nathan E. Egge's avatar Nathan E. Egge

Fix TestSuperframeIndexIsOptional to work with daala_ec.

The superframe index is also mandatory with daala_ec for the same
 reason it is mandatory with ans, e.g., with both entropy coder
 backends the buffer length is not known when calling aom_reader_init(),
 see commit 15c1aa60.
This patch disables the same tests for CONFIG_DAALA_EC that are disabled
 when CONFIG_ANS is enabled with CONFIG_EXT_TILE.

BUG=aomedia:116

Change-Id: Icc89a01492296d56abfb966093a3c06f2f228650
parent 4d44f5ab
......@@ -112,11 +112,11 @@ TEST_P(SuperframeTest, TestSuperframeIndexIsOptional) {
#endif // CONFIG_EXT_REFS
}
// The superframe index is currently mandatory with ANS due to the decoder
// starting at the end of the buffer.
// The superframe index is currently mandatory with both ANS and DAALA_EC due
// to the decoder starting at the end of the buffer.
#if CONFIG_EXT_TILE
// Single tile does not work with ANS (see comment above).
#if CONFIG_ANS
#if CONFIG_ANS || CONFIG_DAALA_EC
const int tile_col_values[] = { 1, 2 };
#else
const int tile_col_values[] = { 1, 2, 32 };
......
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