Fix T66706: crash in viewport shading popover for workbench engine
authorBrecht Van Lommel <brechtvanlommel@gmail.com>
Thu, 11 Jul 2019 12:13:22 +0000 (14:13 +0200)
committerBrecht Van Lommel <brechtvanlommel@gmail.com>
Thu, 11 Jul 2019 13:03:13 +0000 (15:03 +0200)
Differential Revision: https://developer.blender.org/D5226

source/blender/makesrna/intern/rna_space.c

index 1c76753fb1ba2014a8fcd59065d9b4bf3b25243d..2ba5e2ae68c7785c6876b96845c69c71172be5d8 100644 (file)
@@ -1067,11 +1067,7 @@ static const EnumPropertyItem *rna_View3DShading_color_type_itemf(bContext *UNUS
 
   int totitem = 0;
 
-  if (shading->type == OB_SOLID) {
-    r_free = false;
-    return rna_enum_shading_color_type_items;
-  }
-  else if (shading->type == OB_WIRE) {
+  if (shading->type == OB_WIRE) {
     EnumPropertyItem *item = NULL;
     RNA_enum_items_add_value(
         &item, &totitem, rna_enum_shading_color_type_items, V3D_SHADING_SINGLE_COLOR);
@@ -1084,8 +1080,9 @@ static const EnumPropertyItem *rna_View3DShading_color_type_itemf(bContext *UNUS
     return item;
   }
   else {
-    *r_free = false;
-    return NULL;
+    /* Solid mode, or lookdev mode for workbench engine. */
+    r_free = false;
+    return rna_enum_shading_color_type_items;
   }
 }