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

Fix a problem in FX about param position assumption #7053

Merged
merged 1 commit into from
Jun 19, 2023

Conversation

baconpaul
Copy link
Collaborator

The FX unit assumed &FxStorage == &FxStorage->type in one spot, which it had since epoch, but with the addition of a member at the top of the class, that assumption broke, and blammeroonie.

So do three things

  1. Fix that assumption in SurgeFXProcessor
  2. Move the new member to the end of the class just in case. Like we need to clean up the rack client which may do the same. And
  3. Add a big honkin comment explaning this

Closes #7052

The FX unit assumed &FxStorage == &FxStorage->type in one
spot, which it had since epoch, but with the addition of
a member at the top of the class, that assumption broke, and
blammeroonie.

So do three things

1. Fix that assumption in SurgeFXProcessor
2. Move the new member to the end of the class just in case.
   Like we need to clean up the rack client which may do the same.
   And
3. Add a big honkin comment explaning this

Closes surge-synthesizer#7052
@baconpaul baconpaul merged commit dad169b into surge-synthesizer:main Jun 19, 2023
@baconpaul baconpaul deleted the fx-assume-7052 branch July 8, 2024 12:00
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.

FX plugin borked
1 participant