Skip to content

refactor: options in cpp side#929

Merged
maciejmakowski2003 merged 10 commits intomainfrom
refactor/node-options
Feb 6, 2026
Merged

refactor: options in cpp side#929
maciejmakowski2003 merged 10 commits intomainfrom
refactor/node-options

Conversation

@mdydek
Copy link
Contributor

@mdydek mdydek commented Jan 31, 2026

Closes #

⚠️ Breaking changes ⚠️

Introduced changes

  • Moved default options to cpp in order to be able to initialize shadow node state with appropriate values without calling getters

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added/Conducted relevant tests
  • Performed self-review of the code
  • Updated Web Audio API coverage
  • Added support for web
  • Updated old arch android spec file

Copy link
Contributor

@poneciak57 poneciak57 left a comment

Choose a reason for hiding this comment

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

LGTM

@maciejmakowski2003 maciejmakowski2003 merged commit 0f41ecd into main Feb 6, 2026
4 checks passed
@maciejmakowski2003 maciejmakowski2003 deleted the refactor/node-options branch February 6, 2026 12:47
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.

4 participants