Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[media] Minor clean up without functional changes #4197

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

xiaomings
Copy link
Contributor

@xiaomings xiaomings commented Oct 3, 2024

We are working on tool to snapshot media implementation in #4056 to allow a media implementation snapshot to run side by side with the default media implementation on Android TV.

The change in drm_system.cc fixes a link conflict when the snapshot and the default media implementation both have the operator== defined in global namespace.

The change in player_write_samples.cc reduces the complexity of parsing function prototype in the tool.

b/327287075

@xiaomings xiaomings enabled auto-merge (squash) October 3, 2024 00:31
We are working on tool to snapshot media implementation in
  youtube#4056
to allow a media implementation snapshot to run side by side with the
default media implementation on Android TV.

The change in drm_system.cc fixes a link conflict when the snapshot and
the default media implementation both have the operator== defined in
global namespace.

The change in player_write_samples.cc reduces the complexity of parsing
function prototype in the tool.

b/327287075
@kaidokert kaidokert merged commit 9209ea1 into youtube:25.lts.1+ Oct 7, 2024
272 of 301 checks passed
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.

3 participants