diff options
| author | Campbell Barton <campbell@blender.org> | 2022-08-31 08:58:33 +0300 |
|---|---|---|
| committer | Campbell Barton <campbell@blender.org> | 2022-08-31 09:01:29 +0300 |
| commit | 70035e64732415aee039196e81ab31554e5577b1 (patch) | |
| tree | daa4a821962b0948fd038265fd16d95707b98fcb /source/blender/editors/object | |
| parent | ff651a08b50cade1adf6e4d7a810500c88eb1987 (diff) | |
Cleanup: break before the default case in switch statements
While missing the break before a default that only breaks isn't
an error, it means adding new cases needs to remember to add the
break for an existing case, changing the default case will also
result in an unintended fall-through.
Also avoid `default:;` and add an explicit break.
Diffstat (limited to 'source/blender/editors/object')
| -rw-r--r-- | source/blender/editors/object/object_vgroup.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source/blender/editors/object/object_vgroup.cc b/source/blender/editors/object/object_vgroup.cc index 0e0f5bd94cc..7a61adfb95c 100644 --- a/source/blender/editors/object/object_vgroup.cc +++ b/source/blender/editors/object/object_vgroup.cc @@ -1773,7 +1773,8 @@ static void vgroup_lock_all(Object *ob, int action, int mask) continue; } break; - default:; + default: + break; } if (dg->flag & DG_LOCK_WEIGHT) { @@ -1795,7 +1796,8 @@ static void vgroup_lock_all(Object *ob, int action, int mask) continue; } break; - default:; + default: + break; } switch (action) { |