Skip to content

Commit

Permalink
Format comments
Browse files Browse the repository at this point in the history
  • Loading branch information
y-guyon committed Aug 11, 2023
1 parent 127bd7e commit 7ac605a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
14 changes: 7 additions & 7 deletions src/read.c
Original file line number Diff line number Diff line change
Expand Up @@ -1862,7 +1862,7 @@ static avifBool avifParseContentLightLevelInformationBox(avifProperty * prop, co
return AVIF_TRUE;
}

// Implementation of section 2.3.3. of AV1 Codec ISO Media File Format Binding specification v1.2.0.
// Implementation of section 2.3.3 of AV1 Codec ISO Media File Format Binding specification v1.2.0.
// See https://aomediacodec.github.io/av1-isobmff/v1.2.0.html#av1codecconfigurationbox-syntax.
static avifBool avifParseCodecConfiguration(avifROStream * s, avifCodecConfigurationBox * config, const char * configPropName, avifDiagnostics * diag)
{
Expand Down Expand Up @@ -1898,12 +1898,12 @@ static avifBool avifParseCodecConfiguration(avifROStream * s, avifCodecConfigura
// }
AVIF_CHECK(avifROStreamSkip(s, 1));

// According to section 2.2.1. of AV1 Image File Format specification v1.1.0:
// "- Sequence Header OBUs should not be present in the AV1CodecConfigurationBox."
// "- If a Sequence Header OBU is present in the AV1CodecConfigurationBox,
// it shall match the Sequence Header OBU in the AV1 Image Item Data."
// "- Metadata OBUs, if present, shall match the values given in other item properties,
// such as the PixelInformationProperty or ColourInformationBox."
// According to section 2.2.1 of AV1 Image File Format specification v1.1.0:
// - Sequence Header OBUs should not be present in the AV1CodecConfigurationBox.
// - If a Sequence Header OBU is present in the AV1CodecConfigurationBox,
// it shall match the Sequence Header OBU in the AV1 Image Item Data.
// - Metadata OBUs, if present, shall match the values given in other item properties,
// such as the PixelInformationProperty or ColourInformationBox.
// See https://aomediacodec.github.io/av1-avif/v1.1.0.html#av1-configuration-item-property.
// For simplicity, the constraints above are not enforced.
// The following is skipped by avifParseItemPropertyContainerBox().
Expand Down
4 changes: 2 additions & 2 deletions src/write.c
Original file line number Diff line number Diff line change
Expand Up @@ -2190,7 +2190,7 @@ avifResult avifEncoderWrite(avifEncoder * encoder, const avifImage * image, avif
return avifEncoderFinish(encoder, output);
}

// Implementation of section 2.3.3. of AV1 Codec ISO Media File Format Binding specification v1.2.0.
// Implementation of section 2.3.3 of AV1 Codec ISO Media File Format Binding specification v1.2.0.
// See https://aomediacodec.github.io/av1-isobmff/v1.2.0.html#av1codecconfigurationbox-syntax.
static void writeCodecConfig(avifRWStream * s, const avifCodecConfigurationBox * cfg)
{
Expand Down Expand Up @@ -2221,7 +2221,7 @@ static void writeCodecConfig(avifRWStream * s, const avifCodecConfigurationBox *
// }
avifRWStreamWriteU8(s, 0);

// According to section 2.2.1. of AV1 Image File Format specification v1.1.0,
// According to section 2.2.1 of AV1 Image File Format specification v1.1.0,
// there is no need to write any OBU here.
// See https://aomediacodec.github.io/av1-avif/v1.1.0.html#av1-configuration-item-property.
// unsigned int (8) configOBUs[];
Expand Down

0 comments on commit 7ac605a

Please sign in to comment.