diff options
author | Bastien Montagne <montagne29@wanadoo.fr> | 2016-05-16 16:54:18 +0300 |
---|---|---|
committer | Bastien Montagne <montagne29@wanadoo.fr> | 2016-05-16 16:54:18 +0300 |
commit | 460d94449348647b20fb0d69f60ecc0e6cc36370 (patch) | |
tree | cea4d45ed92cef1b26b00a7fd5563d3552152cc5 /source/blender/makesrna/intern/rna_brush.c | |
parent | 22d53e558d4c3204ecfe5a0775d9900f4afcb38f (diff) |
Cleanup rna_Brush_direction_itemf()
Some variants of gcc compilation were reporting 'control reaching end of non-void function' error
in this switch/case maze. Either use break everywhere or not at all (which is simpler, since we
only always return anyway...).
Diffstat (limited to 'source/blender/makesrna/intern/rna_brush.c')
-rw-r--r-- | source/blender/makesrna/intern/rna_brush.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/source/blender/makesrna/intern/rna_brush.c b/source/blender/makesrna/intern/rna_brush.c index e13919dd1a1..ac348c1750c 100644 --- a/source/blender/makesrna/intern/rna_brush.c +++ b/source/blender/makesrna/intern/rna_brush.c @@ -530,10 +530,12 @@ static EnumPropertyItem *rna_Brush_direction_itemf(bContext *C, PointerRNA *ptr, switch ((BrushMaskTool)me->mask_tool) { case BRUSH_MASK_DRAW: return prop_direction_items; - break; + case BRUSH_MASK_SMOOTH: return prop_default_items; - break; + + default: + return prop_default_items; } case SCULPT_TOOL_FLATTEN: @@ -554,7 +556,6 @@ static EnumPropertyItem *rna_Brush_direction_itemf(bContext *C, PointerRNA *ptr, default: return prop_default_items; } - break; case ePaintTexture2D: case ePaintTextureProjective: @@ -565,7 +566,6 @@ static EnumPropertyItem *rna_Brush_direction_itemf(bContext *C, PointerRNA *ptr, default: return prop_default_items; } - break; default: return prop_default_items; |