Clang trunk considers stuff in an anonymous namespace with internal linkage.

I think that's right, and the previous behavior was just a bug.

Rejigger the tests that show this difference since they aren't testing this in
particular, and avoids adding more per-platform tests.

Author: emilio

tests/expectations/tests/namespace.rs

@@ -1,9 +1,7 @@
 /* automatically generated by rust-bindgen */
 
-
 #![allow(dead_code, non_snake_case, non_camel_case_types, non_upper_case_globals)]
 
-
 #[allow(non_snake_case, non_camel_case_types, non_upper_case_globals)]
 pub mod root {
     #[allow(unused_imports)]
@@ -24,10 +22,6 @@ pub mod root {
     pub mod _bindgen_mod_id_13 {
         #[allow(unused_imports)]
         use self::super::super::root;
-        extern "C" {
-            #[link_name = "\u{1}_ZN12_GLOBAL__N_13fooEv"]
-            pub fn foo();
-        }
         #[repr(C)]
         #[derive(Debug, Default, Copy, Clone)]
         pub struct A {
@@ -51,18 +45,6 @@ pub mod root {
                 concat!("Offset of field: ", stringify!(A), "::", stringify!(b))
             );
         }
-        extern "C" {
-            #[link_name = "\u{1}_ZN12_GLOBAL__N_11A20lets_hope_this_worksEv"]
-            pub fn A_lets_hope_this_works(
-                this: *mut root::_bindgen_mod_id_13::A,
-            ) -> ::std::os::raw::c_int;
-        }
-        impl A {
-            #[inline]
-            pub unsafe fn lets_hope_this_works(&mut self) -> ::std::os::raw::c_int {
-                A_lets_hope_this_works(self)
-            }
-        }
     }
     #[repr(C)]
     #[derive(Debug)]

tests/headers/namespace.hpp

@@ -11,11 +11,8 @@ namespace whatever {
 namespace {
     namespace empty {}
 
-    void foo();
     struct A {
         whatever::whatever_int_t b;
-    public:
-        int lets_hope_this_works();
     };
 }
 
@@ -26,11 +23,9 @@ class C: public A {
     T m_c_arr[10];
 };
 
-
 template<>
 class C<int>;
 
-
 namespace w {
     typedef unsigned int whatever_int_t;