[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [33868] trunk/blender/source/blender: use ICON_NULL define rather then 0, makes UI calls less confusing.

Campbell Barton ideasman42 at gmail.com
Thu Dec 23 03:43:43 CET 2010


Revision: 33868
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=33868
Author:   campbellbarton
Date:     2010-12-23 03:43:40 +0100 (Thu, 23 Dec 2010)

Log Message:
-----------
use ICON_NULL define rather then 0, makes UI calls less confusing. (no functional change)
 eg: uiItemR(row, &dvar_ptr, "type", 0, "", 0); -> uiItemR(row, &dvar_ptr, "type", 0, "", ICON_NULL);

Modified Paths:
--------------
    trunk/blender/source/blender/editors/animation/anim_channels_defines.c
    trunk/blender/source/blender/editors/animation/fmodifier_ui.c
    trunk/blender/source/blender/editors/animation/keyingsets.c
    trunk/blender/source/blender/editors/armature/editarmature.c
    trunk/blender/source/blender/editors/armature/poselib.c
    trunk/blender/source/blender/editors/armature/poseobject.c
    trunk/blender/source/blender/editors/curve/editcurve.c
    trunk/blender/source/blender/editors/gpencil/gpencil_buttons.c
    trunk/blender/source/blender/editors/include/UI_interface.h
    trunk/blender/source/blender/editors/interface/interface_handlers.c
    trunk/blender/source/blender/editors/interface/interface_icons.c
    trunk/blender/source/blender/editors/interface/interface_layout.c
    trunk/blender/source/blender/editors/interface/interface_regions.c
    trunk/blender/source/blender/editors/interface/interface_templates.c
    trunk/blender/source/blender/editors/interface/interface_utils.c
    trunk/blender/source/blender/editors/interface/interface_widgets.c
    trunk/blender/source/blender/editors/object/object_add.c
    trunk/blender/source/blender/editors/object/object_constraint.c
    trunk/blender/source/blender/editors/object/object_relations.c
    trunk/blender/source/blender/editors/object/object_select.c
    trunk/blender/source/blender/editors/screen/screen_ops.c
    trunk/blender/source/blender/editors/sound/sound_ops.c
    trunk/blender/source/blender/editors/space_buttons/buttons_ops.c
    trunk/blender/source/blender/editors/space_file/file_panels.c
    trunk/blender/source/blender/editors/space_graph/graph_buttons.c
    trunk/blender/source/blender/editors/space_graph/graph_edit.c
    trunk/blender/source/blender/editors/space_image/image_buttons.c
    trunk/blender/source/blender/editors/space_image/image_header.c
    trunk/blender/source/blender/editors/space_image/image_ops.c
    trunk/blender/source/blender/editors/space_info/info_ops.c
    trunk/blender/source/blender/editors/space_info/space_info.c
    trunk/blender/source/blender/editors/space_logic/logic_window.c
    trunk/blender/source/blender/editors/space_nla/nla_buttons.c
    trunk/blender/source/blender/editors/space_nla/nla_edit.c
    trunk/blender/source/blender/editors/space_node/drawnode.c
    trunk/blender/source/blender/editors/space_node/node_draw.c
    trunk/blender/source/blender/editors/space_node/node_header.c
    trunk/blender/source/blender/editors/space_outliner/outliner.c
    trunk/blender/source/blender/editors/space_text/text_header.c
    trunk/blender/source/blender/editors/space_text/text_ops.c
    trunk/blender/source/blender/editors/space_view3d/view3d_buttons.c
    trunk/blender/source/blender/editors/space_view3d/view3d_header.c
    trunk/blender/source/blender/editors/space_view3d/view3d_select.c
    trunk/blender/source/blender/editors/space_view3d/view3d_toolbar.c
    trunk/blender/source/blender/editors/transform/transform_ops.c
    trunk/blender/source/blender/windowmanager/intern/wm_operators.c

Modified: trunk/blender/source/blender/editors/animation/anim_channels_defines.c
===================================================================
--- trunk/blender/source/blender/editors/animation/anim_channels_defines.c	2010-12-23 02:14:03 UTC (rev 33867)
+++ trunk/blender/source/blender/editors/animation/anim_channels_defines.c	2010-12-23 02:43:40 UTC (rev 33868)
@@ -3253,7 +3253,7 @@
 						uiBut *but;
 						
 						/* create the slider button, and assign relevant callback to ensure keyframes are inserted... */
-						but= uiDefAutoButR(block, &ptr, prop, array_index, "", 0, (int)v2d->cur.xmax-offset, ymid, SLIDER_WIDTH, (int)ymaxc-yminc);
+						but= uiDefAutoButR(block, &ptr, prop, array_index, "", ICON_NULL, (int)v2d->cur.xmax-offset, ymid, SLIDER_WIDTH, (int)ymaxc-yminc);
 						
 						/* assign keyframing function according to slider type */
 						if (ale->type == ANIMTYPE_SHAPEKEY)

Modified: trunk/blender/source/blender/editors/animation/fmodifier_ui.c
===================================================================
--- trunk/blender/source/blender/editors/animation/fmodifier_ui.c	2010-12-23 02:14:03 UTC (rev 33867)
+++ trunk/blender/source/blender/editors/animation/fmodifier_ui.c	2010-12-23 02:43:40 UTC (rev 33868)
@@ -225,14 +225,14 @@
 	
 	/* add the settings */
 	col= uiLayoutColumn(layout, 1);
-		uiItemR(col, &ptr, "function_type", 0, "", 0);
-		uiItemR(col, &ptr, "use_additive", UI_ITEM_R_TOGGLE, NULL, 0);
+		uiItemR(col, &ptr, "function_type", 0, "", ICON_NULL);
+		uiItemR(col, &ptr, "use_additive", UI_ITEM_R_TOGGLE, NULL, ICON_NULL);
 	
 	col= uiLayoutColumn(layout, 0); // no grouping for now
-		uiItemR(col, &ptr, "amplitude", 0, NULL, 0);
-		uiItemR(col, &ptr, "phase_multiplier", 0, NULL, 0);
-		uiItemR(col, &ptr, "phase_offset", 0, NULL, 0);
-		uiItemR(col, &ptr, "value_offset", 0, NULL, 0);
+		uiItemR(col, &ptr, "amplitude", 0, NULL, ICON_NULL);
+		uiItemR(col, &ptr, "phase_multiplier", 0, NULL, ICON_NULL);
+		uiItemR(col, &ptr, "phase_offset", 0, NULL, ICON_NULL);
+		uiItemR(col, &ptr, "value_offset", 0, NULL, ICON_NULL);
 }
 
 /* --------------- */
@@ -253,15 +253,15 @@
 	
 	/* before range */
 	col= uiLayoutColumn(split, 1);
-		uiItemL(col, "Before:", 0);
-		uiItemR(col, &ptr, "mode_before", 0, "", 0);
-		uiItemR(col, &ptr, "cycles_before", 0, NULL, 0);
+		uiItemL(col, "Before:", ICON_NULL);
+		uiItemR(col, &ptr, "mode_before", 0, "", ICON_NULL);
+		uiItemR(col, &ptr, "cycles_before", 0, NULL, ICON_NULL);
 		
 	/* after range */
 	col= uiLayoutColumn(split, 1);
-		uiItemL(col, "After:", 0);
-		uiItemR(col, &ptr, "mode_after", 0, "", 0);
-		uiItemR(col, &ptr, "cycles_after", 0, NULL, 0);
+		uiItemL(col, "After:", ICON_NULL);
+		uiItemR(col, &ptr, "mode_after", 0, "", ICON_NULL);
+		uiItemR(col, &ptr, "cycles_after", 0, NULL, ICON_NULL);
 }
 
 /* --------------- */
@@ -276,20 +276,20 @@
 	RNA_pointer_create(id, &RNA_FModifierNoise, fcm, &ptr);
 	
 	/* blending mode */
-	uiItemR(layout, &ptr, "blend_type", 0, NULL, 0);
+	uiItemR(layout, &ptr, "blend_type", 0, NULL, ICON_NULL);
 	
 	/* split into 2 columns */
 	split= uiLayoutSplit(layout, 0.5f, 0);
 	
 	/* col 1 */
 	col= uiLayoutColumn(split, 0);
-		uiItemR(col, &ptr, "scale", 0, NULL, 0);
-		uiItemR(col, &ptr, "strength", 0, NULL, 0);
+		uiItemR(col, &ptr, "scale", 0, NULL, ICON_NULL);
+		uiItemR(col, &ptr, "strength", 0, NULL, ICON_NULL);
 	
 	/* col 2 */
 	col= uiLayoutColumn(split, 0);
-		uiItemR(col, &ptr, "phase", 0, NULL, 0);
-		uiItemR(col, &ptr, "depth", 0, NULL, 0);
+		uiItemR(col, &ptr, "phase", 0, NULL, ICON_NULL);
+		uiItemR(col, &ptr, "depth", 0, NULL, ICON_NULL);
 }
 
 /* --------------- */
@@ -469,12 +469,12 @@
 	
 	/* general settings */
 	col= uiLayoutColumn(layout, 1);
-		uiItemL(col, "Envelope:", 0);
-		uiItemR(col, &ptr, "reference_value", 0, NULL, 0);
+		uiItemL(col, "Envelope:", ICON_NULL);
+		uiItemR(col, &ptr, "reference_value", 0, NULL, ICON_NULL);
 		
 		row= uiLayoutRow(col, 1);
-			uiItemR(row, &ptr, "default_min", 0, "Min", 0);
-			uiItemR(row, &ptr, "default_max", 0, "Max", 0);
+			uiItemR(row, &ptr, "default_min", 0, "Min", ICON_NULL);
+			uiItemR(row, &ptr, "default_max", 0, "Max", ICON_NULL);
 			
 	/* control points header */
 	// TODO: move this control-point control stuff to using the new special widgets for lists
@@ -526,13 +526,13 @@
 		
 		/* x-minimum */
 		col= uiLayoutColumn(split, 1);
-			uiItemR(col, &ptr, "use_min_x", 0, NULL, 0);
-			uiItemR(col, &ptr, "min_x", 0, NULL, 0);
+			uiItemR(col, &ptr, "use_min_x", 0, NULL, ICON_NULL);
+			uiItemR(col, &ptr, "min_x", 0, NULL, ICON_NULL);
 			
 		/* y-minimum*/
 		col= uiLayoutColumn(split, 1);
-			uiItemR(col, &ptr, "use_min_y", 0, NULL, 0);
-			uiItemR(col, &ptr, "min_y", 0, NULL, 0);
+			uiItemR(col, &ptr, "use_min_y", 0, NULL, ICON_NULL);
+			uiItemR(col, &ptr, "min_y", 0, NULL, ICON_NULL);
 	}
 	
 	/* row 2: maximum */
@@ -544,13 +544,13 @@
 		
 		/* x-minimum */
 		col= uiLayoutColumn(split, 1);
-			uiItemR(col, &ptr, "use_max_x", 0, NULL, 0);
-			uiItemR(col, &ptr, "max_x", 0, NULL, 0);
+			uiItemR(col, &ptr, "use_max_x", 0, NULL, ICON_NULL);
+			uiItemR(col, &ptr, "max_x", 0, NULL, ICON_NULL);
 			
 		/* y-minimum*/
 		col= uiLayoutColumn(split, 1);
-			uiItemR(col, &ptr, "use_max_y", 0, NULL, 0);
-			uiItemR(col, &ptr, "max_y", 0, NULL, 0);
+			uiItemR(col, &ptr, "use_max_y", 0, NULL, ICON_NULL);
+			uiItemR(col, &ptr, "max_y", 0, NULL, ICON_NULL);
 	}
 }
 
@@ -567,24 +567,24 @@
 	
 	/* block 1: "stepping" settings */
 	col= uiLayoutColumn(layout, 0);
-		uiItemR(col, &ptr, "frame_step", 0, NULL, 0);
-		uiItemR(col, &ptr, "frame_offset", 0, NULL, 0);
+		uiItemR(col, &ptr, "frame_step", 0, NULL, ICON_NULL);
+		uiItemR(col, &ptr, "frame_offset", 0, NULL, ICON_NULL);
 		
 	/* block 2: start range settings */
 	col= uiLayoutColumn(layout, 1);
-		uiItemR(col, &ptr, "use_frame_start", 0, NULL, 0);
+		uiItemR(col, &ptr, "use_frame_start", 0, NULL, ICON_NULL);
 		
 		subcol = uiLayoutColumn(col, 1);
 		uiLayoutSetActive(subcol, RNA_boolean_get(&ptr, "use_frame_start"));
-			uiItemR(subcol, &ptr, "frame_start", 0, NULL, 0);
+			uiItemR(subcol, &ptr, "frame_start", 0, NULL, ICON_NULL);
 			
 	/* block 3: end range settings */
 	col= uiLayoutColumn(layout, 1);
-		uiItemR(col, &ptr, "use_frame_end", 0, NULL, 0);
+		uiItemR(col, &ptr, "use_frame_end", 0, NULL, ICON_NULL);
 		
 		subcol = uiLayoutColumn(col, 1);
 		uiLayoutSetActive(subcol, RNA_boolean_get(&ptr, "use_frame_end"));
-			uiItemR(subcol, &ptr, "frame_end", 0, NULL, 0);
+			uiItemR(subcol, &ptr, "frame_end", 0, NULL, ICON_NULL);
 }
 
 /* --------------- */
@@ -616,16 +616,16 @@
 		uiBlockSetEmboss(block, UI_EMBOSSN);
 		
 		/* expand */
-		uiItemR(subrow, &ptr, "show_expanded", UI_ITEM_R_ICON_ONLY, "", 0);
+		uiItemR(subrow, &ptr, "show_expanded", UI_ITEM_R_ICON_ONLY, "", ICON_NULL);
 		
 		/* checkbox for 'active' status (for now) */
-		uiItemR(subrow, &ptr, "active", UI_ITEM_R_ICON_ONLY, "", 0);
+		uiItemR(subrow, &ptr, "active", UI_ITEM_R_ICON_ONLY, "", ICON_NULL);
 		
 		/* name */
 		if (fmi)
-			uiItemL(subrow, fmi->name, 0);
+			uiItemL(subrow, fmi->name, ICON_NULL);
 		else
-			uiItemL(subrow, "<Unknown Modifier>", 0);
+			uiItemL(subrow, "<Unknown Modifier>", ICON_NULL);
 		
 		/* right-align ------------------------------------------- */
 		subrow= uiLayoutRow(row, 0);
@@ -633,7 +633,7 @@
 		
 		
 		/* 'mute' button */
-		uiItemR(subrow, &ptr, "mute", UI_ITEM_R_ICON_ONLY, "", 0);
+		uiItemR(subrow, &ptr, "mute", UI_ITEM_R_ICON_ONLY, "", ICON_NULL);
 		
 		uiBlockSetEmboss(block, UI_EMBOSSN);
 		

Modified: trunk/blender/source/blender/editors/animation/keyingsets.c
===================================================================
--- trunk/blender/source/blender/editors/animation/keyingsets.c	2010-12-23 02:14:03 UTC (rev 33867)
+++ trunk/blender/source/blender/editors/animation/keyingsets.c	2010-12-23 02:43:40 UTC (rev 33868)
@@ -700,14 +700,14 @@
 	uiLayout *layout;
 	int i = 0;
 	
-	pup= uiPupMenuBegin(C, title, 0);
+	pup= uiPupMenuBegin(C, title, ICON_NULL);
 	layout= uiPupMenuLayout(pup);
 	
 	/* active Keying Set 
 	 *	- only include entry if it exists
 	 */
 	if (scene->active_keyingset) {
-		uiItemIntO(layout, "Active Keying Set", 0, op_name, "type", i++);
+		uiItemIntO(layout, "Active Keying Set", ICON_NULL, op_name, "type", i++);
 		uiItemS(layout);
 	}
 	else
@@ -719,7 +719,7 @@
 	if (scene->keyingsets.first) {
 		for (ks= scene->keyingsets.first; ks; ks= ks->next) {
 			if (ANIM_keyingset_context_ok_poll(C, ks))
-				uiItemIntO(layout, ks->name, 0, op_name, "type", i++);
+				uiItemIntO(layout, ks->name, ICON_NULL, op_name, "type", i++);
 		}
 		uiItemS(layout);
 	}
@@ -729,7 +729,7 @@
 	for (ks= builtin_keyingsets.first; ks; ks= ks->next) {
 		/* only show KeyingSet if context is suitable */
 		if (ANIM_keyingset_context_ok_poll(C, ks))
-			uiItemIntO(layout, ks->name, 0, op_name, "type", i--);
+			uiItemIntO(layout, ks->name, ICON_NULL, op_name, "type", i--);
 	}
 	
 	uiPupMenuEnd(C, pup);

Modified: trunk/blender/source/blender/editors/armature/editarmature.c
===================================================================
--- trunk/blender/source/blender/editors/armature/editarmature.c	2010-12-23 02:14:03 UTC (rev 33867)
+++ trunk/blender/source/blender/editors/armature/editarmature.c	2010-12-23 02:43:40 UTC (rev 33868)
@@ -3946,7 +3946,7 @@
 static int armature_parent_set_invoke(bContext *C, wmOperator *UNUSED(op), wmEvent *UNUSED(event))
 {
 	EditBone *actbone = CTX_data_active_bone(C);
-	uiPopupMenu *pup= uiPupMenuBegin(C, "Make Parent ", 0);
+	uiPopupMenu *pup= uiPupMenuBegin(C, "Make Parent ", ICON_NULL);
 	uiLayout *layout= uiPupMenuLayout(pup);
 	int allchildbones = 0;
 	

Modified: trunk/blender/source/blender/editors/armature/poselib.c
===================================================================
--- trunk/blender/source/blender/editors/armature/poselib.c	2010-12-23 02:14:03 UTC (rev 33867)
+++ trunk/blender/source/blender/editors/armature/poselib.c	2010-12-23 02:43:40 UTC (rev 33868)
@@ -304,17 +304,17 @@
 		return OPERATOR_CANCELLED;
 	
 	/* start building */
-	pup= uiPupMenuBegin(C, op->type->name, 0);
+	pup= uiPupMenuBegin(C, op->type->name, ICON_NULL);
 	layout= uiPupMenuLayout(pup);
 	uiLayoutSetOperatorContext(layout, WM_OP_EXEC_DEFAULT);
 	
 	/* add new (adds to the first unoccupied frame) */
-	uiItemIntO(layout, "Add New", 0, "POSELIB_OT_pose_add", "frame", poselib_get_free_index(ob->poselib));

@@ Diff output truncated at 10240 characters. @@




More information about the Bf-blender-cvs mailing list