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

implement audio backend fallbacks #1857

Merged
merged 11 commits into from
Jan 21, 2023

Conversation

Alto1772
Copy link
Contributor

@Alto1772 Alto1772 commented Oct 26, 2022

As described in this comment: #1837 (comment)

This method does is if it fails to initialize at first, it loops and Init()s through all available audio apis listed in audioBackends at libultraship/libultraship/ImGuiImpl.cpp until one returns successfully, and then overwriting the config file to use that one.

also I've changed something in GameMenuBar.cpp that fixes the zero entries in the the drop-down list so that it disables when there's only one entry. Separated into as #1958

note: if u don't understand what i've just said here, just review my code, approve it or ask questions, or just don't.

Build Artifacts

@dcvz
Copy link
Contributor

dcvz commented Nov 14, 2022

Hi @Alto1772, libultraship (LUS) has now been separated from this repo. Would you mind rebasing and making a PR for LUS changes against the repo here?

Also, I think the question about whether we want the fallback to be in LUS was just a question from bria.. I think we definitely do want to have that on the LUS side rather than SoH.

@briaguya-ai
Copy link
Contributor

now that we have libultraship pulled in as a submodule, you'll need to PR the libultraship side of things upstream to https://github.com/Kenix3/libultraship

@briaguya-ai briaguya-ai added the do not merge Not ready or not valid changes label Nov 14, 2022
@dcvz
Copy link
Contributor

dcvz commented Nov 16, 2022

Reliant on: Kenix3/libultraship#28

Copy link
Contributor

@briaguya-ai briaguya-ai left a comment

Choose a reason for hiding this comment

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

looks like the name here doesn't match LUS, would you mind updating this to work with the latest version?

@briaguya-ai briaguya-ai merged commit 52a9764 into HarbourMasters:develop Jan 21, 2023
@Alto1772 Alto1772 deleted the audio-fallback branch February 7, 2023 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Not ready or not valid changes libultraship
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants