replace builders with generated traits on ObjectBuilder

src/analysis/class_builder.rs

@@ -24,40 +24,11 @@ pub fn analyze(
     }
 
     let mut names = HashSet::<String>::new();
-    let mut builder_properties = vec![(
+    let builder_properties = vec![(
         analyze_properties(env, type_tid, props, obj, imports, &mut names),
         type_tid,
     )];
 
-    for &super_tid in env.class_hierarchy.supertypes(type_tid) {
-        let type_ = env.type_(super_tid);
-
-        let super_properties = match type_ {
-            library::Type::Class(class) => &class.properties,
-            library::Type::Interface(iface) => &iface.properties,
-            _ => continue,
-        };
-        let super_obj =
-            if let Some(super_obj) = env.config.objects.get(&super_tid.full_name(&env.library)) {
-                super_obj
-            } else {
-                continue;
-            };
-
-        let new_builder_properties = (
-            analyze_properties(
-                env,
-                super_tid,
-                super_properties,
-                super_obj,
-                imports,
-                &mut names,
-            ),
-            super_tid,
-        );
-        builder_properties.push(new_builder_properties);
-    }
-
     builder_properties
 }
 

src/analysis/object.rs

@@ -39,7 +39,6 @@ pub struct Info {
     pub notify_signals: Vec<signals::Info>,
     pub properties: Vec<properties::Property>,
     pub builder_properties: Vec<(Vec<properties::Property>, TypeId)>,
-    pub builder_postprocess: Option<String>,
     pub child_properties: ChildProperties,
     pub signatures: Signatures,
     /// Specific to fundamental types
@@ -84,6 +83,12 @@ impl Info {
         self.signals.iter().any(|s| s.action_emit_name.is_some())
     }
 
+    pub fn builder_trait_prefix(&self) -> &str {
+        self.trait_name
+            .strip_suffix("Ext")
+            .unwrap_or(self.trait_name.as_str())
+    }
+
     /// Returns the location of the function within this object
     pub fn function_location(&self, fn_info: &functions::Info) -> LocationInObject {
         if self.final_type
@@ -306,7 +311,6 @@ pub fn class(env: &Env, obj: &GObject, deps: &[library::TypeId]) -> Option<Info>
         notify_signals,
         properties,
         builder_properties,
-        builder_postprocess: obj.builder_postprocess.clone(),
         child_properties,
         signatures,
         ref_fn: klass.ref_fn.clone(),
@@ -390,6 +394,13 @@ pub fn interface(env: &Env, obj: &GObject, deps: &[library::TypeId]) -> Option<I
         deps,
     );
 
+    let builder_properties =
+        class_builder::analyze(env, &iface.properties, iface_tid, obj, &mut imports);
+
+    if has_builder_properties(&builder_properties) {
+        imports.add("glib::prelude::*");
+    }
+
     let base = InfoBase {
         full_name,
         type_id: iface_tid,
@@ -420,6 +431,7 @@ pub fn interface(env: &Env, obj: &GObject, deps: &[library::TypeId]) -> Option<I
         signals,
         notify_signals,
         properties,
+        builder_properties,
         signatures,
         ..Default::default()
     };

src/codegen/doc/mod.rs

@@ -340,7 +340,7 @@ fn create_object_doc(w: &mut dyn Write, env: &Env, info: &analysis::object::Info
     })?;
 
     if has_builder {
-        let builder_ty = TypeStruct::new(SType::Impl, &format!("{}Builder", info.name));
+        let builder_ty = TypeStruct::new(SType::Trait, &format!("{}BuilderExt", info.name));
 
         let mut builder_properties: Vec<_> = properties.iter().collect();
         for parent_info in &info.supertypes {

src/codegen/mod.rs

@@ -82,12 +82,6 @@ pub fn generate_mod_rs(
             writeln!(w, "#[doc(hidden)]")?;
             writeln!(w, "pub mod traits {{")?;
             general::write_vec(w, traits)?;
-            writeln!(w, "}}")?;
-        }
-
-        if !builders.is_empty() {
-            writeln!(w, "#[doc(hidden)]")?;
-            writeln!(w, "pub mod builders {{")?;
             general::write_vec(w, builders)?;
             writeln!(w, "}}")?;
         }