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

XT Crash with overlay and FX select #4541

Closed
mkruselj opened this issue May 11, 2021 · 5 comments
Closed

XT Crash with overlay and FX select #4541

mkruselj opened this issue May 11, 2021 · 5 comments
Labels
Bug Report Item submitted using the Bug Report template
Milestone

Comments

@mkruselj
Copy link
Collaborator

  1. Load an effect in an FX slot
  2. Open MSEG edit view
  3. Select an empty FX slot
  4. Select the FX slot in which you've loaded an effect
  5. Crash!
@mkruselj mkruselj added the Bug Report Item submitted using the Bug Report template label May 11, 2021
@mkruselj mkruselj added this to the Surge XT 1.0 milestone May 11, 2021
@baconpaul
Copy link
Collaborator

baconpaul commented May 11, 2021

So a quicker way to see the bug is

  1. init patch
  2. mseg and open editor
  3. with editor open select an FX in slot 0 - this dismisses the editor overlay - this is the problem. The rebuild didn't preserve state properly and now you are screwed
  4. change anything and it crashes

so clearly the editor overlay 'memory' code has broken in juce land, or with the introduction of the various new overlays.

@selenologist
Copy link
Collaborator

Seems another case is

  1. Assign some modulation to (Classic) Oscillator Pitch.
  2. Right click and edit the modulation w/ a typein
  3. Attempt to enter a value out of bounds of modulation, even by a tiny amount (e.g. 14.00001 semitones)
  4. Crash

(typeins still seem to work as long as you're in-bounds)

@ghost
Copy link

ghost commented Jun 25, 2021

This seems to be solved (?)
At least for Linux none of the above examples crash anymore.

@mkruselj
Copy link
Collaborator Author

Yeah I also can't get it to happen with above steps. Seems fixed, so closing the issue!

@baconpaul
Copy link
Collaborator

The crash is fixed (since I rewrote all the objects) but there’s still a set of bugs on my list for things like “change between two lfos which are mseg with the mseg open and does it work” and all the other stare which used to crash but is still wrong in the sge idle

the crash was because the ownership was too liberal. It’s now more conservative but the state management may be off.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Report Item submitted using the Bug Report template
Projects
None yet
Development

No branches or pull requests

3 participants