[Bf-blender-cvs] [d5c177bc904] master: Cleanup: use switch statement instead of if/else on enum value...

Bastien Montagne noreply at git.blender.org
Tue Mar 24 16:30:21 CET 2020


Commit: d5c177bc90434f8af5bc4a5cadda9cf02737882f
Author: Bastien Montagne
Date:   Tue Mar 24 16:25:30 2020 +0100
Branches: master
https://developer.blender.org/rBd5c177bc90434f8af5bc4a5cadda9cf02737882f

Cleanup: use switch statement instead of if/else on enum value...

===================================================================

M	source/blender/modifiers/intern/MOD_solidify.c

===================================================================

diff --git a/source/blender/modifiers/intern/MOD_solidify.c b/source/blender/modifiers/intern/MOD_solidify.c
index 8ea0a602b65..41852272787 100644
--- a/source/blender/modifiers/intern/MOD_solidify.c
+++ b/source/blender/modifiers/intern/MOD_solidify.c
@@ -70,11 +70,13 @@ static void requiredDataMask(Object *UNUSED(ob),
 static Mesh *applyModifier(ModifierData *md, const ModifierEvalContext *ctx, Mesh *mesh)
 {
   const SolidifyModifierData *smd = (SolidifyModifierData *)md;
-  if (smd->mode == MOD_SOLIDIFY_MODE_EXTRUDE) {
-    return MOD_solidify_extrude_applyModifier(md, ctx, mesh);
-  }
-  else if (smd->mode == MOD_SOLIDIFY_MODE_NONMANIFOLD) {
-    return MOD_solidify_nonmanifold_applyModifier(md, ctx, mesh);
+  switch (smd->mode) {
+    case MOD_SOLIDIFY_MODE_EXTRUDE:
+      return MOD_solidify_extrude_applyModifier(md, ctx, mesh);
+    case MOD_SOLIDIFY_MODE_NONMANIFOLD:
+      return MOD_solidify_nonmanifold_applyModifier(md, ctx, mesh);
+    default:
+      BLI_assert(0);
   }
   return mesh;
 }



More information about the Bf-blender-cvs mailing list