Skip to content

Replace OpenAL with mojoAL#608

Merged
jlnr merged 10 commits intogosu:masterfrom
cyberarm:replace_openal_with_mojoal
Jan 20, 2022
Merged

Replace OpenAL with mojoAL#608
jlnr merged 10 commits intogosu:masterfrom
cyberarm:replace_openal_with_mojoal

Conversation

@cyberarm
Copy link
Copy Markdown
Collaborator

No description provided.

@cyberarm cyberarm force-pushed the replace_openal_with_mojoal branch from 91cc173 to 4720d3d Compare January 17, 2022 18:28
@cyberarm
Copy link
Copy Markdown
Collaborator Author

@jlnr Do you want podspec updated to use mojoAL or leave that be for now?

@jlnr
Copy link
Copy Markdown
Member

jlnr commented Jan 17, 2022

Yes please, let's replace it all :)

@cyberarm cyberarm force-pushed the replace_openal_with_mojoal branch 2 times, most recently from 0ca6638 to 9d9da16 Compare January 17, 2022 23:48
@cyberarm cyberarm marked this pull request as ready for review January 17, 2022 23:52
@cyberarm
Copy link
Copy Markdown
Collaborator Author

@jlnr I got all the CI passing. iOS and OSX should probably get manually checked to ensure they work as expected.
I tested sound on Windows and Linux, however I am unable to test iOS and OSX.

@cyberarm
Copy link
Copy Markdown
Collaborator Author

iOS still uses OpenAL since mojoAL relies on SDL and Gosu's iOS implementation doesn't use SDL as far as I know.

@cyberarm cyberarm force-pushed the replace_openal_with_mojoal branch from 896f47e to 50b1c9b Compare January 18, 2022 14:29
@cyberarm cyberarm force-pushed the replace_openal_with_mojoal branch from 50b1c9b to efd152b Compare January 18, 2022 14:39
Copy link
Copy Markdown
Member

@jlnr jlnr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had to add the header_mappings_dir so that installation through CocoaPods also works on macOS. Otherwise it seems to work just fine. Thanks so much!! Can't wait to never see oal-soft warnings in my Linux terminals :)

Comment on lines +656 to +661
#ifdef __SSE__
if (!SDL_HasSSE()) {
SDL_QuitSubSystem(SDL_INIT_AUDIO);
return NULL; /* whoa! Better order a new Pentium III from Gateway 2000! */
}
#endif
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:D

@jlnr jlnr merged commit 936978c into gosu:master Jan 20, 2022
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.

2 participants