Skip to content

Add base implementation of surround extension#84

Merged
abique merged 4 commits intofree-audio:nextfrom
Schroedingers-Cat:next
Jul 8, 2025
Merged

Add base implementation of surround extension#84
abique merged 4 commits intofree-audio:nextfrom
Schroedingers-Cat:next

Conversation

@Schroedingers-Cat
Copy link
Copy Markdown
Contributor

This PR adds the surround extension to the clap-helpers so that consumers of the clap-helpers can implement it into their plugins. It also adds some minimal level checks to the base implementation.

The plugin functions is_channel_mask_supported and get_channel_map ensure they're called when the plugin is not active due to the surround extension's comment (see If the host decides to change the project's surround setup and If the plugin wants to change its surround setup) using them only after the plugin has been deactivated.

Comment thread include/clap/helpers/plugin.hxx Outdated
Comment thread include/clap/helpers/plugin.hxx Outdated
free-audio#84#discussion_r2192490266
free-audio#84#discussion_r2192491651
@abique
Copy link
Copy Markdown
Contributor

abique commented Jul 8, 2025

LGTM

@abique abique merged commit 4d8c96f into free-audio:next Jul 8, 2025
13 checks passed
@Schroedingers-Cat Schroedingers-Cat deleted the next branch July 8, 2025 13:37
abique pushed a commit that referenced this pull request Nov 28, 2025
* Add base implementation of surround extension

* Comment out windows clang test due to broken env

* Remove wrong check

#84#discussion_r2192490266

* Remove wrong check

#84#discussion_r2192491651
abique pushed a commit that referenced this pull request Nov 28, 2025
* Add base implementation of surround extension

* Comment out windows clang test due to broken env

* Remove wrong check

#84#discussion_r2192490266

* Remove wrong check

#84#discussion_r2192491651
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