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

Always print Timidity as supported decoder if built #82

Open
ericoporto opened this issue Mar 8, 2023 · 0 comments
Open

Always print Timidity as supported decoder if built #82

ericoporto opened this issue Mar 8, 2023 · 0 comments

Comments

@ericoporto
Copy link
Contributor

ericoporto commented Mar 8, 2023

When Timidity is included with the build, it should always print

Supported sound decoders:
- MIDI decoder, using a subset of TiMidity : MIDI,MID,

Currently if either the timidity.cfg is not found or it has something wrong in it or in the pointed soundfont, then this line doesn't show up. But this line is the best diagnostic we have that SDL2_Sound was indeed built with Timidity support.


Not needed, but this would be perfect (I want a pony):

  • in release config, when loading timidity.cfg it should log information about all steps in an internal buffer, and once an attempt is made to read a Midi file for the first time, if timidity was not correctly initialized, it should print to the log the previous accumulated internal log buffer lines and then this buffer should be emptied. (without Reading timidity soundfont from SDL_rwops #69, we don't own the logic to finding things and it's hard to debug what's going on when it's on someone's else machine)
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

No branches or pull requests

1 participant