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.
BUG=aomedia:727
Change-Id: I2340bbcd740afe74c2e6fb3cf2e7a420db2b4f40
diff --git a/av1/common/alloccommon.c b/av1/common/alloccommon.c
index 0db9ff6..c878939 100644
--- a/av1/common/alloccommon.c
+++ b/av1/common/alloccommon.c
@@ -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);
