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

Support for MP3 files < MAX_FRAME_SYNC_MATCHES #125

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

misyltoad
Copy link

If we match > 1 frame, and the next frame header is all NULL or a TAG then say that we matched a frame, as we likely reached the end of the file.

Fixes: #102

misyltoad added a commit to misyltoad/VAudio-MiniMP3 that referenced this pull request Aug 9, 2024
If we match > 1 frame, and the next frame header is all NULL or a TAG then say that we matched a frame, as we likely reached the end of the file.

lieff/minimp3#125
@manxorist
Copy link
Contributor

This makes the tests fail:

manx@sagnix:~/projects/minimp3/minimp3.git$ ./scripts/test.sh
~/projects/minimp3/minimp3.git ~/projects/minimp3/minimp3.git
rate=48000 samples=2304 max_diff=0 PSNR=99.000000
vectors/ILL2_center2.bit exited with code=0
rate=48000 samples=2304 max_diff=0 PSNR=99.000000
vectors/ILL2_dual.bit exited with code=0
rate=48000 samples=2304 max_diff=0 PSNR=99.000000
vectors/ILL2_dynx22.bit exited with code=0
rate=48000 samples=2304 max_diff=0 PSNR=99.000000
vectors/ILL2_dynx31.bit exited with code=0
rate=48000 samples=2304 max_diff=0 PSNR=99.000000
vectors/ILL2_dynx32.bit exited with code=0
error: reference and produced number of samples do not match (20736/0)
vectors/ILL2_ext_switching.bit exited with code=1

I have not looked into why that is.

If we match > 1 frame, and the next frame header is all NULL or a TAG then say that we matched a frame, as we likely reached the end of the file.

Fixes: lieff#102
manxorist added a commit to OpenMPT/openmpt that referenced this pull request Aug 15, 2024
…eleases/tag/openmpt-2024-08-15-v3> commit 2811a29e4115199209fe91ae5217c9c5fc611fa6 (2024-08-15). This applies the following pull requests: <lieff/minimp3#125>.

git-svn-id: https://source.openmpt.org/svn/openmpt/trunk/OpenMPT@21446 56274372-70c3-4bfc-bfc3-4c3a0b034d27
manxorist added a commit to OpenMPT/openmpt that referenced this pull request Aug 15, 2024
[Var] minimp3: Update to fork <https://github.com/manxorist/minimp3/releases/tag/openmpt-2024-08-15-v3> commit 2811a29e4115199209fe91ae5217c9c5fc611fa6 (2024-08-15). This applies the following pull requests: <lieff/minimp3#125>.
........


git-svn-id: https://source.openmpt.org/svn/openmpt/branches/OpenMPT-1.31@21447 56274372-70c3-4bfc-bfc3-4c3a0b034d27
@misyltoad
Copy link
Author

curious as to if it is fixed on what I just pushed today? i had a typo

@manxorist
Copy link
Contributor

curious as to if it is fixed on what I just pushed today? i had a typo

Yes, the tests pass now.

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.

Issue with very short mp3 files
2 participants