[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [55429] trunk/blender/source/blender: code cleanup: use booleans for mesh and selection code.

Campbell Barton ideasman42 at gmail.com
Wed Mar 20 00:17:45 CET 2013


Revision: 55429
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=55429
Author:   campbellbarton
Date:     2013-03-19 23:17:44 +0000 (Tue, 19 Mar 2013)
Log Message:
-----------
code cleanup: use booleans for mesh and selection code.

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/BKE_mesh.h
    trunk/blender/source/blender/blenkernel/BKE_multires.h
    trunk/blender/source/blender/blenkernel/intern/brush.c
    trunk/blender/source/blender/blenkernel/intern/mesh.c
    trunk/blender/source/blender/blenkernel/intern/multires.c
    trunk/blender/source/blender/editors/animation/anim_markers.c
    trunk/blender/source/blender/editors/armature/armature_select.c
    trunk/blender/source/blender/editors/armature/pose_select.c
    trunk/blender/source/blender/editors/curve/editcurve.c
    trunk/blender/source/blender/editors/gpencil/gpencil_edit.c
    trunk/blender/source/blender/editors/include/ED_armature.h
    trunk/blender/source/blender/editors/include/ED_curve.h
    trunk/blender/source/blender/editors/include/ED_mball.h
    trunk/blender/source/blender/editors/include/ED_mesh.h
    trunk/blender/source/blender/editors/include/ED_object.h
    trunk/blender/source/blender/editors/include/ED_particle.h
    trunk/blender/source/blender/editors/include/ED_view3d.h
    trunk/blender/source/blender/editors/include/UI_interface.h
    trunk/blender/source/blender/editors/interface/interface_layout.c
    trunk/blender/source/blender/editors/interface/interface_utils.c
    trunk/blender/source/blender/editors/io/io_ops.c
    trunk/blender/source/blender/editors/mask/mask_select.c
    trunk/blender/source/blender/editors/mesh/editface.c
    trunk/blender/source/blender/editors/mesh/editmesh_add.c
    trunk/blender/source/blender/editors/mesh/editmesh_knife.c
    trunk/blender/source/blender/editors/mesh/editmesh_loopcut.c
    trunk/blender/source/blender/editors/mesh/editmesh_rip.c
    trunk/blender/source/blender/editors/mesh/editmesh_select.c
    trunk/blender/source/blender/editors/mesh/editmesh_tools.c
    trunk/blender/source/blender/editors/mesh/editmesh_utils.c
    trunk/blender/source/blender/editors/mesh/mesh_data.c
    trunk/blender/source/blender/editors/mesh/mesh_intern.h
    trunk/blender/source/blender/editors/mesh/mesh_navmesh.c
    trunk/blender/source/blender/editors/mesh/mesh_ops.c
    trunk/blender/source/blender/editors/mesh/meshtools.c
    trunk/blender/source/blender/editors/metaball/mball_edit.c
    trunk/blender/source/blender/editors/object/object_lattice.c
    trunk/blender/source/blender/editors/object/object_modifier.c
    trunk/blender/source/blender/editors/object/object_relations.c
    trunk/blender/source/blender/editors/object/object_select.c
    trunk/blender/source/blender/editors/physics/particle_edit.c
    trunk/blender/source/blender/editors/render/render_ops.c
    trunk/blender/source/blender/editors/screen/screendump.c
    trunk/blender/source/blender/editors/sound/sound_ops.c
    trunk/blender/source/blender/editors/space_console/console_intern.h
    trunk/blender/source/blender/editors/space_console/console_ops.c
    trunk/blender/source/blender/editors/space_file/file_panels.c
    trunk/blender/source/blender/editors/space_image/image_ops.c
    trunk/blender/source/blender/editors/space_sequencer/sequencer_select.c
    trunk/blender/source/blender/editors/space_view3d/view3d_select.c
    trunk/blender/source/blender/editors/uvedit/uvedit_ops.c
    trunk/blender/source/blender/python/intern/bpy_operator_wrap.c
    trunk/blender/source/blender/windowmanager/WM_api.h
    trunk/blender/source/blender/windowmanager/intern/wm_cursors.c
    trunk/blender/source/blender/windowmanager/intern/wm_operators.c

Modified: trunk/blender/source/blender/blenkernel/BKE_mesh.h
===================================================================
--- trunk/blender/source/blender/blenkernel/BKE_mesh.h	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/blenkernel/BKE_mesh.h	2013-03-19 23:17:44 UTC (rev 55429)
@@ -212,7 +212,7 @@
         struct MVert *mverts, int numVerts,
         struct MLoop *mloop, struct MPoly *mpolys, int numLoops, int numPolys, float (*polyNors_r)[3],
         struct MFace *mfaces, int numFaces, int *origIndexFace, float (*faceNors_r)[3],
-        const short only_face_normals);
+        const bool only_face_normals);
 
 void BKE_mesh_calc_normals(
         struct MVert *mverts, int numVerts,

Modified: trunk/blender/source/blender/blenkernel/BKE_multires.h
===================================================================
--- trunk/blender/source/blender/blenkernel/BKE_multires.h	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/blenkernel/BKE_multires.h	2013-03-19 23:17:44 UTC (rev 55429)
@@ -75,7 +75,7 @@
 
 struct MultiresModifierData *find_multires_modifier_before(struct Scene *scene,
                                                            struct ModifierData *lastmd);
-struct MultiresModifierData *get_multires_modifier(struct Scene *scene, struct Object *ob, int use_first);
+struct MultiresModifierData *get_multires_modifier(struct Scene *scene, struct Object *ob, bool use_first);
 struct DerivedMesh *get_multires_dm(struct Scene *scene, struct MultiresModifierData *mmd,
                                     struct Object *ob);
 void multiresModifier_del_levels(struct MultiresModifierData *, struct Object *, int direction);

Modified: trunk/blender/source/blender/blenkernel/intern/brush.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/brush.c	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/blenkernel/intern/brush.c	2013-03-19 23:17:44 UTC (rev 55429)
@@ -57,12 +57,14 @@
 
 static RNG *brush_rng;
 
-void BKE_brush_system_init(void) {
+void BKE_brush_system_init(void)
+{
 	brush_rng = BLI_rng_new(0);
 	BLI_rng_srandom(brush_rng, 31415682);
 }
 
-void BKE_brush_system_exit(void) {
+void BKE_brush_system_exit(void)
+{
 	BLI_rng_free(brush_rng);
 }
 

Modified: trunk/blender/source/blender/blenkernel/intern/mesh.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/mesh.c	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/blenkernel/intern/mesh.c	2013-03-19 23:17:44 UTC (rev 55429)
@@ -1814,7 +1814,7 @@
                                       MLoop *mloop, MPoly *mpolys,
                                       int numLoops, int numPolys, float (*polyNors_r)[3],
                                       MFace *mfaces, int numFaces, int *origIndexFace, float (*faceNors_r)[3],
-                                      const short only_face_normals)
+                                      const bool only_face_normals)
 {
 	float (*pnors)[3] = polyNors_r, (*fnors)[3] = faceNors_r;
 	int i;

Modified: trunk/blender/source/blender/blenkernel/intern/multires.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/multires.c	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/blenkernel/intern/multires.c	2013-03-19 23:17:44 UTC (rev 55429)
@@ -310,7 +310,7 @@
 /* used for applying scale on mdisps layer and syncing subdivide levels when joining objects
  * use_first - return first multires modifier if all multires'es are disabled
  */
-MultiresModifierData *get_multires_modifier(Scene *scene, Object *ob, int use_first)
+MultiresModifierData *get_multires_modifier(Scene *scene, Object *ob, bool use_first)
 {
 	ModifierData *md;
 	MultiresModifierData *mmd = NULL, *firstmmd = NULL;

Modified: trunk/blender/source/blender/editors/animation/anim_markers.c
===================================================================
--- trunk/blender/source/blender/editors/animation/anim_markers.c	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/editors/animation/anim_markers.c	2013-03-19 23:17:44 UTC (rev 55429)
@@ -1037,7 +1037,7 @@
 	}
 }
 
-static int ed_marker_select(bContext *C, const wmEvent *event, int extend, int camera)
+static int ed_marker_select(bContext *C, const wmEvent *event, bool extend, bool camera)
 {
 	ListBase *markers = ED_context_get_markers(C);
 	ARegion *ar = CTX_wm_region(C);
@@ -1106,8 +1106,8 @@
 
 static int ed_marker_select_invoke(bContext *C, wmOperator *op, const wmEvent *event)
 {
-	short extend = RNA_boolean_get(op->ptr, "extend");
-	short camera = 0;
+	bool extend = RNA_boolean_get(op->ptr, "extend");
+	bool camera = false;
 #ifdef DURIAN_CAMERA_SWITCH
 	camera = RNA_boolean_get(op->ptr, "camera");
 #endif

Modified: trunk/blender/source/blender/editors/armature/armature_select.c
===================================================================
--- trunk/blender/source/blender/editors/armature/armature_select.c	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/editors/armature/armature_select.c	2013-03-19 23:17:44 UTC (rev 55429)
@@ -454,7 +454,7 @@
 }
 
 /* context: editmode armature in view3d */
-int mouse_armature(bContext *C, const int mval[2], int extend, int deselect, int toggle)
+bool mouse_armature(bContext *C, const int mval[2], bool extend, bool deselect, bool toggle)
 {
 	Object *obedit = CTX_data_edit_object(C);
 	bArmature *arm = obedit->data;
@@ -550,10 +550,10 @@
 		}
 		
 		WM_event_add_notifier(C, NC_OBJECT | ND_BONE_SELECT, vc.obedit);
-		return 1;
+		return true;
 	}
 
-	return 0;
+	return false;
 }
 
 

Modified: trunk/blender/source/blender/editors/armature/pose_select.c
===================================================================
--- trunk/blender/source/blender/editors/armature/pose_select.c	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/editors/armature/pose_select.c	2013-03-19 23:17:44 UTC (rev 55429)
@@ -67,7 +67,7 @@
 /* called from editview.c, for mode-less pose selection */
 /* assumes scene obact and basact is still on old situation */
 int ED_do_pose_selectbuffer(Scene *scene, Base *base, unsigned int *buffer, short hits,
-                            short extend, short deselect, short toggle)
+                            bool extend, bool deselect, bool toggle)
 {
 	Object *ob = base->object;
 	Bone *nearBone;
@@ -575,7 +575,7 @@
 
 /* -------------------------------------- */
 
-static short pose_select_same_group(bContext *C, Object *ob, short extend)
+static short pose_select_same_group(bContext *C, Object *ob, bool extend)
 {
 	bArmature *arm = (ob) ? ob->data : NULL;
 	bPose *pose = (ob) ? ob->pose : NULL;
@@ -607,7 +607,7 @@
 		}
 		
 		/* deselect all bones before selecting new ones? */
-		if ((extend == 0) && (pchan->bone->flag & BONE_UNSELECTABLE) == 0)
+		if ((extend == false) && (pchan->bone->flag & BONE_UNSELECTABLE) == 0)
 			pchan->bone->flag &= ~BONE_SELECTED;
 	}
 	CTX_DATA_END;
@@ -634,7 +634,7 @@
 	return changed;
 }
 
-static short pose_select_same_layer(bContext *C, Object *ob, short extend)
+static short pose_select_same_layer(bContext *C, Object *ob, bool extend)
 {
 	bPose *pose = (ob) ? ob->pose : NULL;
 	bArmature *arm = (ob) ? ob->data : NULL;
@@ -652,7 +652,7 @@
 			layers |= pchan->bone->layer;
 			
 		/* deselect all bones before selecting new ones? */
-		if ((extend == 0) && (pchan->bone->flag & BONE_UNSELECTABLE) == 0)
+		if ((extend == false) && (pchan->bone->flag & BONE_UNSELECTABLE) == 0)
 			pchan->bone->flag &= ~BONE_SELECTED;
 	}
 	CTX_DATA_END;
@@ -673,7 +673,7 @@
 	return changed;
 }
 
-static int pose_select_same_keyingset(bContext *C, Object *ob, short extend)
+static int pose_select_same_keyingset(bContext *C, Object *ob, bool extend)
 {
 	KeyingSet *ks = ANIM_scene_get_active_keyingset(CTX_data_scene(C));
 	KS_Path *ksp;
@@ -690,7 +690,7 @@
 		return 0;
 		
 	/* if not extending selection, deselect all selected first */
-	if (extend == 0) {
+	if (extend == false) {
 		CTX_DATA_BEGIN (C, bPoseChannel *, pchan, visible_pose_bones)
 		{
 			if ((pchan->bone->flag & BONE_UNSELECTABLE) == 0)

Modified: trunk/blender/source/blender/editors/curve/editcurve.c
===================================================================
--- trunk/blender/source/blender/editors/curve/editcurve.c	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/editors/curve/editcurve.c	2013-03-19 23:17:44 UTC (rev 55429)
@@ -4201,7 +4201,7 @@
 
 /***************** pick select from 3d view **********************/
 
-int mouse_nurb(bContext *C, const int mval[2], int extend, int deselect, int toggle)
+bool mouse_nurb(bContext *C, const int mval[2], bool extend, bool deselect, bool toggle)
 {
 	Object *obedit = CTX_data_edit_object(C);
 	Curve *cu = obedit->data;
@@ -4314,10 +4314,10 @@
 
 		WM_event_add_notifier(C, NC_GEOM | ND_SELECT, obedit->data);
 
-		return 1;
+		return true;
 	}
 	
-	return 0;
+	return false;
 }
 
 /******************** spin operator ***********************/

Modified: trunk/blender/source/blender/editors/gpencil/gpencil_edit.c
===================================================================
--- trunk/blender/source/blender/editors/gpencil/gpencil_edit.c	2013-03-19 21:37:22 UTC (rev 55428)
+++ trunk/blender/source/blender/editors/gpencil/gpencil_edit.c	2013-03-19 23:17:44 UTC (rev 55429)
@@ -1526,7 +1526,7 @@
 	return OPERATOR_FINISHED;
 }
 
-static int gp_convert_draw_check_prop(PointerRNA *ptr, PropertyRNA *prop)
+static bool gp_convert_draw_check_prop(PointerRNA *ptr, PropertyRNA *prop)
 {
 	const char *prop_id = RNA_property_identifier(prop);
 	int link_strokes = RNA_boolean_get(ptr, "use_link_strokes");
@@ -1542,7 +1542,7 @@
 	    strcmp(prop_id, "radius_multiplier") == 0 ||
 	    strcmp(prop_id, "use_link_strokes") == 0)
 	{
-		return TRUE;
+		return true;
 	}
 	
 	/* Never show this prop */
@@ -1550,44 +1550,44 @@
 		return FALSE;
 
 	if (link_strokes) {
-		/* Only show when link_stroke is TRUE */
+		/* Only show when link_stroke is true */
 		if (strcmp(prop_id, "timing_mode") == 0)
-			return TRUE;
+			return true;
 		
 		if (timing_mode != GP_STROKECONVERT_TIMING_NONE) {
-			/* Only show when link_stroke is TRUE and stroke timing is enabled */
+			/* Only show when link_stroke is true and stroke timing is enabled */

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list