1489468 - Use cbindgen to expose Baldrdash rust bindings to C

Status: RESOLVED INACTIVE

(Core :: JavaScript: WebAssembly, enhancement, P5)

Description

[Benjamin Bouvier [:bbouvier] (inactive)]

There's already the cbindgen crate in tree, used for WebRender, from which we can draw inspiration: https://searchfox.org/mozilla-central/source/gfx/webrender_bindings/src/bindings.rs

Using cbindgen will avoid possible function signature mismatches between Rust and C.

Comment 1

[22shubh22]

Hey, I am interested. Can anybody guide me through this issue?

Comment 2

[Benjamin Bouvier [:bbouvier] (inactive)]

That's a good question; I haven't used cbindgen myself. The way I'd do it would be to look at cbindgen documentation, then look at other uses of cbindgen in tree, then see what I can replace by using cbindgen (and if that's actually better or as good as the current implementation). To be transparent, I don't think this is an easy bug, because it might involve build hardships (e.g. do we need to pass platform specific build flags to cbindgen, which ones, etc). Feel free to try and let us know, though, if you feel like it :)

Comment 3

[Emilio Cobos Álvarez (:emilio)]

I'm happy to mentor this if you want. Grepping RunCbindgen.py should show how it's used.