From 8f75a72b001dbe71ff1b37efcb25749c6cd3dbae Mon Sep 17 00:00:00 2001 From: Hanno Braun Date: Fri, 24 May 2024 22:13:43 +0200 Subject: [PATCH] Make enum variant name more explicit --- crates/fj-core/src/validate/references.rs | 2 +- crates/fj-core/src/validate/sketch.rs | 6 ++++-- crates/fj-core/src/validate/solid.rs | 6 ++++-- 3 files changed, 9 insertions(+), 5 deletions(-) diff --git a/crates/fj-core/src/validate/references.rs b/crates/fj-core/src/validate/references.rs index d067087c9..a39554294 100644 --- a/crates/fj-core/src/validate/references.rs +++ b/crates/fj-core/src/validate/references.rs @@ -72,7 +72,7 @@ pub enum ObjectNotExclusivelyOwned { /// [`Cycle`] referenced by more than one [`Region`] #[error(transparent)] - Cycle { + MultipleReferencesToCycle { /// The invalid references references: MultipleReferences, }, diff --git a/crates/fj-core/src/validate/sketch.rs b/crates/fj-core/src/validate/sketch.rs index 08e9d9cae..d1942b4e8 100644 --- a/crates/fj-core/src/validate/sketch.rs +++ b/crates/fj-core/src/validate/sketch.rs @@ -79,7 +79,7 @@ impl SketchValidationError { validate_references!( errors, SketchValidationError; referenced_edges, MultipleReferencesToHalfEdge; - referenced_cycles, Cycle; + referenced_cycles, MultipleReferencesToCycle; ); } @@ -167,7 +167,9 @@ mod tests { core, invalid_sketch, ValidationError::ObjectNotExclusivelyOwned( - ObjectNotExclusivelyOwned::Cycle { references: _ } + ObjectNotExclusivelyOwned::MultipleReferencesToCycle { + references: _ + } ) ); diff --git a/crates/fj-core/src/validate/solid.rs b/crates/fj-core/src/validate/solid.rs index b63d8f644..8b681afa9 100644 --- a/crates/fj-core/src/validate/solid.rs +++ b/crates/fj-core/src/validate/solid.rs @@ -168,7 +168,7 @@ impl SolidValidationError { referenced_regions, MultipleReferencesToRegion; referenced_faces, MultipleReferencesToFace; referenced_edges, MultipleReferencesToHalfEdge; - referenced_cycles, Cycle; + referenced_cycles, MultipleReferencesToCycle; ); } } @@ -334,7 +334,9 @@ mod tests { core, invalid_solid, ValidationError::ObjectNotExclusivelyOwned( - ObjectNotExclusivelyOwned::Cycle { references: _ } + ObjectNotExclusivelyOwned::MultipleReferencesToCycle { + references: _ + } ) );