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

typo: Bass is not the same as Base in the AudioPlayback context #961

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

Conversation

barbeque-squared
Copy link
Member

I delete* some of the bundled mp3's in my local build. I'm used to the error in the logs and never really paid attention to them, but with the recent removal of Bass for the audio decoding on Windows (#955), I started wondering why my Linux logs were referring to Bass of all things in the first place:

ERROR:  Audio-file does not exist: "/usr/share/ultrastardx/sounds/null.mp3" [UAudio_FFmpeg]
STATUS: Could not open "/usr/share/ultrastardx/sounds/null.mp3" [TAudioPlayback_Bass.OpenStream]

TAudioPlayback_Bass.OpenStream

Turns out it's just a typo. A very confusing one because both UAudioPlaybackBase.pas and UAudioPlayback_Bass.pas are actual files.

* = I actually rename stuff like applause.mp3 to null.mp3 in UMusic TSoundLibrary.LoadSounds, but the result is the same

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.

1 participant