Add remove-function-prefix option; Add test

Author: justsmth

src/options.rs

@@ -1,3 +1,4 @@
+use std::fmt::Debug;
 use bindgen::{
     builder, AliasVariation, Builder, CodegenConfig, EnumVariation,
     MacroTypeVariation, RustTarget, DEFAULT_ANON_FIELDS_PREFIX,
@@ -8,6 +9,34 @@ use std::fs::File;
 use std::io::{self, stderr, Error, ErrorKind, Write};
 use std::path::PathBuf;
 use std::str::FromStr;
+use bindgen::callbacks::ParseCallbacks;
+
+#[derive(Debug)]
+pub struct LinkNameOverrideParseCallback {
+    pub remove_function_prefix: Option<String>,
+}
+
+impl LinkNameOverrideParseCallback {
+    pub fn new(prefix: &str) -> Self {
+        LinkNameOverrideParseCallback {
+            remove_function_prefix: Some(prefix.to_string()),
+        }
+    }
+}
+
+impl ParseCallbacks for LinkNameOverrideParseCallback {
+    fn link_name_override(
+        &self,
+        function_name: &str,
+    ) -> Option<String> {
+        if let Some(prefix) = &self.remove_function_prefix {
+            if function_name.starts_with(prefix) {
+                return Some(function_name[prefix.len()..].to_string());
+            }
+        }
+        None
+    }
+}
 
 /// Construct a new [`Builder`](./struct.Builder.html) from command line flags.
 pub fn builder_from_flags<I>(
@@ -545,6 +574,11 @@ where
             Arg::new("vtable-generation")
                 .long("vtable-generation")
                 .help("Enables generation of vtable functions."),
+            Arg::new("remove-function-prefix")
+                .long("remove-function-prefix")
+                .multiple_occurrences(true)
+                .takes_value(true)
+                .help("Remove prefix when generating Rust function name.")
         ]) // .args()
         .get_matches_from(args);
 
@@ -1015,6 +1049,11 @@ where
         builder = builder.vtable_generation(true);
     }
 
+    if let Some(prefix) = matches.value_of("remove-function-prefix") {
+        let lnopc = LinkNameOverrideParseCallback::new(prefix);
+        builder = builder.parse_callbacks(Box::new(lnopc));
+    }
+
     let verbose = matches.is_present("verbose");
 
     Ok((builder, output, verbose))

tests/expectations/tests/issue-1375-prefixed-functions.rs

@@ -0,0 +1,11 @@
+#![allow(
+    dead_code,
+    non_snake_case,
+    non_camel_case_types,
+    non_upper_case_globals
+)]
+
+extern "C" {
+    #[link_name = "\u{1}my_custom_prefix_function_name"]
+    pub fn function_name(x: ::std::os::raw::c_int);
+}

tests/headers/issue-1375-prefixed-functions.h

@@ -0,0 +1,5 @@
+// bindgen-flags: --remove-function-prefix my_custom_prefix_
+// bindgen-parse-callbacks: remove-function-prefix-my_custom_prefix_
+
+void my_custom_prefix_function_name(const int x);
+

tests/parse_callbacks/mod.rs

@@ -1,4 +1,5 @@
 use bindgen::callbacks::*;
+use crate::options::LinkNameOverrideParseCallback;
 
 #[derive(Debug)]
 struct EnumVariantRename;
@@ -36,7 +37,15 @@ pub fn lookup(cb: &str) -> Box<dyn ParseCallbacks> {
         "enum-variant-rename" => Box::new(EnumVariantRename),
         "blocklisted-type-implements-trait" => {
             Box::new(BlocklistedTypeImplementsTrait)
+        },
+        call_back => {
+            if call_back.starts_with("remove-function-prefix-") {
+                let prefix = call_back.split("remove-function-prefix-").last().to_owned();
+                let lnopc = LinkNameOverrideParseCallback::new(prefix.unwrap());
+                Box::new(lnopc)
+            } else {
+                panic!("Couldn't find name ParseCallbacks: {}", cb)
+            }
         }
-        _ => panic!("Couldn't find name ParseCallbacks: {}", cb),
     }
 }