Skip to content

Improve starboard::shared::starboard::media::AvcParameterSets class to handle optional parameters - #5326

Merged
jasonzhangxx merged 4 commits into
youtube:25.lts.1+from
iuriionishchenko:Improve_AvcParameterSets_to_add_optional_parameter
May 21, 2025
Merged

jasonzhangxx merged 4 commits into
youtube:25.lts.1+from
iuriionishchenko:Improve_AvcParameterSets_to_add_optional_parameter

Conversation

@iuriionishchenko

Copy link
Copy Markdown
Collaborator

Improve starboard::shared::starboard::media::AvcParameterSets class to handle optional parameters

b/392649786

Change-Id: Ib00588c227ce5b638cbc563e47a329092c19104c

@iuriionishchenko
iuriionishchenko force-pushed the Improve_AvcParameterSets_to_add_optional_parameter branch from 8febb79 to 7ff8686 Compare April 10, 2025 07:57
@iuriionishchenko
iuriionishchenko force-pushed the Improve_AvcParameterSets_to_add_optional_parameter branch from 7ff8686 to bb2d98b Compare April 15, 2025 11:23
@jasonzhangxx

Copy link
Copy Markdown
Contributor

Can you help to make a pr to apply the change to main branch?

@amurovanyi amurovanyi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@iuriionishchenko

Copy link
Copy Markdown
Collaborator Author

Can you help to make a pr to apply the change to main branch?

I have created new pr #5590

@iuriionishchenko
iuriionishchenko force-pushed the Improve_AvcParameterSets_to_add_optional_parameter branch from 6547efe to 60f892f Compare April 29, 2025 14:47
…o handle optional parameters

b/392649786

Change-Id: Ib00588c227ce5b638cbc563e47a329092c19104c
@iuriionishchenko
iuriionishchenko force-pushed the Improve_AvcParameterSets_to_add_optional_parameter branch from 60f892f to da37c4d Compare April 29, 2025 14:56
@iuriionishchenko

Copy link
Copy Markdown
Collaborator Author

There is one failing. But the failing is not related on my changes. I think it can be merged.

@jasonzhangxx
jasonzhangxx merged commit ec6f19e into youtube:25.lts.1+ May 21, 2025
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.

3 participants