Skip to content

Commit fa8db07

Browse files
committed
Move a comment about still image encoding
That comment describes the (addImageFlags & AVIF_ADD_IMAGE_FLAG_SINGLE) check, which was moved in AOMediaCodec#2970.
1 parent d245b00 commit fa8db07

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

src/codec_aom.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -672,6 +672,7 @@ static avifResult aomCodecEncodeImage(avifCodec * codec,
672672
// - Option of choosing tune IQ (which requires AOM_USAGE_ALL_INTRA)
673673
avifBool useAllIntraForLayered = encoder->extraLayerCount == 1 &&
674674
codec->internal->qualityFirstLayer <= TWO_LAYER_ALL_INTRA_QUALITY_THRESHOLD;
675+
// Also use all-intra encoding when encoding still images.
675676
avifBool useAllIntra = (addImageFlags & AVIF_ADD_IMAGE_FLAG_SINGLE) || useAllIntraForLayered;
676677

677678
// Map encoder speed to AOM usage + CpuUsed:
@@ -687,7 +688,7 @@ static avifResult aomCodecEncodeImage(avifCodec * codec,
687688
// Speed 9: RealTime CpuUsed 9
688689
// Speed 10: RealTime CpuUsed 9
689690
unsigned int aomUsage = AOM_USAGE_GOOD_QUALITY;
690-
// Use AOM_USAGE_ALL_INTRA (added in https://crbug.com/aomedia/2959) for still image encoding if available.
691+
// Use AOM_USAGE_ALL_INTRA (added in https://crbug.com/aomedia/2959) if available.
691692
#if defined(AOM_USAGE_ALL_INTRA)
692693
if (useAllIntra) {
693694
aomUsage = AOM_USAGE_ALL_INTRA;

0 commit comments

Comments
 (0)