commit b314e490af69c217b77c70b6a1e4ddbbf1f5d2c6
author Joe Drago <jdrago@netflix.com> Tue Sep 14 16:57:00 2021 -0700
committer Joe Drago <joedrago@gmail.com> Tue Sep 14 17:12:21 2021 -0700
tree 042d4eb7fdefc55b0f73a3cfc85f8a92e623e7bd
parent 2c1f62768f309144fa71e5a698de8a026a6b71dd

Rename avifParseMoovBox to avifParseMovieBox for consistency

The naming convention of `avifParse*()` appends the name of the box as defined in the standards,
with the exception of this function, which incorrectly used "MoovBox" when BMFF names the moov
box "MovieBox" (BMFF Section 8.2.1).

src/read.c

diff --git a/src/read.c b/src/read.c
index 3b38148..58cfddd 100644
--- a/src/read.c
+++ b/src/read.c
@@ -2683,7 +2683,7 @@
     return AVIF_TRUE;
 }
 
-static avifBool avifParseMoovBox(avifDecoderData * data, const uint8_t * raw, size_t rawLen, uint32_t imageSizeLimit)
+static avifBool avifParseMovieBox(avifDecoderData * data, const uint8_t * raw, size_t rawLen, uint32_t imageSizeLimit)
 {
     BEGIN_STREAM(s, raw, rawLen, data->diag, "Box[moov]");
 
@@ -2793,7 +2793,7 @@
             metaSeen = AVIF_TRUE;
         } else if (!memcmp(header.type, "moov", 4)) {
             CHECKERR(!moovSeen, AVIF_RESULT_BMFF_PARSE_FAILED);
-            CHECKERR(avifParseMoovBox(data, boxContents.data, boxContents.size, decoder->imageSizeLimit), AVIF_RESULT_BMFF_PARSE_FAILED);
+            CHECKERR(avifParseMovieBox(data, boxContents.data, boxContents.size, decoder->imageSizeLimit), AVIF_RESULT_BMFF_PARSE_FAILED);
             moovSeen = AVIF_TRUE;
         }