diff options
author | Geoffrey Thomas <geofft@ldpreload.com> | 2021-06-03 10:47:26 -0400 |
---|---|---|
committer | Christian Poveda <christian.poveda@ferrous-systems.com> | 2022-09-23 21:13:37 -0500 |
commit | cc78b6fdb6e829e5fb8fa1639f2182cb49333569 (patch) | |
tree | 09b846c15394d331489c2ae0ff6bf611736aa428 /tests/headers/func_with_array_arg.h | |
parent | 6de2d3d1c17bb4dd432daabf2cd858254663f49e (diff) |
Map size_t to usize by default and check compatibility (fixes #1901, #1903)
This addresses the underlying issue identified in #1671, that size_t
(integer that can hold any object size) isn't guaranteed to match usize,
which is defined more like uintptr_t (integer that can hold any
pointer). However, on almost all platforms, this is true, and in fact
Rust already uses usize extensively in contexts where size_t would be
more appropriate, such as slice indexing. So, it's better for ergonomics
when interfacing with C code to map the C size_t type to usize. (See
also discussion in rust-lang/rust#65473 about how usize really should be
defined as size_t, not uintptr_t.)
The previous fix for #1671 removed the special case for size_t and
defaulted to binding it as a normal typedef. This change effectively
reverts that and goes back to mapping size_t to usize (and ssize_t to
isize), but also ensures that if size_t is emitted, the typedef'd type
of size_t in fact is compatible with usize (defined by checking that the
size and alignment match the target pointer width). For (hypothetical)
platforms where this is not true, or for compatibility with the default
behavior of bindgen between 0.53 and this commit, onwards, you can
disable this mapping with --no-size_t-is-usize.
Diffstat (limited to 'tests/headers/func_with_array_arg.h')
0 files changed, 0 insertions, 0 deletions