[Bf-blender-cvs] [6dbc820633e] blender2.8: Fix crash creating enum tooltips

Campbell Barton noreply at git.blender.org
Thu Nov 29 22:03:57 CET 2018


Commit: 6dbc820633eb06d66bb9cbf6880ec7b37c622cd9
Author: Campbell Barton
Date:   Fri Nov 30 08:01:41 2018 +1100
Branches: blender2.8
https://developer.blender.org/rB6dbc820633eb06d66bb9cbf6880ec7b37c622cd9

Fix crash creating enum tooltips

Error from recent changes to shortcut display.

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

M	source/blender/editors/interface/interface.c

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

diff --git a/source/blender/editors/interface/interface.c b/source/blender/editors/interface/interface.c
index 443ff55d8b4..82238744205 100644
--- a/source/blender/editors/interface/interface.c
+++ b/source/blender/editors/interface/interface.c
@@ -1059,6 +1059,11 @@ static bool ui_but_event_property_operator_string(
         char *buf, const size_t buf_len)
 {
 	/* context toggle operator names to check... */
+
+	/* This function could use a refactor to generalize button type to operator relationship
+	 * as well as which operators use properties.
+	 * - Campbell
+	 * */
 	const char *ctx_toggle_opnames[] = {
 		"WM_OT_context_toggle",
 		"WM_OT_context_toggle_enum",
@@ -1092,8 +1097,9 @@ static bool ui_but_event_property_operator_string(
 	if ((but->type == UI_BTYPE_BUT_MENU) && (but->block->handle != NULL)) {
 		uiBut *but_parent = but->block->handle->popup_create_vars.but;
 		if ((but->type == UI_BTYPE_BUT_MENU) &&
+		    (but_parent && but_parent->rnaprop) &&
 		    (RNA_property_type(but_parent->rnaprop) == PROP_ENUM) &&
-		    but_parent && (but_parent->menu_create_func == ui_def_but_rna__menu))
+		    (but_parent->menu_create_func == ui_def_but_rna__menu))
 		{
 			prop_enum_value = (int)but->hardmin;
 			ptr = &but_parent->rnapoin;



More information about the Bf-blender-cvs mailing list