buttplug-rs-ffi icon indicating copy to clipboard operation
buttplug-rs-ffi copied to clipboard

Move exception catch for missing natives

Open kitlith opened this issue 2 years ago • 3 comments

This should make the "missing natives" exception more visible, since it'll be closer to the top of the stack.

Also added some handling for setting up a log handler.

while I was working on this, I noticed callback = (ctx, str) -> cb.log(str);, which is not a static callback like most/every other callback in this library. i should probably fix that at some point.

kitlith avatar Feb 07 '23 21:02 kitlith

hm. looks like CI partially broke.

kitlith avatar Feb 07 '23 21:02 kitlith

thought: should i change the message from "missing natives" to "failed to load natives"?

kitlith avatar Feb 07 '23 23:02 kitlith

That's probably a little clearer yeah.

qdot avatar Feb 07 '23 23:02 qdot