Enable layout tests for generated bindings

rust3ds / ctru-rs

Rust wrapper for libctru

https://rust3ds.github.io/ctru-rs/

Other

120 stars 18 forks source link

Enable layout tests for generated bindings #168

Closed ian-h-chamberlain closed 6 months ago

ian-h-chamberlain commented 7 months ago

Closes #167

Bindgen can help us catch ABI issues sometimes! Let's use it, and see if it catches any issues.

ian-h-chamberlain commented 7 months ago

Well, I guess this didn't fix the issue mentioned in #162, and didn't flag any other issue, so I'm not sure how helpful it will be, but we may as well enable it since it works with very little effort.

FenrirWolf commented 7 months ago

Yeah, bindgen can't detect enum-related layout issues if it legit thinks every enum will be 4 bytes wide. But we might as well have these tests too.