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

Refactor Player.Listener and update metadata #949

Merged
merged 5 commits into from
Sep 4, 2024

Conversation

ne0rrmatrix
Copy link
Contributor

@ne0rrmatrix ne0rrmatrix commented Aug 28, 2024

Support Libraries Version (eg: 23.3.0):

Does this change any of the generated binding API's?

No.

Describe your contribution

- Removed `onCues` method in `Player.Listener`.
- Removed `onPositionDiscontinuity` method from `Player.Listener`.
- Added `<remove-node>` tags to address issues with `Media3.ExoPlayer`.
- Made all methods in `Player.Listener` interface abstract.
… depracted method and not the one that is not deprecated.
@ne0rrmatrix ne0rrmatrix requested a review from jpobst August 28, 2024 22:27
@jpobst
Copy link
Contributor

jpobst commented Aug 28, 2024

/azp run

Copy link

No pipelines are associated with this pull request.

@moljac
Copy link
Contributor

moljac commented Sep 2, 2024

/azp run

Copy link

No pipelines are associated with this pull request.

Copy link
Contributor

@moljac moljac left a comment

Choose a reason for hiding this comment

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

LGTM

@moljac moljac merged commit 5b769b2 into dotnet:main Sep 4, 2024
1 check passed
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.

Cannot replace a media item or seek in Exoplayer when adding a listener
3 participants