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

Missing ‘stopTimer()’ in the timer callback in foleys_ApplicationSettings.cpp #80

Open
freeeco opened this issue Apr 25, 2023 · 2 comments

Comments

@freeeco
Copy link

freeeco commented Apr 25, 2023

Missing ‘stopTimer()’ in the timer callback in foleys_ApplicationSettings.cpp is causing a hang when using the MIDI learn function.

@ffAudio
Copy link
Owner

ffAudio commented Apr 25, 2023

The Timer is not meant to be a one off. Instead the ApplicationSettings is checked every second. Only if it differs the settings are reloaded.
The reason is so you can use e.g. Midi learn in one plugin instance and the other will react to it.
Or like in the FoleysSynth example you can store presets there that are shared amongs all instances (as an alternative approach of files)

I didn't observe a hang, can you describe what you did to replicate the problem?

@freeeco
Copy link
Author

freeeco commented Apr 26, 2023

Oh yeah, that makes sense now :)

I didn't observe a hang, can you describe what you did to replicate the problem?

If I drag and drop a CC onto a slider with the MIDI learn component in the FoleysSynth example, then the plugin hangs.

I added a breakpoint to the 'valueTreeChildAdded' callback in ApplicationSettings and it seems to continuously be being called, so maybe it's getting stuck in some kind of feedback loop?

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

No branches or pull requests

2 participants