-
Notifications
You must be signed in to change notification settings - Fork 122
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
LADSPA UGen error handling/reporting #210
Comments
man, does anyone even use LADSPA anymore? it's kind of a dying project and the plugins i've seen for it aren't even that good... |
There are some few projects (that are even still maintained), but LV2 definitely is all the rage now :) |
I can confirm this issue in version 3.11.2 and disabling ladspa solves it. I would argue though that LADSPA is still useful especially as long as you can't run LV2 plugins in SuperCollider, but even then the simplicity of LADSPA could be a advantage. |
Couple of questions:
A |
it looks like this is just an initial size; the array is reallocated with more space later as needed: https://github.com/supercollider/sc3-plugins/blob/master/source/LadspaUGen/LadspaUGen.cpp#L116-L123
i thought the crash is due to a segmentation fault? |
Ok, I misread and misunderstood. So the only way to probe plugins 100% safely is to spawn them into a separate process? I think Christof's VSTPlugin does something like that. |
yes, as far as i know. i don't think it's possible to recover from a segfault signal, at least on Linux. |
A mailing list thread on sc-users recently revealed that the LADSPA UGen (running in Linux) can prevent the server from booting, without posting any error messages:
https://www.listarc.bham.ac.uk/lists/sc-users/msg62057.html
Because of the remark about VSTs under Wine, I hazarded the guess that LADSPA might be trying to get info about one or more VSTs and aborting the server in the attempt (with, incorrectly, a 0 result code). So I asked the user to rename the LadspaUGen.so files to prevent them from loading, and then the server booted normally.
Suggestions:
Our LADSPA UGen should ideally catch exceptions and continue without access to the specific plug-in.
If that's
possibleimpossible for whatever reason, then there should be some output so that the user has some idea what the problem is. (Sorry for typos.)If that is
it possibleimpossible for whatever reason, then we should make it possible to disable the LADSPA plug-in as a build switch. (This isn't the only problem with LADSPA [LADSPA: ladspalist utility crashes on certain plugins #23] -- if it has bugs that render it unusable and it isn't going to be maintained, then it should be possible for users to do away with it in their own environment.)The text was updated successfully, but these errors were encountered: