Skip to content

Allow mid-stream starting of flac streams.#390

Open
GeoffClements wants to merge 3 commits intopdeljanov:dev-0.6from
GeoffClements:flac-skip
Open

Allow mid-stream starting of flac streams.#390
GeoffClements wants to merge 3 commits intopdeljanov:dev-0.6from
GeoffClements:flac-skip

Conversation

@GeoffClements
Copy link
Copy Markdown

@GeoffClements GeoffClements commented Jun 26, 2025

Related to issue: #372

This change affects symphonia-bundle-flac only. It adds the ability for Symphonia to create a FlacReader and FlacDecoder when presented with a stream that starts at a random position.

Normal stream starts, i.e. when the marker "fLaC" is present, are unaffected by this change.

A new branch, flac-skip has been created to isolate this change, this was branched off the dev-0.6 branch.

All checks are passing.

@sscobici
Copy link
Copy Markdown
Contributor

Please make PR into the dev branch

@GeoffClements GeoffClements changed the base branch from master to dev-0.6 June 26, 2025 11:27
@GeoffClements
Copy link
Copy Markdown
Author

GeoffClements commented Jun 26, 2025

Please make PR into the dev branch

Apologies, I'm a bit of a PR noob, this is my second ever.

@sscobici
Copy link
Copy Markdown
Contributor

No problem. You can still reduce the number of changed lines and name the commits accordingly by following the contributing guide. I don't remember exactly, but I think fmt should be used from the nightly version.
cargo +nightly fmt

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.

2 participants