commit c075e5578a61adbb9fde2a1c5589a27c461cc879
author Wan-Teh Chang <wtc@google.com> Mon Feb 08 11:45:32 2021 -0800
committer Wan-Teh Chang <wtc@google.com> Tue Feb 09 02:28:41 2021 +0000
tree 346306c28332cbc616a099a143ed779e1601463b
parent 886b981ef516508a330346c6b756dc0f9a775ac4

Remove TODO comment on cpi check in encoder_encode

The cpi != NULL check that the comment refers to has been removed. See
the corresponding libvpx CL:
https://chromium-review.googlesource.com/c/webm/libvpx/+/327699

Change-Id: I41054aaadce0993b1b098b3960e940ce24af3346

av1/av1_cx_iface.c

diff --git a/av1/av1_cx_iface.c b/av1/av1_cx_iface.c
index c48390ab..9ca6be4 100644
--- a/av1/av1_cx_iface.c
+++ b/av1/av1_cx_iface.c
@@ -2128,8 +2128,6 @@
 
   if (img != NULL) {
     res = validate_img(ctx, img);
-    // TODO(jzern) the checks related to cpi's validity should be treated as a
-    // failure condition, encoder setup is done fully in init() currently.
     if (res == AOM_CODEC_OK) {
       size_t data_sz = ALIGN_POWER_OF_TWO(ctx->cfg.g_w, 5) *
                        ALIGN_POWER_OF_TWO(ctx->cfg.g_h, 5) * get_image_bps(img);