Revert "Manually handle generated types containing references to opaques"
authorMatt Corallo <git@bluematt.me>
Tue, 29 Mar 2022 21:01:57 +0000 (21:01 +0000)
committerMatt Corallo <git@bluematt.me>
Tue, 29 Mar 2022 22:53:34 +0000 (22:53 +0000)
This reverts commit 3798e35c9b8f23fa1500b8bc9867fef319287133.

c-bindings-gen/src/blocks.rs

index 0e1617a3fc35ada2d9ec76f1420e1cbeba0ac991..e97fb65e88e4c0a58dfcf82e959e48515aecbc9f 100644 (file)
@@ -272,22 +272,14 @@ pub fn write_tuple_block<W: std::io::Write>(w: &mut W, mangled_container: &str,
        writeln!(w, "pub struct {} {{", mangled_container).unwrap();
        for (idx, ty) in types.iter().enumerate() {
                writeln!(w, "\t/// The element at position {}", idx).unwrap();
-               if ty.starts_with("&'static ") {
-                       writeln!(w, "\tpub {}: {},", ('a' as u8 + idx as u8) as char, &ty[9..]).unwrap();
-               } else {
-                       writeln!(w, "\tpub {}: {},", ('a' as u8 + idx as u8) as char, ty).unwrap();
-               }
+               writeln!(w, "\tpub {}: {},", ('a' as u8 + idx as u8) as char, ty).unwrap();
        }
        writeln!(w, "}}").unwrap();
 
        let mut tuple_str = "(".to_owned();
        for (idx, ty) in types.iter().enumerate() {
                if idx != 0 { tuple_str += ", "; }
-               if ty.starts_with("&'static ") {
-                       tuple_str += &ty[9..];
-               } else {
-                       tuple_str += ty;
-               }
+               tuple_str += ty;
        }
        tuple_str += ")";
 
@@ -314,14 +306,8 @@ pub fn write_tuple_block<W: std::io::Write>(w: &mut W, mangled_container: &str,
                writeln!(w, "impl Clone for {} {{", mangled_container).unwrap();
                writeln!(w, "\tfn clone(&self) -> Self {{").unwrap();
                writeln!(w, "\t\tSelf {{").unwrap();
-               for (idx, ty) in types.iter().enumerate() {
-                       if ty.starts_with("&'static ") {
-                               // Assume blindly the type is opaque. If its not we'll fail to build.
-                               // Really we should never have derived structs with a reference type.
-                               write!(w, "\t\t\t{}: {} {{ inner: self.{}.inner, is_owned: false}},", ('a' as u8 + idx as u8) as char, &ty[9..], ('a' as u8 + idx as u8) as char).unwrap();
-                       } else{
-                               writeln!(w, "\t\t\t{}: Clone::clone(&self.{}),", ('a' as u8 + idx as u8) as char, ('a' as u8 + idx as u8) as char).unwrap();
-                       }
+               for idx in 0..types.len() {
+                       writeln!(w, "\t\t\t{}: Clone::clone(&self.{}),", ('a' as u8 + idx as u8) as char, ('a' as u8 + idx as u8) as char).unwrap();
                }
                writeln!(w, "\t\t}}").unwrap();
                writeln!(w, "\t}}").unwrap();
@@ -341,14 +327,8 @@ pub fn write_tuple_block<W: std::io::Write>(w: &mut W, mangled_container: &str,
        }
        writeln!(w, ") -> {} {{", mangled_container).unwrap();
        write!(w, "\t{} {{ ", mangled_container).unwrap();
-       for (idx, ty) in types.iter().enumerate() {
-               if ty.starts_with("&'static ") {
-                       // Assume blindly the type is opaque. If its not we'll fail to build.
-                       // Really we should never have derived structs with a reference type.
-                       write!(w, "{}: {} {{ inner: {}.inner, is_owned: false}}, ", ('a' as u8 + idx as u8) as char, &ty[9..], ('a' as u8 + idx as u8) as char).unwrap();
-               } else {
-                       write!(w, "{}, ", ('a' as u8 + idx as u8) as char).unwrap();
-               }
+       for idx in 0..types.len() {
+               write!(w, "{}, ", ('a' as u8 + idx as u8) as char).unwrap();
        }
        writeln!(w, "}}\n}}\n").unwrap();