Unify coding order of MC filters between blocks and frame header. In VP9, the order for frame header was: [0] smooth, [1] regular, [2] sharp, [3] bilinear. Per-block, the order was [0] regular, [1] smooth and [2] sharp. For VP10, swap smooth/regular in the frame header so that the block ordering and frame header ordering are interchangeable. See issue #1046. Change-Id: Ic9ec5964874375e40cd59bef50b489a76cbe4365
diff --git a/vp10/decoder/decodeframe.c b/vp10/decoder/decodeframe.c index 1efede2..518dec0 100644 --- a/vp10/decoder/decodeframe.c +++ b/vp10/decoder/decodeframe.c
@@ -1139,12 +1139,7 @@ } static INTERP_FILTER read_interp_filter(struct vpx_read_bit_buffer *rb) { - const INTERP_FILTER literal_to_filter[] = { EIGHTTAP_SMOOTH, - EIGHTTAP, - EIGHTTAP_SHARP, - BILINEAR }; - return vpx_rb_read_bit(rb) ? SWITCHABLE - : literal_to_filter[vpx_rb_read_literal(rb, 2)]; + return vpx_rb_read_bit(rb) ? SWITCHABLE : vpx_rb_read_literal(rb, 2); } static void setup_display_size(VP10_COMMON *cm,
diff --git a/vp10/encoder/bitstream.c b/vp10/encoder/bitstream.c index 4ce3531..356bc16 100644 --- a/vp10/encoder/bitstream.c +++ b/vp10/encoder/bitstream.c
@@ -852,11 +852,9 @@ static void write_interp_filter(INTERP_FILTER filter, struct vpx_write_bit_buffer *wb) { - const int filter_to_literal[] = { 1, 0, 2, 3 }; - vpx_wb_write_bit(wb, filter == SWITCHABLE); if (filter != SWITCHABLE) - vpx_wb_write_literal(wb, filter_to_literal[filter], 2); + vpx_wb_write_literal(wb, filter, 2); } static void fix_interp_filter(VP10_COMMON *cm, FRAME_COUNTS *counts) {