Skip to content

Move equal_dimensions_and_border to yv12config.h#5032

Open
Arjit201 wants to merge 1 commit into
AOMediaCodec:mainfrom
Arjit201:fix-equal-dimensions-border-duplication
Open

Move equal_dimensions_and_border to yv12config.h#5032
Arjit201 wants to merge 1 commit into
AOMediaCodec:mainfrom
Arjit201:fix-equal-dimensions-border-duplication

Conversation

@Arjit201
Copy link
Copy Markdown

@Arjit201 Arjit201 commented Jun 4, 2026

Summary

Move equal_dimensions_and_border() to avm_scale/yv12config.h and remove the duplicate implementations from:

  • av2/decoder/decoder.c
  • av2/encoder/encoder_utils.h

Closes #5030

This consolidates the helper into a single shared definition associated with YV12_BUFFER_CONFIG and eliminates duplicate code.

Testing

  • Verified only a single definition remains via git grep.
  • Existing call sites continue to use the shared helper.

@Arjit201 Arjit201 force-pushed the fix-equal-dimensions-border-duplication branch from a54d5ad to 97cd13f Compare June 4, 2026 14:55
@urvangjoshi
Copy link
Copy Markdown
Contributor

@Arjit201 : Thanks for your first contribution here! Pls sign Contributor licence agreement (CLA) here: https://aomedia.org/about/legal/
Let me know when you have done so.

@urvangjoshi
Copy link
Copy Markdown
Contributor

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

equal_dimensions_and_border() is defined in both av2/decoder/decoder.c and av2/encoder/encoder_utils.h

3 participants