8000 Add seek function for DefaultExtractorInput · Issue #2381 · androidx/media · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add seek function for DefaultExtractorInput #2381

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.

Sign up for GitHub

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

Jump to bottom
Closed
tcysteph opened this issue Apr 29, 2025 · 2 comments
Closed

Add seek function for DefaultExtractorInput #2381

tcysteph opened this issue Apr 29, 2025 · 2 comments
Assignees
@rohitjoins
Labels
enhancement needs triage

Comments

@tcysteph
Copy link

Before filing a feature request:

When filing a feature request:

Replace the content in the sections below.

[REQUIRED] Use case description

When using DefaultExtractorInput, there is no seek function to help advance the position.
When trying to use Mp4Extractor to read through the data and advance the position in the extractorinput, there is no corresponding seek function to advance the position.

Proposed solution

Implement the seek function to advance the position.

Alternatives considered
@rohitjoins rohitjoins self-assigned this Apr 30, 2025
@rohitjoins
Copy link
Contributor

Hi @tcysteph,

The ExtractorInput interface is designed for sequential reading using methods like read and skip. The seek() functionality resides within the Extractor interface itself. This is because the Extractor is responsible for understanding the media structure and maintaining the necessary mapping (like time to byte position) to enable seeking.

@icbaker
Copy link
Collaborator
icbaker commented Apr 30, 2025

When trying to use Mp4Extractor to read through the data and advance the position in the extractorinput, there is no corresponding seek function to advance the position.

I may be wrong, but this reads like it might be an XY Problem.

It might be worth filing a new 'question' issue with a higher-level overview of what you're trying to do - it may be there's a way we can help without adding the specific seek operation you're asking for (which I agree with @rohitjoins, I don't think makes sense as you've described it).

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

@rohitjoins rohitjoins

Labels
enhancement needs triage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@icbaker @tcysteph @rohitjoins
0