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.

Sign up for GitHub

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

Jump to bottom

(WIP) adding music audio support back into the game #299

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

(WIP) adding music audio support back into the game #299

wants to merge 3 commits into from

Conversation

chino
Copy link
Member

@chino chino commented Oct 1, 2022 •
edited
Loading

This is a clean rebase and starting place from: #288

Overall direction:

  • we could either port ONLY music to ogg (supporting two code paths)
  • or we could port all the wav files to ogg (supporting the single code path)

There is still a bit of work to do here:

  • determine if alut is needed or could be removed
  • add alut support to forsaken-libs
  • add ogg and vorbisfile support to forsaken-libs
  • possibly replace LoadLevel global with a simple call to load a new audio file?
  • test it on different platforms
  • check for any type of bugs, leaks, or general issues
  • find or generate the vorbis files from original cd audio
  • generate a music.dat file
  • would it be better to have music.dat contain a mapping of level names to music files...?

chino added 2 commits October 1, 2022 00:50
@chino
gitignore some files
b910cb1
@chino
update math error detection
8c7653c 
The old SVID (SysV) matherr() is being deprecated as of glibc 2.27.
- https://man7.org/linux/man-pages/man3/matherr.3.html

Updating non-win32 targets (except for MINGW) to use the new approach.
- https://man7.org/linux/man-pages/man7/math_error.7.html

For now leaving the old code as-is in case it's still required under
msvc.
@chino chino mentioned this pull request Oct 1, 2022
Closed
@sonarqubecloud SonarQubeCloud
Copy link

Please retry analysis of this Pull-Request directly on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@chino