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

feat: import and play m4a #35

Merged
merged 5 commits into from
Dec 22, 2024
Merged

feat: import and play m4a #35

merged 5 commits into from
Dec 22, 2024

Conversation

daiyam
Copy link
Contributor

@daiyam daiyam commented Dec 11, 2024

This PR imports and plays M4A audio file.

M4A are quite tricky since we need to read the first buffer to get the channels configuration.

References:

@daiyam daiyam changed the title feat: import and play m4a [WIP] feat: import and play m4a Dec 11, 2024
@daiyam
Copy link
Contributor Author

daiyam commented Dec 11, 2024

It doesn't play the next song, yet...

@daiyam daiyam changed the title [WIP] feat: import and play m4a feat: import and play m4a Dec 11, 2024
@daiyam
Copy link
Contributor Author

daiyam commented Dec 12, 2024

The tags are missing.

@daiyam
Copy link
Contributor Author

daiyam commented Dec 13, 2024

The tags can't be written but I also have the issue with MP3 so that issue is outside the current PR.

@basharovV
Copy link
Owner

Thanks! I'll merge this, but note that there's an issue in Symphonia which causes some m4a files to fail to read with Mp4: No "moov" atom found error, see pdeljanov/Symphonia#289. The same files work fine in other apps.

@basharovV basharovV merged commit cd27ef0 into basharovV:main Dec 22, 2024
@daiyam daiyam deleted the feat-m4a branch December 22, 2024 19:48
@daiyam
Copy link
Contributor Author

daiyam commented Dec 22, 2024

So far, no issue with my m4a files.

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