[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [44008] branches/bmesh/blender/source/ blender/editors: Code Cleanup: use less confusing names for selection flushing.
Campbell Barton
ideasman42 at gmail.com
Fri Feb 10 04:03:49 CET 2012
Revision: 44008
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=44008
Author: campbellbarton
Date: 2012-02-10 03:03:42 +0000 (Fri, 10 Feb 2012)
Log Message:
-----------
Code Cleanup: use less confusing names for selection flushing.
Modified Paths:
--------------
branches/bmesh/blender/source/blender/editors/include/ED_mesh.h
branches/bmesh/blender/source/blender/editors/mesh/bmesh_select.c
branches/bmesh/blender/source/blender/editors/mesh/bmesh_tools.c
branches/bmesh/blender/source/blender/editors/mesh/bmeshutils.c
branches/bmesh/blender/source/blender/editors/mesh/editbmesh_add.c
branches/bmesh/blender/source/blender/editors/mesh/loopcut.c
branches/bmesh/blender/source/blender/editors/object/object_hook.c
branches/bmesh/blender/source/blender/editors/object/object_vgroup.c
branches/bmesh/blender/source/blender/editors/space_view3d/view3d_select.c
branches/bmesh/blender/source/blender/editors/uvedit/uvedit_ops.c
Modified: branches/bmesh/blender/source/blender/editors/include/ED_mesh.h
===================================================================
--- branches/bmesh/blender/source/blender/editors/include/ED_mesh.h 2012-02-10 02:42:15 UTC (rev 44007)
+++ branches/bmesh/blender/source/blender/editors/include/ED_mesh.h 2012-02-10 03:03:42 UTC (rev 44008)
@@ -159,10 +159,10 @@
/*exactly the same as EDBM_selectmode_flush, but you pass in the selectmode
instead of using the current one*/
-void EDBM_select_flush(struct BMEditMesh *em, int selectmode);
+void EDBM_select_mode_flush(struct BMEditMesh *em, int selectmode);
-void EDBM_deselectmode_flush(struct BMEditMesh *em);
-void EDBM_selectmode_flush(struct BMEditMesh *em);
+void EDBM_deselect_flush(struct BMEditMesh *em);
+void EDBM_select_flush(struct BMEditMesh *em);
void EDBM_selectmode_set(struct BMEditMesh *em);
void EDBM_convertsel(struct BMEditMesh *em, short oldmode, short selectmode);
Modified: branches/bmesh/blender/source/blender/editors/mesh/bmesh_select.c
===================================================================
--- branches/bmesh/blender/source/blender/editors/mesh/bmesh_select.c 2012-02-10 02:42:15 UTC (rev 44007)
+++ branches/bmesh/blender/source/blender/editors/mesh/bmesh_select.c 2012-02-10 03:03:42 UTC (rev 44008)
@@ -766,7 +766,7 @@
/* select the output */
BMO_HeaderFlag_Buffer(em->bm, &bmop, "edgeout", BM_SELECT, BM_ALL);
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
/* finish the operator */
if (!EDBM_FinishOp(em, &bmop, op, TRUE)) {
@@ -817,7 +817,7 @@
return OPERATOR_CANCELLED;
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
/* dependencies graph and notification stuff */
DAG_id_tag_update(ob->data, OB_RECALC_DATA);
@@ -956,14 +956,14 @@
eed = edarray[edindex];
walker_select(em, BMW_EDGERING, eed, 1);
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
else{
for(edindex = 0; edindex < totedgesel; edindex +=1) {
eed = edarray[edindex];
walker_select(em, BMW_LOOP, eed, 1);
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
MEM_freeN(edarray);
// if (EM_texFaceCheck())
@@ -1041,7 +1041,7 @@
walker_select(em, BMW_LOOP, eed, select);
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
// if (EM_texFaceCheck())
/* sets as active, useful for other tools */
@@ -1394,7 +1394,7 @@
edgetag_context_set(em, vc.scene, e, act); /* switch the edge option */
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
/* even if this is selected it may not be in the selection list */
if (edgetag_context_check(vc.scene, em, e)==0)
@@ -1506,7 +1506,7 @@
}
}
- EDBM_selectmode_flush(vc.em);
+ EDBM_select_flush(vc.em);
// if (EM_texFaceCheck()) {
@@ -1575,7 +1575,7 @@
efa = BMIter_New(&iter, em->bm, BM_FACES_OF_MESH, NULL);
for ( ; efa; efa=BMIter_Step(&iter)) BM_Select(em->bm, efa, FALSE);*/
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
else if (em->selectmode & SCE_SELECT_EDGE) {
/* deselect vertices, and select again based on edge select */
@@ -1590,7 +1590,7 @@
}
/* selects faces based on edge status */
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
else if (em->selectmode & SCE_SELECT_FACE) {
/* deselect eges, and select again based on face select */
@@ -1809,7 +1809,7 @@
BM_Select(bm, e->v2, sel);
}
BMW_End(&walker);
- EDBM_select_flush(em, SCE_SELECT_VERTEX);
+ EDBM_select_mode_flush(em, SCE_SELECT_VERTEX);
}
WM_event_add_notifier(C, NC_GEOM|ND_SELECT, obedit);
@@ -1908,7 +1908,7 @@
}
BMW_End(&walker);
}
- EDBM_select_flush(em, SCE_SELECT_VERTEX);
+ EDBM_select_mode_flush(em, SCE_SELECT_VERTEX);
WM_event_add_notifier(C, NC_GEOM|ND_SELECT, obedit);
@@ -2052,7 +2052,7 @@
BMO_pop(bm);
/* Flush selection up */
- EDBM_select_flush(em, flushtype);
+ EDBM_select_mode_flush(em, flushtype);
}
static void deselect_nth_active(BMEditMesh *em, BMVert **v_p, BMEdge **e_p, BMFace **f_p)
@@ -2067,7 +2067,7 @@
*e_p= NULL;
*f_p= NULL;
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
ese= (BMEditSelection*)em->bm->selected.last;
if (ese) {
@@ -2399,7 +2399,7 @@
BM_Select(em->bm, eve, TRUE);
}
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
else if (em->selectmode & SCE_SELECT_EDGE) {
BM_ITER(eed, &iter, em->bm, BM_EDGES_OF_MESH, NULL) {
@@ -2407,7 +2407,7 @@
BM_Select(em->bm, eed, TRUE);
}
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
else {
BM_ITER(efa, &iter, em->bm, BM_FACES_OF_MESH, NULL) {
@@ -2415,7 +2415,7 @@
BM_Select(em->bm, efa, TRUE);
}
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
WM_event_add_notifier(C, NC_GEOM|ND_SELECT, obedit->data);
Modified: branches/bmesh/blender/source/blender/editors/mesh/bmesh_tools.c
===================================================================
--- branches/bmesh/blender/source/blender/editors/mesh/bmesh_tools.c 2012-02-10 02:42:15 UTC (rev 44007)
+++ branches/bmesh/blender/source/blender/editors/mesh/bmesh_tools.c 2012-02-10 03:03:42 UTC (rev 44008)
@@ -1445,7 +1445,7 @@
BM_Hide(em->bm, h, 1);
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
/*original hide flushing comment (OUTDATED):
hide happens on least dominant select mode, and flushes up, not down! (helps preventing errors in subsurf) */
@@ -1532,7 +1532,7 @@
}
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
static int reveal_mesh_exec(bContext *C, wmOperator *UNUSED(op))
@@ -2296,7 +2296,7 @@
return OPERATOR_CANCELLED;
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
/* dependencies graph and notification stuff */
/* DAG_object_flush_update(scene, ob, OB_RECALC_DATA);
@@ -2541,7 +2541,7 @@
BM_Select(bm, ripvert, TRUE);
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
BLI_assert(singlesel ? (bm->totvertsel > 0) : (bm->totedgesel > 0));
@@ -2788,7 +2788,7 @@
}
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
WM_event_add_notifier(C, NC_GEOM|ND_DATA, obedit->data);
return OPERATOR_FINISHED;
@@ -3303,7 +3303,7 @@
/*Flush the selection to clear edge/face selections to match
selected vertices*/
- EDBM_select_flush(em, SCE_SELECT_VERTEX);
+ EDBM_select_mode_flush(em, SCE_SELECT_VERTEX);
/*A "while(true)" loop should work here as each iteration should
select and remove at least one vertex and when all vertices
@@ -3342,7 +3342,7 @@
/*Flush the selection to get edge/face selections matching
the vertex selection*/
- EDBM_select_flush(em, SCE_SELECT_VERTEX);
+ EDBM_select_mode_flush(em, SCE_SELECT_VERTEX);
if (bm->totvert == bm->totvertsel) {
/* Every vertex selected, nothing to separate, work is done */
@@ -3849,7 +3849,7 @@
}
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
WM_event_add_notifier(C, NC_GEOM|ND_SELECT, obedit->data);
return OPERATOR_FINISHED;
@@ -3905,7 +3905,7 @@
}
}
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
WM_event_add_notifier(C, NC_GEOM|ND_SELECT, obedit->data);
return OPERATOR_FINISHED;
@@ -3933,7 +3933,7 @@
int extend= RNA_boolean_get(op->ptr, "extend");
EDBM_select_mirrored(obedit, em, extend);
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
WM_event_add_notifier(C, NC_GEOM|ND_SELECT, obedit->data);
return OPERATOR_FINISHED;
Modified: branches/bmesh/blender/source/blender/editors/mesh/bmeshutils.c
===================================================================
--- branches/bmesh/blender/source/blender/editors/mesh/bmeshutils.c 2012-02-10 02:42:15 UTC (rev 44007)
+++ branches/bmesh/blender/source/blender/editors/mesh/bmeshutils.c 2012-02-10 03:03:42 UTC (rev 44008)
@@ -418,21 +418,24 @@
return (tm->face_index && index<tm->bm->totface && index>=0) ? tm->face_index[index] : NULL;
}
-void EDBM_select_flush(BMEditMesh *em, int selectmode)
+void EDBM_select_mode_flush(BMEditMesh *em, int selectmode)
{
em->bm->selectmode = selectmode;
BM_SelectMode_Flush(em->bm);
em->bm->selectmode = em->selectmode;
}
-void EDBM_deselectmode_flush(BMEditMesh *em)
+void EDBM_deselect_flush(BMEditMesh *em)
{
+ /* function below doesnt use. just do this to keep the values in sync */
+ em->bm->selectmode = em->selectmode;
BM_DeSelect_Flush(em->bm);
}
-void EDBM_selectmode_flush(BMEditMesh *em)
+void EDBM_select_flush(BMEditMesh *em)
{
+ /* function below doesnt use. just do this to keep the values in sync */
em->bm->selectmode = em->selectmode;
BM_Select_Flush(em->bm);
}
@@ -449,7 +452,7 @@
BMO_HeaderFlag_Buffer(em->bm, &bmop, "geomout", BM_SELECT, BM_ALL);
BMO_Finish_Op(em->bm, &bmop);
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
void EDBM_select_less(BMEditMesh *em)
@@ -464,7 +467,7 @@
BMO_HeaderFlag_Buffer(em->bm, &bmop, "geomout", BM_SELECT, BM_ALL);
BMO_Finish_Op(em->bm, &bmop);
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
}
int EDBM_get_actSelection(BMEditMesh *em, BMEditSelection *ese)
Modified: branches/bmesh/blender/source/blender/editors/mesh/editbmesh_add.c
===================================================================
--- branches/bmesh/blender/source/blender/editors/mesh/editbmesh_add.c 2012-02-10 02:42:15 UTC (rev 44007)
+++ branches/bmesh/blender/source/blender/editors/mesh/editbmesh_add.c 2012-02-10 03:03:42 UTC (rev 44008)
@@ -152,7 +152,7 @@
/* Primitive has all verts selected, use vert select flush
to push this up to edges & faces. */
- EDBM_select_flush(em, SCE_SELECT_VERTEX);
+ EDBM_select_mode_flush(em, SCE_SELECT_VERTEX);
DAG_id_tag_update(obedit->data, OB_RECALC_DATA);
WM_event_add_notifier(C, NC_GEOM|ND_DATA, obedit->data);
Modified: branches/bmesh/blender/source/blender/editors/mesh/loopcut.c
===================================================================
--- branches/bmesh/blender/source/blender/editors/mesh/loopcut.c 2012-02-10 02:42:15 UTC (rev 44007)
+++ branches/bmesh/blender/source/blender/editors/mesh/loopcut.c 2012-02-10 03:03:42 UTC (rev 44008)
@@ -333,11 +333,11 @@
/* sets as active, useful for other tools */
if(em->selectmode & SCE_SELECT_VERTEX)
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
if(em->selectmode & SCE_SELECT_EDGE)
- EDBM_selectmode_flush(em);
+ EDBM_select_flush(em);
- EDBM_selectmode_flush(lcd->em);
+ EDBM_select_flush(lcd->em);
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list