Skip to content

Commit

Permalink
Handle the const struct * and struct * patterns.
Browse files Browse the repository at this point in the history
Given that C keeps a different namespace for `struct`s and aliases. The
following patterns
```c
typedef const struct foo {
    void *inner;
} *foo;

typedef struct bar {
    void *inner;
} *bar;
```
are valid C code and produces both a `struct` and a pointer called `foo`
and `bar` in different namespaces. Given that Rust does not make this
distinction, we add the `_ptr` prefix to the pointer type aliases to
avoid any name collisions.
  • Loading branch information
pvdrz committed Oct 18, 2022
1 parent d241e95 commit e0fdc7f
Show file tree
Hide file tree
Showing 3 changed files with 113 additions and 0 deletions.
83 changes: 83 additions & 0 deletions bindgen-tests/tests/expectations/tests/struct_ptr.rs

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

12 changes: 12 additions & 0 deletions bindgen-tests/tests/headers/struct_ptr.h
@@ -0,0 +1,12 @@
typedef const struct foo {
char inner;
} *foo;

typedef struct bar {
char inner;
} *bar;

void takes_foo_ptr(foo);
void takes_foo_struct(struct foo);
void takes_bar_ptr(bar);
void takes_bar_struct(struct bar);
18 changes: 18 additions & 0 deletions bindgen/ir/ty.rs
Expand Up @@ -1089,6 +1089,7 @@ impl Type {
}
CXType_Typedef => {
let inner = cursor.typedef_type().expect("Not valid Type?");
let inner_spelling = inner.spelling();
let inner =
Item::from_ty_or_ref(inner, location, None, ctx);
if inner == potential_id {
Expand All @@ -1099,6 +1100,23 @@ impl Type {
// within the clang parsing.
TypeKind::Opaque
} else {
// Check if this type definition is an alias to a pointer of a `const
// struct` with the same name and add the `_ptr` suffix to it to avoid name
// collisions.
if !ctx.options().c_naming {
if let Some(inner_name) = inner_spelling
.strip_prefix("const struct ")
.or_else(|| {
inner_spelling.strip_prefix("struct ")
})
.and_then(|s| s.strip_suffix(" *"))
{
if inner_name == name {
name += "_ptr";
}
}
}

TypeKind::Alias(inner)
}
}
Expand Down

0 comments on commit e0fdc7f

Please sign in to comment.