sjaehn / BOops

Sound glitch effect sequencer LV2 plugin
GNU General Public License v3.0
73 stars 2 forks source link

Strange error in lv2vst #11

Closed KottV closed 3 years ago

KottV commented 3 years ago

Hi,

When BOops loaded via https://github.com/x42/lv2vst then selecting Oops effect throws an error:

Creating VST instance: lv2vst
Initialising VST: lv2vst (1.0.0.0)
Opening VST UI: B.Oops
terminate called after throwing an instance of 'std::invalid_argument'
  what():  Can't open /usr/lib64/lv2/BOops.lv2inc/oops44100.wav.

it can be fixed by simple changing https://github.com/sjaehn/BOops/blob/master/src/FxOops.hpp#L50 to

strcat (samplepath, "/inc/oops44100.wav");

but there is no such error in other native LV2 hosts, such jalv.gtk or Carla.

sjaehn commented 3 years ago

Thanks for info. I haven't tested it with lv2vst yet. It looks like lv2vst instantiates the plugin with its bundle_path but without closing slash.

Also interesting: What about the UI (after your fix)? The background image effect controls background and all slot labels are implemented in the same way: pluginPath + "inc/" + filename. Thus, you shouldn't see any slot/effect labels. What do you see?

And of course the line below your fix is a bug. I should catch that exception.

sjaehn commented 3 years ago

I fixed it by checking for the closing slash at plugin instantiation in 382baa655ee6fb6ebefccb4738cefd9359e4939d . Also fixed the lacking exception handling. I hope it will work now.

I'll also take this fix (in the UI) along to all other B.* plugins.

KottV commented 3 years ago

Images loads fine with my "fix", seems like the only sample path was affected изображение

KottV commented 3 years ago

I fixed it by checking for the closing slash at plugin instantiation in 382baa6 . Also fixed the lacking exception handling. I hope it will work now.

Great! It works. Thank you.