Return Results from native function wrappers

[akonradi-signal]

This doesn't bubble the Result return type all the way to the top, and it uses the existing status code as the error type instead of defining a new type as suggested in #802 (comment).

Progress on #802

[kjvalencik]

Woah! This is awesome. Thanks for taking this on.

I notice a lot of let () = .... Is that an artifact of a code mod tool or is that serving a purpose?

[akonradi-signal]

It serves the purpose of indicating that there's no additional information like a boolean success value, that needs to be handled.

[akonradi-signal]

I should note that it's not required, and I'm happy to remove the let () = s if that doesn't seem useful.

[kjvalencik]

I think I would prefer removing it. It feels like a technique that would make more sense back before the #[must_use] annotation existed.

[dherman]

This is a great improvement, thank you for doing this!

[akonradi-signal]

Rebased on latest main to fix build conflicts

[dherman]

Merged, thank you so much for tackling this!