Allow creation of Vector<BoxedRef<_>> instances and using them as ToInputArray

Author: twistedfall

binding-generator/src/class.rs

@@ -160,11 +160,6 @@ impl<'tu, 'ge> Class<'tu, 'ge> {
 		}
 	}
 
-	pub fn is_trait(&self) -> bool {
-		self.kind().is_boxed()
-		//		self.is_abstract() || self.has_descendants() || settings::FORCE_CLASS_TRAIT.contains(self.cpp_name(CppNameStyle::Reference).as_ref())
-	}
-
 	/// Special case of an empty class with only an anonymous enum inside (e.g. DrawLinesMatchesFlags)
 	pub fn as_enum(&self) -> Option<Enum<'tu>> {
 		match self {
@@ -429,7 +424,7 @@ impl<'tu, 'ge> Class<'tu, 'ge> {
 							&& !fld_type_kind.is_char_ptr_string(fld_type_ref.type_hint())
 						{
 							fld_type_ref.set_type_hint(TypeRefTypeHint::PrimitivePtrAsRaw);
-						} else if fld_type_kind.as_class().map_or(false, |cls| cls.is_trait()) {
+						} else if fld_type_kind.as_class().map_or(false, |cls| cls.kind().is_trait()) {
 							fld_type_ref.set_type_hint(TypeRefTypeHint::TraitClassConcrete);
 						}
 						let fld_type_kind = fld_type_ref.kind();
@@ -718,7 +713,7 @@ impl fmt::Debug for Class<'_, '_> {
 		if self.is_polymorphic() {
 			props.push("polymorphic");
 		}
-		if self.is_trait() {
+		if self.kind().is_trait() {
 			props.push("trait");
 		}
 		if self.as_enum().is_some() {
@@ -800,7 +795,13 @@ impl ClassKind {
 
 	pub fn is_boxed(self) -> bool {
 		match self {
-			// fixme: Self::System to return true is kind of hack to make sure that system classes are passed by void*, probably better to handle in explicitly in CppPassByVoidPtrRenderLane
+			Self::Boxed | Self::BoxedForced => true,
+			Self::Simple | Self::Other | Self::System => false,
+		}
+	}
+
+	pub fn is_trait(&self) -> bool {
+		match self {
 			Self::Boxed | Self::BoxedForced | Self::System => true,
 			Self::Simple | Self::Other => false,
 		}

binding-generator/src/class/desc.rs

@@ -37,6 +37,13 @@ impl<'tu, 'ge> ClassDesc<'tu, 'ge> {
 		}
 	}
 
+	pub fn system(cpp_refname: impl Into<Rc<str>>, rust_module: impl Into<Rc<str>>) -> Self {
+		Self {
+			kind: ClassKind::System,
+			..Self::boxed(cpp_refname, rust_module)
+		}
+	}
+
 	/// `cv::Scalar`
 	pub fn cv_scalar() -> Class<'tu, 'ge> {
 		Class::new_desc(Self::simple("cv::Scalar", "core"))
@@ -104,7 +111,7 @@ impl<'tu, 'ge> ClassDesc<'tu, 'ge> {
 
 	/// `cv::String`
 	pub fn cv_string() -> Class<'tu, 'ge> {
-		Class::new_desc(Self::boxed("cv::String", "core"))
+		Class::new_desc(Self::system("cv::String", "core"))
 	}
 
 	/// `cv::MatConstIterator`

binding-generator/src/type_ref/kind.rs

@@ -63,7 +63,7 @@ impl<'tu, 'ge> TypeRefKind<'tu, 'ge> {
 		match self {
 			TypeRefKind::Typedef(tdef) => tdef.underlying_type_ref().kind().extern_pass_kind(),
 			TypeRefKind::Class(inner) if !inner.string_type().is_some() => {
-				if inner.kind().is_boxed() {
+				if inner.kind().is_trait() {
 					ExternPassKind::ByVoidPtr
 				} else {
 					ExternPassKind::ByPtr
@@ -212,10 +212,6 @@ impl<'tu, 'ge> TypeRefKind<'tu, 'ge> {
 		}
 	}
 
-	pub fn as_simple_class(&self) -> Option<Cow<Class<'tu, 'ge>>> {
-		self.as_class().filter(|cls| cls.kind().is_simple())
-	}
-
 	pub fn as_vector(&self) -> Option<Cow<Vector<'tu, 'ge>>> {
 		match self {
 			TypeRefKind::StdVector(out) => Some(Borrowed(out)),

binding-generator/src/writer/rust_native/class.rs

@@ -51,8 +51,8 @@ fn gen_rust_class(c: &Class, opencv_version: &str) -> String {
 	static TRAIT_TPL: Lazy<CompiledInterpolation> = Lazy::new(|| include_str!("tpl/class/trait.tpl.rs").compile_interpolation());
 
 	let type_ref = c.type_ref();
-	let is_trait = c.is_trait();
 	let class_kind = c.kind();
+	let is_trait = class_kind.is_trait();
 	let doc_comment = c.rendered_doc_comment("///", opencv_version);
 
 	let mut out = String::new();
@@ -577,7 +577,7 @@ pub trait ClassExt {
 impl ClassExt for Class<'_, '_> {
 	fn rust_trait_name(&self, style: NameStyle, constness: Constness) -> Cow<str> {
 		let mut out = self.rust_name(style);
-		if self.is_trait() {
+		if self.kind().is_trait() {
 			if constness.is_const() {
 				out.to_mut().push_str("TraitConst");
 			} else {

binding-generator/src/writer/rust_native/func.rs

@@ -293,7 +293,7 @@ impl RustNativeGeneratedElement for Func<'_, '_> {
 
 		let doc_comment = self.rendered_doc_comment("///", _opencv_version);
 		let visibility = if let Some(cls) = as_instance_method {
-			if cls.is_trait() {
+			if cls.kind().is_trait() {
 				""
 			} else {
 				"pub "

binding-generator/src/writer/rust_native/mod.rs

@@ -165,7 +165,7 @@ impl GeneratorVisitor for RustNativeBindingWriter<'_> {
 
 	fn visit_class(&mut self, class: Class) {
 		self.emit_debug_log(&class);
-		if class.is_trait() {
+		if class.kind().is_trait() {
 			self.prelude_traits.push(format!(
 				"super::{}",
 				class.rust_trait_name(NameStyle::decl(), Constness::Const).into_owned()

binding-generator/src/writer/rust_native/smart_ptr.rs

@@ -104,7 +104,7 @@ impl RustNativeGeneratedElement for SmartPtr<'_, '_> {
 		]);
 
 		let mut impls = String::new();
-		if let Some(cls) = pointee_kind.as_class().filter(|cls| cls.is_trait()) {
+		if let Some(cls) = pointee_kind.as_class().filter(|cls| cls.kind().is_trait()) {
 			inter_vars.extend([
 				("base_rust_as_raw_const", cls.rust_as_raw_name(Constness::Const).into()),
 				("base_rust_as_raw_mut", cls.rust_as_raw_name(Constness::Mut).into()),
@@ -195,7 +195,7 @@ fn extern_functions<'tu, 'ge>(ptr: &SmartPtr<'tu, 'ge>) -> Vec<Func<'tu, 'ge>> {
 		pointee_type.with_inherent_constness(Constness::Mut),
 	));
 	out.push(FuncDesc::method_delete(smartptr_class.clone()));
-	if let Some(cls) = pointee_kind.as_class().filter(|cls| cls.is_trait()) {
+	if let Some(cls) = pointee_kind.as_class().filter(|cls| cls.kind().is_trait()) {
 		for base in all_bases(&cls) {
 			out.push(method_cast_to_base(
 				smartptr_class.clone(),
@@ -266,7 +266,7 @@ fn method_new<'tu, 'ge>(
 ) -> Func<'tu, 'ge> {
 	let pointee_kind = pointee_type.kind();
 	let val = if pointee_kind.is_copy(pointee_type.type_hint()) {
-		if pointee_kind.as_simple_class().is_some() {
+		if pointee_kind.as_class().map_or(false, |cls| cls.kind().is_simple()) {
 			panic!("Ptr with simple class is not supported");
 		} else {
 			format!("new {typ}(val)", typ = pointee_type.cpp_name(CppNameStyle::Reference)).into()

binding-generator/src/writer/rust_native/tpl/vector/rust_boxed_ref.tpl.rs

@@ -0,0 +1,3 @@
+
+vector_boxed_ref! { {{inner_rust_full}} }
+

binding-generator/src/writer/rust_native/tpl/vector/rust_copy_non_bool.tpl.rs

@@ -1,5 +1,7 @@
+
 vector_copy_non_bool! { {{inner_rust_full}},
 	{{extern_data}}, {{extern_data_mut}}, {{extern_from_slice}},
 	{{extern_clone}},
 }
 
+

binding-generator/src/writer/rust_native/tpl/vector/rust_extern.tpl.rs

@@ -10,3 +10,4 @@ vector_extern! { {{inner_rust_full}},
 	{{extern_push}}, {{extern_insert}},
 }
 
+