Merge pull request #346 from palantir/no-panics

Remove fallible builders
palantir · Apr 21, 2024 · 0e905ef · 0e905ef
2 parents e5d6df3 + ff47372
commit 0e905ef
Show file tree

Hide file tree

Showing 106 changed files with 4,187 additions and 3,046 deletions.
diff --git a/changelog/@unreleased/pr-346.v2.yml b/changelog/@unreleased/pr-346.v2.yml
@@ -0,0 +1,5 @@
+type: break
+break:
+  description: Removed support for fallible builders.
+  links:
+  - https://github.com/palantir/conjure-rust/pull/346
diff --git a/conjure-codegen/src/context.rs b/conjure-codegen/src/context.rs
@@ -40,7 +40,6 @@ struct TypeContext {
 pub struct Context {
     types: HashMap<TypeName, TypeContext>,
     exhaustive: bool,
-    staged_builders: bool,
     strip_prefix: Vec<String>,
     version: Option<String>,
 }
@@ -49,14 +48,12 @@ impl Context {
     pub fn new(
         defs: &ConjureDefinition,
         exhaustive: bool,
-        staged_builders: bool,
         strip_prefix: Option<&str>,
         version: Option<&str>,
     ) -> Context {
         let mut context = Context {
             types: HashMap::new(),
             exhaustive,
-            staged_builders,
             strip_prefix: vec![],
             version: version.map(str::to_owned),
         };
@@ -103,10 +100,6 @@ impl Context {
         self.exhaustive
     }
 
-    pub fn staged_builders(&self) -> bool {
-        self.staged_builders
-    }
-
     fn needs_box(&self, def: &Type) -> bool {
         match def {
             Type::Primitive(_) => false,

diff --git a/conjure-codegen/src/example_types/another/different_package.rs b/conjure-codegen/src/example_types/another/different_package.rs
diff --git a/conjure-codegen/src/example_types/product/alias_as_map_key_example.rs b/conjure-codegen/src/example_types/product/alias_as_map_key_example.rs