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

Feature: Allow choosing audio backend #1588

Merged
merged 6 commits into from
Sep 26, 2022

Conversation

dcvz
Copy link
Contributor

@dcvz dcvz commented Sep 25, 2022

This PR allows users to choose an audio backend and fixes the following bugs:

  • Does not search for X11 on non-Linux platforms
  • Remembers gfx backend selected

@dcvz dcvz force-pushed the feature/choose-audio-backend branch 2 times, most recently from 66bef57 to 6caefc3 Compare September 25, 2022 15:08
@dcvz dcvz force-pushed the feature/choose-audio-backend branch from 6caefc3 to d70303d Compare September 25, 2022 15:29
@dcvz dcvz added the do not merge Not ready or not valid changes label Sep 25, 2022
Copy link
Collaborator

@Kenix3 Kenix3 left a comment

Choose a reason for hiding this comment

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

When you change the audio backend, what is changing the APlayer on the window?

@dcvz
Copy link
Contributor Author

dcvz commented Sep 25, 2022

When you change the audio backend, what is changing the APlayer on the window?

Currently it requires a reload to apply. Just like the gfx backend option. Didn't look into whether APlayer can be swapped at runtime

@dcvz dcvz removed the do not merge Not ready or not valid changes label Sep 25, 2022
@Kenix3 Kenix3 merged commit dbc4d81 into HarbourMasters:develop Sep 26, 2022
dcvz added a commit to dcvz/Shipwright that referenced this pull request Sep 27, 2022
* Fix X11 being searched for on macOS

* Add ability to switch audio backends

* Fix issues with c arrays of std dynamic size items

* Fix old uses of sizeof()

* Remove current selected option from list

* Don’t show items if we only have one option
@dcvz dcvz deleted the feature/choose-audio-backend branch September 28, 2022 07:44
@briaguya-ai briaguya-ai mentioned this pull request Oct 1, 2022
Kenix3 pushed a commit to Kenix3/Shipwright that referenced this pull request Oct 19, 2022
* Fix X11 being searched for on macOS

* Add ability to switch audio backends

* Fix issues with c arrays of std dynamic size items

* Fix old uses of sizeof()

* Remove current selected option from list

* Don’t show items if we only have one option
dcvz added a commit to briaguya-ai/Shipwright that referenced this pull request Nov 12, 2022
* Fix X11 being searched for on macOS

* Add ability to switch audio backends

* Fix issues with c arrays of std dynamic size items

* Fix old uses of sizeof()

* Remove current selected option from list

* Don’t show items if we only have one option
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants