Skip to content

Use libaom AOMD_SET_FRAME_SIZE_LIMIT if available#3062

Draft
wantehchang wants to merge 2 commits intoAOMediaCodec:mainfrom
wantehchang:libaom-set-frame-size-limit
Draft

Use libaom AOMD_SET_FRAME_SIZE_LIMIT if available#3062
wantehchang wants to merge 2 commits intoAOMediaCodec:mainfrom
wantehchang:libaom-set-frame-size-limit

Conversation

@wantehchang
Copy link
Collaborator

@wantehchang wantehchang requested a review from y-guyon February 26, 2026 01:49
@wantehchang
Copy link
Collaborator Author

Yannis: I marked this PR as a draft. I will wait until libaom v3.14.0 is released to merge this PR.

@wantehchang wantehchang force-pushed the libaom-set-frame-size-limit branch from 6a45411 to 7b8eed8 Compare February 26, 2026 20:05

aom_codec_iface_t * const decoderInterface = aom_codec_av1_dx();
#if !defined(AOM_CTRL_AOMD_SET_FRAME_SIZE_LIMIT)
// The AOMD_SET_FRAME_SIZE_LIMIT codec control (added in libaom v3.14.0),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leftover comma

aom_codec_iface_t * const decoderInterface = aom_codec_av1_dx();
#if !defined(AOM_CTRL_AOMD_SET_FRAME_SIZE_LIMIT)
// The AOMD_SET_FRAME_SIZE_LIMIT codec control (added in libaom v3.14.0),
// is the best way to impose a maximum on AV1 frame size. When
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"best" is vague. Is there a more accurate adjective?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants