From 691654487de7e254df26dee4209c0ba3163aa864 Mon Sep 17 00:00:00 2001 From: Hanno Braun Date: Mon, 21 Feb 2022 16:11:50 +0100 Subject: [PATCH] Change order of fields in `Face` I think this makes more sense, since the surface is the defining element of the face, and the edges only make sense in reference to the surface. This way, it's also consistent with the order of fields in `Edge`. --- src/kernel/topology/faces.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/kernel/topology/faces.rs b/src/kernel/topology/faces.rs index 4c38e0471..051189787 100644 --- a/src/kernel/topology/faces.rs +++ b/src/kernel/topology/faces.rs @@ -51,6 +51,9 @@ pub enum Face { /// A face is defined by a surface, and is bounded by edges that lie in that /// surface. Face { + /// The surface that defines this face + surface: Surface, + /// The edges that bound the face /// /// # Implementation Note @@ -62,9 +65,6 @@ pub enum Face { /// more specialized data structure here, that specifies the edges in /// surface coordinates. edges: Edges, - - /// The surface that defines this face - surface: Surface, }, /// The triangles of the face