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

message printed while doing .ds file preview #6633

Closed
anytizer opened this issue Feb 7, 2023 · 1 comment · Fixed by #6610
Closed

message printed while doing .ds file preview #6633

anytizer opened this issue Feb 7, 2023 · 1 comment · Fixed by #6610
Labels

Comments

@anytizer
Copy link
Contributor

anytizer commented Feb 7, 2023

Open LMMS from terminal.
Navigate to My Samples > drumsynth.
Click on any .ds file to preview.

Following message appears in the terminal:

SampleBuffer::decodeSampleOGGVorbis(): not an Ogg Vorbis file

On every click on a .ds file, the message appears.

Call: SampleBuffer.cpp

It does not print such message for .ogg, .flac or .wav files but for .ds only. Application performs normally.

@anytizer anytizer added the bug label Feb 7, 2023
@sakertooth
Copy link
Contributor

This occurs due to the fact that when loading a SampleBuffer from an audio file, SampleBuffer tries to decode it using various decoders until it successfully loads (and if all the decoders it tried still couldn't load it, it bails and creates an empty buffer). This means that it will unnecessarily try to decode DrumSynth files using its OGG decoders, which would obviously fail. #6610 fixes this issue, as it only decodes the audio file using the DrumSynth decoder if the filename extension is .ds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants