commit c434716cb6cc174f1051b59fe82ad39af38f1e5d
author James Zern <jzern@google.com> Thu May 05 12:09:09 2022 -0700
committer James Zern <jzern@google.com> Wed May 11 02:10:09 2022 +0000
tree 1a62015330858fcecf3b4972c08eee7b44228d6c
parent 92c4bb0f195dc5525b9c2d66131e8d07d7c066de

aom_film_grain_table_lookup: clarify unchecked return

when erasing a segment ignoring the return value from
aom_film_grain_table_lookup is safe

Change-Id: I0898c11d66060674f2ba44dc2a078f21976ccce4

aom_dsp/grain_table.c

diff --git a/aom_dsp/grain_table.c b/aom_dsp/grain_table.c
index ad9bdb9..3505f9f 100644
--- a/aom_dsp/grain_table.c
+++ b/aom_dsp/grain_table.c
@@ -260,7 +260,10 @@
       // If segments aren't aligned, delete from the beginning of subsequent
       // segments
       if (end_time > entry_end_time) {
-        aom_film_grain_table_lookup(t, entry_end_time, end_time, 1, 0);
+        // Ignoring the return value here is safe since we're erasing from the
+        // beginning of subsequent entries.
+        aom_film_grain_table_lookup(t, entry_end_time, end_time, /*erase=*/1,
+                                    NULL);
       }
       return 1;
     }