Commit ee0fd929 authored by David Barker's avatar David Barker
Browse files

Clarify comment in av1_set_mb_mi()

The code currently pads the decoded frame width and height to a
multiple of 8 luma pixels, but there is a TODO suggesting that
this may be changed to only require a multiple of 4 in future.

But, as per the comments on the linked bug, there are good reasons
to keep the decoded width and height as multiples of 8. So delete
the outdated TODO and instead outline the reasons why the current
behaviour is helpful.


Change-Id: I2340bbcd740afe74c2e6fb3cf2e7a420db2b4f40
parent d7150530
......@@ -20,8 +20,11 @@
#include "av1/common/onyxc_int.h"
void av1_set_mb_mi(AV1_COMMON *cm, int width, int height) {
// TODO(jingning): Fine tune the loop filter operations and bring this
// back to integer multiple of 4 for cb4x4.
// Ensure that the decoded width and height are both multiples of
// 8 luma pixels (note: this may only be a multiple of 4 chroma pixels if
// subsampling is used).
// This simplifies the implementation of various experiments,
// eg. cdef, which operates on units of 8x8 luma pixels.
const int aligned_width = ALIGN_POWER_OF_TWO(width, 3);
const int aligned_height = ALIGN_POWER_OF_TWO(height, 3);
Supports Markdown
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