-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add missing FluidSynth checks, consistency tweak #2109
Conversation
|
So, do I make the change in this PR? |
Yes, please. |
Done in a720d33. |
Thank you! I can confirm that the engine now compiles and runs if fluidsynth is not installed at build time. However, could you please address these warnings as well:
|
Will do tomorrow. |
Done in 6957e20. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thank you!
Should fix this Doomworld report.
While we're on this, I noticed that the following check uses
#ifdef
instead of#if defined
:woof/src/i_oalmusic.c
Lines 612 to 617 in 1640988
Why? Should we change it or all the others for consistency?