add option to remove type aliases

Author: pvdrz

bindgen-cli/options.rs

@@ -568,6 +568,12 @@ where
             Arg::new("merge-extern-blocks")
                 .long("merge-extern-blocks")
                 .help("Deduplicates extern blocks."),
+            Arg::new("remove-alias")
+                .long("remove-alias")
+                .help("Remove type aliases matching <regex>.")
+                .value_name("regex")
+                .multiple_occurrences(true)
+                .number_of_values(1),
             Arg::new("V")
                 .long("version")
                 .help("Prints the version, and exits"),
@@ -1088,5 +1094,11 @@ where
         builder = builder.merge_extern_blocks(true);
     }
 
+    if let Some(remove_alias) = matches.values_of("remove-alias") {
+        for regex in remove_alias {
+            builder = builder.remove_alias(regex);
+        }
+    }
+
     Ok((builder, output, verbose))
 }

bindgen-tests/tests/expectations/tests/remove_alias.rs

@@ -0,0 +1,14 @@
+// bindgen-flags: --remove-alias "int.*"
+
+typedef long long int64;
+typedef int int32;
+typedef int32 i32;
+
+struct int32_ {
+    int32 inner;
+};
+
+int64 foo();
+int32 bar();
+i32 baz();
+struct int32_ qux();

bindgen-tests/tests/expectations/tests/remove_template_alias.rs

@@ -0,0 +1,4 @@
+// bindgen-flags: --remove-alias "Wrapped"
+
+template<typename T>
+using Wrapped = T;

bindgen-tests/tests/headers/remove_alias.h

@@ -45,7 +45,7 @@ lazycell = "1"
 lazy_static = "1"
 peeking_take_while = "0.1.2"
 quote = { version = "1", default-features = false }
-syn = { version = "1.0.99", features = ["full", "extra-traits"]}
+syn = { version = "1.0.99", features = ["full", "extra-traits", "visit-mut"]}
 regex = { version = "1.5", default-features = false , features = ["std", "unicode"] }
 which = { version = "4.2.1", optional = true, default-features = false }
 shlex = "1"

bindgen-tests/tests/headers/remove_template_alias.hpp

@@ -1,6 +1,11 @@
 use syn::{Item, ItemForeignMod};
 
-pub(super) fn merge_extern_blocks(items: &mut Vec<Item>) {
+use crate::BindgenOptions;
+
+pub(super) fn merge_extern_blocks(
+    items: &mut Vec<Item>,
+    _options: &BindgenOptions,
+) {
     // Keep all the extern blocks in a different `Vec` for faster search.
     let mut foreign_mods = Vec::<ItemForeignMod>::new();
 

bindgen/Cargo.toml

@@ -5,14 +5,16 @@ use syn::Item;
 use crate::BindgenOptions;
 
 mod merge_extern_blocks;
+mod remove_alias;
 mod sort_semantically;
 
 use merge_extern_blocks::merge_extern_blocks;
+use remove_alias::remove_alias;
 use sort_semantically::sort_semantically;
 
 struct PostProcessingPass {
     should_run: fn(&BindgenOptions) -> bool,
-    run: fn(&mut Vec<Item>),
+    run: fn(&mut Vec<Item>, &BindgenOptions),
 }
 
 // TODO: This can be a const fn when mutable references are allowed in const
@@ -21,13 +23,19 @@ macro_rules! pass {
     ($pass:ident) => {
         PostProcessingPass {
             should_run: |options| options.$pass,
-            run: |items| $pass(items),
+            run: |items, options| $pass(items, options),
         }
     };
 }
 
-const PASSES: &[PostProcessingPass] =
-    &[pass!(merge_extern_blocks), pass!(sort_semantically)];
+const PASSES: &[PostProcessingPass] = &[
+    pass!(merge_extern_blocks),
+    pass!(sort_semantically),
+    PostProcessingPass {
+        should_run: |options| !options.remove_alias.is_empty(),
+        run: |items, options| remove_alias(items, options),
+    },
+];
 
 pub(crate) fn postprocessing(
     items: Vec<TokenStream>,
@@ -56,7 +64,7 @@ pub(crate) fn postprocessing(
 
     for pass in PASSES {
         if (pass.should_run)(options) {
-            (pass.run)(&mut items);
+            (pass.run)(&mut items, options);
         }
     }
 

bindgen/codegen/postprocessing/merge_extern_blocks.rs

@@ -0,0 +1,53 @@
+use syn::visit_mut::{visit_type_mut, VisitMut};
+use syn::{Item, Type};
+
+use crate::BindgenOptions;
+
+pub(super) fn remove_alias(items: &mut Vec<Item>, options: &BindgenOptions) {
+    let visitors: Vec<_> = items
+        .iter()
+        .enumerate()
+        .rev()
+        .filter_map(|(index, item)| {
+            if let Item::Type(alias_item) = item {
+                if alias_item.generics.params.is_empty() {
+                    let ident = alias_item.ident.to_string();
+                    if options.remove_alias.matches(&ident) {
+                        return Some((
+                            index,
+                            Visitor {
+                                ident,
+                                ty: alias_item.ty.clone(),
+                            },
+                        ));
+                    }
+                }
+            }
+            None
+        })
+        .collect();
+
+    for (index, mut visitor) in visitors {
+        items.remove(index);
+        for item in items.iter_mut() {
+            visitor.visit_item_mut(item);
+        }
+    }
+}
+
+struct Visitor {
+    ident: String,
+    ty: Box<Type>,
+}
+
+impl VisitMut for Visitor {
+    fn visit_type_mut(&mut self, ty: &mut Type) {
+        if let Type::Path(type_path) = ty {
+            if type_path.path.is_ident(&self.ident) {
+                *ty = self.ty.as_ref().clone();
+            }
+        }
+
+        visit_type_mut::<_>(self, ty)
+    }
+}

bindgen/codegen/postprocessing/mod.rs

@@ -1,6 +1,8 @@
 use syn::Item;
 
-pub(super) fn sort_semantically(items: &mut [Item]) {
+use crate::BindgenOptions;
+
+pub(super) fn sort_semantically(items: &mut [Item], _options: &BindgenOptions) {
     items.sort_by_key(|item| match item {
         Item::Type(_) => 0,
         Item::Struct(_) => 1,