X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=blobdiff_plain;f=c-bindings-gen%2Fsrc%2Fblocks.rs;h=88172722efbda088aef61443fba43e2af6515e07;hb=09c8d62d937e86eee350898c7187dad498f0ec81;hp=56267792adecc779da22f0bd631449522dade700;hpb=04c5ad83145b290683385b442bb575181c9376a3;p=ldk-c-bindings

diff --git a/c-bindings-gen/src/blocks.rs b/c-bindings-gen/src/blocks.rs
index 5626779..8817272 100644
--- a/c-bindings-gen/src/blocks.rs
+++ b/c-bindings-gen/src/blocks.rs
@@ -442,14 +442,16 @@ pub fn write_method_params<W: std::io::Write>(w: &mut W, sig: &syn::Signature, t
 	for inp in sig.inputs.iter() {
 		match inp {
 			syn::FnArg::Receiver(recv) => {
-				if !recv.attrs.is_empty() || recv.reference.is_none() { unimplemented!(); }
-				write!(w, "this_arg: {}{}",
-					match (self_ptr, recv.mutability.is_some()) {
-						(true, true) => "*mut ",
-						(true, false) => "*const ",
-						(false, true) => "&mut ",
-						(false, false) => "&",
-					}, this_param).unwrap();
+				if !recv.attrs.is_empty() { unimplemented!(); }
+				write!(w, "{}this_arg: {}{}", if recv.reference.is_none() { "mut " } else { "" },
+					if recv.reference.is_some() {
+						match (self_ptr, recv.mutability.is_some()) {
+							(true, true) => "*mut ",
+							(true, false) => "*const ",
+							(false, true) => "&mut ",
+							(false, false) => "&",
+						}
+					} else { "" }, this_param).unwrap();
 				assert!(first_arg);
 				first_arg = false;
 			},
@@ -488,12 +490,7 @@ pub fn write_method_params<W: std::io::Write>(w: &mut W, sig: &syn::Signature, t
 					return;
 				}
 			}
-			if let syn::Type::Reference(r) = &**rtype {
-				// We can't return a reference, cause we allocate things on the stack.
-				types.write_c_type(w, &*r.elem, generics, true);
-			} else {
-				types.write_c_type(w, &*rtype, generics, true);
-			}
+			types.write_c_type(w, &*rtype, generics, true);
 		},
 		_ => {},
 	}
@@ -561,8 +558,9 @@ pub fn write_method_call_params<W: std::io::Write>(w: &mut W, sig: &syn::Signatu
 	for inp in sig.inputs.iter() {
 		match inp {
 			syn::FnArg::Receiver(recv) => {
-				if !recv.attrs.is_empty() || recv.reference.is_none() { unimplemented!(); }
+				if !recv.attrs.is_empty() { unimplemented!(); }
 				if to_c {
+					if recv.reference.is_none() { unimplemented!(); }
 					write!(w, "self.this_arg").unwrap();
 					first_arg = false;
 				}
@@ -634,13 +632,12 @@ pub fn write_method_call_params<W: std::io::Write>(w: &mut W, sig: &syn::Signatu
 				write!(w, "ret").unwrap();
 				types.write_from_c_conversion_suffix(w, &*rtype, generics);
 			} else {
-				let ret_returned = if let syn::Type::Reference(_) = &**rtype { true } else { false };
 				let new_var = types.write_to_c_conversion_new_var(w, &format_ident!("ret"), &rtype, generics, true);
 				if new_var {
 					write!(w, "\n\t{}", extra_indent).unwrap();
 				}
 				types.write_to_c_conversion_inline_prefix(w, &rtype, generics, true);
-				write!(w, "{}ret", if ret_returned && !new_var { "*" } else { "" }).unwrap();
+				write!(w, "ret").unwrap();
 				types.write_to_c_conversion_inline_suffix(w, &rtype, generics, true);
 			}
 		}
@@ -651,7 +648,7 @@ pub fn write_method_call_params<W: std::io::Write>(w: &mut W, sig: &syn::Signatu
 /// Prints concrete generic parameters for a struct/trait/function, including the less-than and
 /// greater-than symbols, if any generic parameters are defined.
 pub fn maybe_write_generics<W: std::io::Write>(w: &mut W, generics: &syn::Generics, types: &TypeResolver, concrete_lifetimes: bool) {
-	let mut gen_types = GenericTypes::new();
+	let mut gen_types = GenericTypes::new(None);
 	assert!(gen_types.learn_generics(generics, types));
 	if !generics.params.is_empty() {
 		write!(w, "<").unwrap();