diff options
author | Alex Crichton <alex@alexcrichton.com> | 2019-03-27 11:00:25 -0700 |
---|---|---|
committer | Alex Crichton <alex@alexcrichton.com> | 2019-03-27 11:21:44 -0700 |
commit | 8662b47b2799aebf83f420dd14d1528aa6055567 (patch) | |
tree | d58281bd6fcb66939332b5bbe624a119e819314b /libc-test | |
parent | 87def1fb806d7869d9d16efff647cd3fe8a3573d (diff) | |
download | rust-libc-8662b47b2799aebf83f420dd14d1528aa6055567.tar.gz |
Address some PR feedback
Diffstat (limited to 'libc-test')
-rw-r--r-- | libc-test/build.rs | 20 |
1 files changed, 5 insertions, 15 deletions
diff --git a/libc-test/build.rs b/libc-test/build.rs index 30c450e491..3d8ddce44c 100644 --- a/libc-test/build.rs +++ b/libc-test/build.rs @@ -1896,37 +1896,27 @@ fn test_wasi(target: &str) { cfg.type_name(move |ty, is_struct, is_union| match ty { "FILE" => ty.to_string(), t if is_union => format!("union {}", t), - t if t.starts_with("__wasi") && t.ends_with("_u") => format!("union {}", t), + t if t.starts_with("__wasi") && t.ends_with("_u") => { + format!("union {}", t) + } t if t.starts_with("__wasi") && is_struct => format!("struct {}", t), t if t.ends_with("_t") => t.to_string(), t if is_struct => format!("struct {}", t), t => t.to_string(), }); - // This is an opaque struct but we go through shenanigans to define values - // for it - cfg.skip_struct(move |ty| ty == "__clockid"); - cfg.field_name(move |_struct, field| { match field { - // deal with fields as rust keywords + // deal with fields as rust keywords "type_" => "type".to_string(), s => s.to_string(), } }); - cfg.skip_static(move |name| { - match name { - // wasi shenanigans for defining CLOCK_REALTIME and such - s if s.starts_with("__CLOCK") => true, - _ => false, - } - }); - // Looks like LLD doesn't merge duplicate imports, so if the Rust // code imports from a module and the C code also imports from a // module we end up with two imports of function pointers which - // improt the same thing bug have different function pointers + // import the same thing but have different function pointers cfg.skip_fn_ptrcheck(|f| f.starts_with("__wasi")); cfg.generate("../src/lib.rs", "main.rs"); |