[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [46903] trunk/blender: style cleanup

Campbell Barton ideasman42 at gmail.com
Tue May 22 18:24:10 CEST 2012


Revision: 46903
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=46903
Author:   campbellbarton
Date:     2012-05-22 16:24:09 +0000 (Tue, 22 May 2012)
Log Message:
-----------
style cleanup

also fix example for mesh uv's

Modified Paths:
--------------
    trunk/blender/doc/python_api/examples/bpy.types.Mesh.py
    trunk/blender/source/blender/editors/interface/interface_widgets.c
    trunk/blender/source/blender/editors/object/object_modifier.c
    trunk/blender/source/blender/editors/sculpt_paint/paint_mask.c
    trunk/blender/source/blender/editors/space_file/fsmenu.c
    trunk/blender/source/blender/modifiers/intern/MOD_skin.c

Modified: trunk/blender/doc/python_api/examples/bpy.types.Mesh.py
===================================================================
--- trunk/blender/doc/python_api/examples/bpy.types.Mesh.py	2012-05-22 16:09:31 UTC (rev 46902)
+++ trunk/blender/doc/python_api/examples/bpy.types.Mesh.py	2012-05-22 16:24:09 UTC (rev 46903)
@@ -28,7 +28,7 @@
 import bpy
 
 me = bpy.context.object.data
-uv_layer = me.uv.layers.active.data
+uv_layer = me.uv_layers.active.data
 
 for poly in me.polygons:
     print("Polygon index: %d, length: %d" % (poly.index, poly.loop_total))

Modified: trunk/blender/source/blender/editors/interface/interface_widgets.c
===================================================================
--- trunk/blender/source/blender/editors/interface/interface_widgets.c	2012-05-22 16:09:31 UTC (rev 46902)
+++ trunk/blender/source/blender/editors/interface/interface_widgets.c	2012-05-22 16:24:09 UTC (rev 46903)
@@ -1165,11 +1165,13 @@
 	
 	if (fstyle->kerning == 1)
 		BLF_disable(fstyle->uifont_id, BLF_KERNING_DEFAULT);
-	
-	//	ui_rasterpos_safe(x, y, but->aspect);
-//	if (but->type==IDPOIN) transopts= 0;	// no translation, of course!
-//	else transopts= ui_translate_buttons();
-	
+
+#if 0
+	ui_rasterpos_safe(x, y, but->aspect);
+	if (but->type == IDPOIN) transopts = 0;	// no translation, of course!
+	else transopts = ui_translate_buttons();
+#endif
+
 	/* cut string in 2 parts - only for menu entries */
 	if ((but->block->flag & UI_BLOCK_LOOP)) {
 		if (ELEM5(but->type, SLI, NUM, TEX, NUMSLI, NUMABS) == 0) {
@@ -3135,9 +3137,9 @@
 				/* no text, with icon */
 				else if (!but->str[0] && but->icon) {
 					if (but->flag & UI_ICON_PREVIEW)
-						wt= widget_type(UI_WTYPE_MENU_ICON_RADIO); /* no arrows */
+						wt = widget_type(UI_WTYPE_MENU_ICON_RADIO); /* no arrows */
 					else
-						wt= widget_type(UI_WTYPE_MENU_RADIO);  /* with arrows */
+						wt = widget_type(UI_WTYPE_MENU_RADIO);  /* with arrows */
 				}
 				/* with menu arrows */
 				else

Modified: trunk/blender/source/blender/editors/object/object_modifier.c
===================================================================
--- trunk/blender/source/blender/editors/object/object_modifier.c	2012-05-22 16:09:31 UTC (rev 46902)
+++ trunk/blender/source/blender/editors/object/object_modifier.c	2012-05-22 16:24:09 UTC (rev 46903)
@@ -161,7 +161,7 @@
 			/* ensure that grid paint mask layer is created */
 			ED_sculpt_mask_layers_ensure(ob, (MultiresModifierData *)new_md);
 		}
-		else if(type == eModifierType_Skin) {
+		else if (type == eModifierType_Skin) {
 			/* ensure skin-node customdata exists */
 			modifier_skin_customdata_ensure(ob);
 		}
@@ -1139,7 +1139,7 @@
 		return OPERATOR_CANCELLED;
 	}
 
-	CTX_DATA_BEGIN (C, Object *, selob, selected_editable_objects)
+	CTX_DATA_BEGIN(C, Object *, selob, selected_editable_objects)
 	{
 		if (selob->type == OB_MESH && selob != ob) {
 			secondob = selob;
@@ -1372,13 +1372,13 @@
 static int skin_poll(bContext *C)
 {
 	return (!CTX_data_edit_object(C) &&
-	        edit_modifier_poll_generic(C, &RNA_SkinModifier, (1<<OB_MESH)));
+	        edit_modifier_poll_generic(C, &RNA_SkinModifier, (1 << OB_MESH)));
 }
 
 static int skin_edit_poll(bContext *C)
 {
 	return (CTX_data_edit_object(C) &&
-	        edit_modifier_poll_generic(C, &RNA_SkinModifier, (1<<OB_MESH)));
+	        edit_modifier_poll_generic(C, &RNA_SkinModifier, (1 << OB_MESH)));
 }
 
 static void skin_root_clear(BMesh *bm, BMVert *bm_vert, GHash *visited)
@@ -1389,7 +1389,7 @@
 	BM_ITER_ELEM (bm_edge, &bm_iter, bm_vert, BM_EDGES_OF_VERT) {
 		BMVert *v2 = BM_edge_other_vert(bm_edge, bm_vert);
 
-		if(!BLI_ghash_lookup(visited, v2)) {
+		if (!BLI_ghash_lookup(visited, v2)) {
 			MVertSkin *vs = CustomData_bmesh_get(&bm->vdata,
 			                                     v2->head.data,
 			                                     CD_MVERT_SKIN);
@@ -1405,7 +1405,7 @@
 
 static int skin_root_mark_exec(bContext *C, wmOperator *UNUSED(op))
 {
-	Object *ob= CTX_data_edit_object(C);
+	Object *ob = CTX_data_edit_object(C);
 	Mesh *me = ob->data;
 	BMesh *bm = me->edit_btmesh->bm;
 	BMVert *bm_vert;
@@ -1417,8 +1417,8 @@
 	modifier_skin_customdata_ensure(ob);
 
 	BM_ITER_MESH (bm_vert, &bm_iter, bm, BM_VERTS_OF_MESH) {
-		if(!BLI_ghash_lookup(visited, bm_vert) &&
-		   bm_vert->head.hflag & BM_ELEM_SELECT)
+		if (!BLI_ghash_lookup(visited, bm_vert) &&
+		    bm_vert->head.hflag & BM_ELEM_SELECT)
 		{
 			MVertSkin *vs = CustomData_bmesh_get(&bm->vdata,
 			                                     bm_vert->head.data,
@@ -1436,7 +1436,7 @@
 	BLI_ghash_free(visited, NULL, NULL);
 
 	DAG_id_tag_update(&ob->id, OB_RECALC_DATA);
-	WM_event_add_notifier(C, NC_OBJECT|ND_MODIFIER, ob);
+	WM_event_add_notifier(C, NC_OBJECT | ND_MODIFIER, ob);
 	
 	return OPERATOR_FINISHED;
 }
@@ -1451,7 +1451,7 @@
 	ot->exec = skin_root_mark_exec;
 	
 	/* flags */
-	ot->flag = OPTYPE_REGISTER|OPTYPE_UNDO;
+	ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 }
 
 typedef enum {
@@ -1461,7 +1461,7 @@
 
 static int skin_loose_mark_clear_exec(bContext *C, wmOperator *op)
 {
-	Object *ob= CTX_data_edit_object(C);
+	Object *ob = CTX_data_edit_object(C);
 	Mesh *me = ob->data;
 	BMesh *bm = me->edit_btmesh->bm;
 	BMVert *bm_vert;
@@ -1487,7 +1487,7 @@
 	}
 
 	DAG_id_tag_update(&ob->id, OB_RECALC_DATA);
-	WM_event_add_notifier(C, NC_OBJECT|ND_MODIFIER, ob);
+	WM_event_add_notifier(C, NC_OBJECT | ND_MODIFIER, ob);
 	
 	return OPERATOR_FINISHED;
 }
@@ -1495,9 +1495,10 @@
 void OBJECT_OT_skin_loose_mark_clear(wmOperatorType *ot)
 {
 	static EnumPropertyItem action_items[] = {
-	{SKIN_LOOSE_MARK, "MARK", 0, "Mark", "Mark selected vertices as loose"},
-	{SKIN_LOOSE_CLEAR, "CLEAR", 0, "Clear", "Set selected vertices as not loose"},
-	{0, NULL, 0, NULL, NULL}};
+		{SKIN_LOOSE_MARK, "MARK", 0, "Mark", "Mark selected vertices as loose"},
+		{SKIN_LOOSE_CLEAR, "CLEAR", 0, "Clear", "Set selected vertices as not loose"},
+		{0, NULL, 0, NULL, NULL}
+	};
 
 	ot->name = "Skin Mark/Clear Loose";
 	ot->description = "Mark/clear selected vertices as loose";
@@ -1507,14 +1508,14 @@
 	ot->exec = skin_loose_mark_clear_exec;
 	
 	/* flags */
-	ot->flag = OPTYPE_REGISTER|OPTYPE_UNDO;
+	ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 
 	RNA_def_enum(ot->srna, "action", action_items, SKIN_LOOSE_MARK, "Action", NULL);
 }
 
 static int skin_radii_equalize_exec(bContext *C, wmOperator *UNUSED(op))
 {
-	Object *ob= CTX_data_edit_object(C);
+	Object *ob = CTX_data_edit_object(C);
 	Mesh *me = ob->data;
 	BMesh *bm = me->edit_btmesh->bm;
 	BMVert *bm_vert;
@@ -1532,7 +1533,7 @@
 	}
 
 	DAG_id_tag_update(&ob->id, OB_RECALC_DATA);
-	WM_event_add_notifier(C, NC_OBJECT|ND_MODIFIER, ob);
+	WM_event_add_notifier(C, NC_OBJECT | ND_MODIFIER, ob);
 	
 	return OPERATOR_FINISHED;
 }
@@ -1547,20 +1548,20 @@
 	ot->exec = skin_radii_equalize_exec;
 	
 	/* flags */
-	ot->flag = OPTYPE_REGISTER|OPTYPE_UNDO;
+	ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 }
 
 static void skin_armature_bone_create(Object *skin_ob,
-									  MVert *mvert, MEdge *medge,
-									  bArmature *arm,
-									  BLI_bitmap edges_visited,
-									  const MeshElemMap *emap,
-									  EditBone *parent_bone,
-									  int parent_v)
+                                      MVert *mvert, MEdge *medge,
+                                      bArmature *arm,
+                                      BLI_bitmap edges_visited,
+                                      const MeshElemMap *emap,
+                                      EditBone *parent_bone,
+                                      int parent_v)
 {
 	int i;
 
-	for(i = 0; i < emap[parent_v].count; i++) {
+	for (i = 0; i < emap[parent_v].count; i++) {
 		int endx = emap[parent_v].indices[i];
 		const MEdge *e = &medge[endx];
 		EditBone *bone;
@@ -1568,14 +1569,14 @@
 		int v;
 
 		/* ignore edge if already visited */
-		if(BLI_BITMAP_GET(edges_visited, endx))
+		if (BLI_BITMAP_GET(edges_visited, endx))
 			continue;
 		BLI_BITMAP_SET(edges_visited, endx);
 
 		v = (e->v1 == parent_v ? e->v2 : e->v1);
 
 		bone = MEM_callocN(sizeof(EditBone),
-						   "skin_armature_bone_create EditBone");
+		                   "skin_armature_bone_create EditBone");
 
 		bone->parent = parent_bone;
 		bone->layer = 1;
@@ -1589,23 +1590,23 @@
 		BLI_addtail(arm->edbo, bone);
 
 		/* add bDeformGroup */
-		if((dg = ED_vgroup_add_name(skin_ob, bone->name))) {
+		if ((dg = ED_vgroup_add_name(skin_ob, bone->name))) {
 			ED_vgroup_vert_add(skin_ob, dg, parent_v, 1, WEIGHT_REPLACE);
 			ED_vgroup_vert_add(skin_ob, dg, v, 1, WEIGHT_REPLACE);
 		}
 		
 		skin_armature_bone_create(skin_ob,
-								  mvert, medge,
-								  arm,
-								  edges_visited,
-								  emap,
-								  bone,
-								  v);
+		                          mvert, medge,
+		                          arm,
+		                          edges_visited,
+		                          emap,
+		                          bone,
+		                          v);
 	}
 }
 
 static Object *modifier_skin_armature_create(struct Scene *scene,
-											 Object *skin_ob)
+                                             Object *skin_ob)
 {
 	BLI_bitmap edges_visited;
 	DerivedMesh *deform_dm;
@@ -1623,10 +1624,10 @@
 
 	/* add vertex weights to original mesh */
 	CustomData_add_layer(&me->vdata,
-						 CD_MDEFORMVERT,
-						 CD_CALLOC,
-						 NULL,
-						 me->totvert);
+	                     CD_MDEFORMVERT,
+	                     CD_CALLOC,
+	                     NULL,
+	                     me->totvert);
 	
 	arm_ob = BKE_object_add(scene, OB_ARMATURE);
 	BKE_object_transform_copy(arm_ob, skin_ob);
@@ -1638,19 +1639,19 @@
 
 	mvert_skin = CustomData_get_layer(&me->vdata, CD_MVERT_SKIN);
 	create_vert_edge_map(&emap, &emap_mem,
-						 me->medge, me->totvert, me->totedge);
+	                     me->medge, me->totvert, me->totedge);
 
 	edges_visited = BLI_BITMAP_NEW(me->totedge, "edge_visited");
 
 	/* note: we use EditBones here, easier to set them up and use
-	 * edit-armature functions to convert back to regular bones */
-	for(v = 0; v < me->totvert; v++) {
-		if(mvert_skin[v].flag & MVERT_SKIN_ROOT) {
+	* edit-armature functions to convert back to regular bones */
+	for (v = 0; v < me->totvert; v++) {
+		if (mvert_skin[v].flag & MVERT_SKIN_ROOT) {
 			EditBone *bone = NULL;
 
 			/* Unless the skin root has just one adjacent edge, create
-			   a fake root bone (have it going off in the Y direction
-			   (arbitrary) */
+			 * a fake root bone (have it going off in the Y direction
+			 * (arbitrary) */
 			if (emap[v].count > 1) {

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list