[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [42015] branches/bmesh/blender/source/ blender: Various typo cleanups, remove unnecessary code etc, found during recent work and separated to a cleanup change
Andrew Wiggin
ender79bl at gmail.com
Sun Nov 20 17:21:14 CET 2011
Revision: 42015
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=42015
Author: ender79
Date: 2011-11-20 16:21:13 +0000 (Sun, 20 Nov 2011)
Log Message:
-----------
Various typo cleanups, remove unnecessary code etc, found during recent work and separated to a cleanup change
Modified Paths:
--------------
branches/bmesh/blender/source/blender/blenkernel/intern/editderivedbmesh.c
branches/bmesh/blender/source/blender/blenkernel/intern/mesh.c
branches/bmesh/blender/source/blender/blenkernel/intern/object.c
branches/bmesh/blender/source/blender/bmesh/intern/bmesh_mods.c
branches/bmesh/blender/source/blender/bmesh/intern/bmesh_newcore.c
branches/bmesh/blender/source/blender/bmesh/intern/in-progress/BME_conversions.c
branches/bmesh/blender/source/blender/bmesh/operators/mesh_conv.c
branches/bmesh/blender/source/blender/bmesh/tools/BME_bevel.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/mesh_data.c
branches/bmesh/blender/source/blender/makesdna/DNA_meshdata_types.h
branches/bmesh/blender/source/blender/makesdna/intern/dna_genfile.c
branches/bmesh/blender/source/blender/makesrna/intern/rna_mesh.c
Modified: branches/bmesh/blender/source/blender/blenkernel/intern/editderivedbmesh.c
===================================================================
--- branches/bmesh/blender/source/blender/blenkernel/intern/editderivedbmesh.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/blenkernel/intern/editderivedbmesh.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -1613,12 +1613,6 @@
CustomData_from_bmeshpoly(&bmdm->dm.faceData, &em->bm->pdata, &em->bm->ldata, 0);
- bmdm->dm.numVertData = bm->totvert;
- bmdm->dm.numEdgeData = bm->totedge;
- bmdm->dm.numFaceData = em->tottri;
- bmdm->dm.numLoopData = bm->totloop;
- bmdm->dm.numPolyData = bm->totface;
-
bmdm->dm.getVertCos = bmDM_getVertCos;
bmdm->dm.getMinMax = bmDM_getMinMax;
Modified: branches/bmesh/blender/source/blender/blenkernel/intern/mesh.c
===================================================================
--- branches/bmesh/blender/source/blender/blenkernel/intern/mesh.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/blenkernel/intern/mesh.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -736,9 +736,9 @@
tex_space_mesh(me);
}
- if (loc_r) VECCOPY(loc_r, me->loc);
- if (rot_r) VECCOPY(rot_r, me->rot);
- if (size_r) VECCOPY(size_r, me->size);
+ if (loc_r) copy_v3_v3(loc_r, me->loc);
+ if (rot_r) copy_v3_v3(rot_r, me->rot);
+ if (size_r) copy_v3_v3(size_r, me->size);
}
float *get_mesh_orco_verts(Object *ob)
@@ -1118,7 +1118,7 @@
nors= dl->nors;
verts= dl->verts;
while(a--) {
- VECCOPY(mvert->co, verts);
+ copy_v3_v3(mvert->co, verts);
normal_float_to_short_v3(mvert->no, nors);
mvert++;
nors+= 3;
@@ -1231,7 +1231,7 @@
a= dl->parts*dl->nr;
data= dl->verts;
while(a--) {
- VECCOPY(mvert->co, data);
+ copy_v3_v3(mvert->co, data);
data+=3;
vertcount++;
mvert++;
@@ -1254,7 +1254,7 @@
a= dl->parts*dl->nr;
data= dl->verts;
while(a--) {
- VECCOPY(mvert->co, data);
+ copy_v3_v3(mvert->co, data);
data+=3;
vertcount++;
mvert++;
@@ -1277,7 +1277,7 @@
a= dl->nr;
data= dl->verts;
while(a--) {
- VECCOPY(mvert->co, data);
+ copy_v3_v3(mvert->co, data);
data+=3;
vertcount++;
mvert++;
@@ -1305,7 +1305,7 @@
a= dl->parts*dl->nr;
data= dl->verts;
while(a--) {
- VECCOPY(mvert->co, data);
+ copy_v3_v3(mvert->co, data);
data+=3;
vertcount++;
mvert++;
@@ -1767,7 +1767,7 @@
mf = mfaces;
for (i=0; i<numFaces; i++, mf++, origIndexFace++) {
if (*origIndexFace < numPolys) {
- VECCOPY(fnors[i], pnors[*origIndexFace]);
+ copy_v3_v3(fnors[i], pnors[*origIndexFace]);
} else {
/*eek, we're not corrusponding to polys*/
printf("error in mesh_calc_normals; tesselation face indices are incorrect. normals may look bad.\n");
@@ -2448,7 +2448,7 @@
{
MVert *v1, *v2, *v3;
- double u[3], v[3], w[3];
+ double u[3], v[3], w[3];
double n[3] = {0.0, 0.0, 0.0}, l;
int i;
Modified: branches/bmesh/blender/source/blender/blenkernel/intern/object.c
===================================================================
--- branches/bmesh/blender/source/blender/blenkernel/intern/object.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/blenkernel/intern/object.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -2556,15 +2556,19 @@
BLI_assert((scene->customdata_mask & CD_MASK_BAREMESH) == CD_MASK_BAREMESH);
if(em) {
makeDerivedMesh(scene, ob, em, scene->customdata_mask, 0); /* was CD_MASK_BAREMESH */
- } else
+ }
+ else {
makeDerivedMesh(scene, ob, NULL, scene->customdata_mask, 0);
+ }
#else /* ensure CD_MASK_BAREMESH for now */
BMEditMesh *em = (ob == scene->obedit)? ((Mesh*)ob->data)->edit_btmesh : NULL;
if(em) {
makeDerivedMesh(scene, ob, em, scene->customdata_mask | CD_MASK_BAREMESH, 0); /* was CD_MASK_BAREMESH */
- } else
+ }
+ else {
makeDerivedMesh(scene, ob, NULL, scene->customdata_mask | CD_MASK_BAREMESH, 0);
+ }
#endif
}
Modified: branches/bmesh/blender/source/blender/bmesh/intern/bmesh_mods.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/intern/bmesh_mods.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/bmesh/intern/bmesh_mods.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -127,8 +127,9 @@
f = e->l->f;
f2 = e->l->radial_next->f;
- if (f != f2 && !BM_Join_TwoFaces(bm, f, f2, NULL))
+ if (f != f2 && !BM_Join_TwoFaces(bm, f, f2, e)) {
return 0;
+ }
return 1;
}
@@ -171,7 +172,7 @@
if (f != f2) {
/*join two remaining faces*/
- if (!BM_Join_TwoFaces(bm, f, f2, NULL)) {
+ if (!BM_Join_TwoFaces(bm, f, f2, e)) {
return 0;
}
}
Modified: branches/bmesh/blender/source/blender/bmesh/intern/bmesh_newcore.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/intern/bmesh_newcore.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/bmesh/intern/bmesh_newcore.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -1599,7 +1599,6 @@
the edges & faces and assign an index to each connected set */
while ((e = BLI_array_pop(stack))) {
BLI_ghash_insert(visithash, e, SET_INT_IN_POINTER(maxindex));
- BM_SetIndex(e, maxindex); /* set_dirty! */ /* BMESH_TODO, check the indexs are is invalid after this function runs */
BM_ITER(l, &liter, bm, BM_LOOPS_OF_EDGE, e) {
nl = (l->v == v) ? l->prev : l->next;
@@ -1611,7 +1610,6 @@
maxindex++;
}
- bm->elem_index_dirty |= BM_EDGE;
/* Make enough verts to split v for each group */
verts = MEM_callocN(sizeof(BMVert *) * maxindex, "bmesh_cutvert");
Modified: branches/bmesh/blender/source/blender/bmesh/intern/in-progress/BME_conversions.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/intern/in-progress/BME_conversions.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/bmesh/intern/in-progress/BME_conversions.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -334,7 +334,7 @@
static void bmeshvert_to_mvert(BMMesh *bm, BMVert *v, MVert *mv, int index, CustomData *data)
{
- copy_v3_v3(mv->co,v->co);
+ copy_v3_v3(mv->co, v->co);
if(bmesh_test_flag(v, BMESH_SELECT)) mv->flag |= 1;
if(bmesh_test_flag(v, BMESH_HIDDEN)) mv->flag |= ME_HIDE;
mv->bweight = (char)(255.0*v1->bweight);
Modified: branches/bmesh/blender/source/blender/bmesh/operators/mesh_conv.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/operators/mesh_conv.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/bmesh/operators/mesh_conv.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -331,7 +331,7 @@
MLoopUV *mloopuv;
int i, j;
- for(i=0; i < numTex; i++){
+ for(i=0; i < numTex; i++) {
texface = CustomData_get_n(&me->fdata, CD_MTFACE, findex, i);
texpoly = CustomData_bmesh_get_n(&bm->pdata, f->head.data, CD_MTEXPOLY, i);
Modified: branches/bmesh/blender/source/blender/bmesh/tools/BME_bevel.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/tools/BME_bevel.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/bmesh/tools/BME_bevel.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -85,13 +85,23 @@
vtd->bm = bm;
vtd->v = v;
- if (co != NULL) copy_v3_v3(vtd->co,co);
- if (org == NULL && is_new) { copy_v3_v3(vtd->org,v->co); } /* default */
- else if (org != NULL) copy_v3_v3(vtd->org,org);
+
+ if (co != NULL) {
+ copy_v3_v3(vtd->co, co);
+ }
+
+ if (org == NULL && is_new) {
+ copy_v3_v3(vtd->org, v->co); /* default */
+ }
+ else if (org != NULL) {
+ copy_v3_v3(vtd->org,org);
+ }
+
if (vec != NULL) {
copy_v3_v3(vtd->vec,vec);
normalize_v3(vtd->vec);
}
+
vtd->loc = loc;
vtd->factor = factor;
@@ -352,19 +362,19 @@
BME_TransData *vtd1, *vtd2;
float max, fac1, fac2, vec1[3], vec2[3], vec3[3];
- BME_bevel_get_vec(vec1,v1,v2,td);
- vtd1 = BME_get_transdata(td,v1);
- vtd2 = BME_get_transdata(td,v2);
+ BME_bevel_get_vec(vec1, v1, v2, td);
+ vtd1 = BME_get_transdata(td, v1);
+ vtd2 = BME_get_transdata(td, v2);
if (vtd1->loc == NULL) {
fac1 = 0;
}
else {
- copy_v3_v3(vec2,vtd1->vec);
- mul_v3_fl(vec2,vtd1->factor);
+ copy_v3_v3(vec2, vtd1->vec);
+ mul_v3_fl(vec2, vtd1->factor);
if (dot_v3v3(vec1, vec1)) {
- project_v3_v3v3(vec2,vec2,vec1);
- fac1 = len_v3(vec2)/value;
+ project_v3_v3v3(vec2, vec2,vec1);
+ fac1 = len_v3(vec2) / value;
}
else {
fac1 = 0;
@@ -375,11 +385,11 @@
fac2 = 0;
}
else {
- copy_v3_v3(vec3,vtd2->vec);
- mul_v3_fl(vec3,vtd2->factor);
+ copy_v3_v3(vec3, vtd2->vec);
+ mul_v3_fl(vec3, vtd2->factor);
if (dot_v3v3(vec1, vec1)) {
- project_v3_v3v3(vec2,vec3,vec1);
- fac2 = len_v3(vec2)/value;
+ project_v3_v3v3(vec2, vec3, vec1);
+ fac2 = len_v3(vec2) / value;
}
else {
fac2 = 0;
@@ -387,7 +397,7 @@
}
if (fac1 || fac2) {
- max = len_v3(vec1)/(fac1 + fac2);
+ max = len_v3(vec1) / (fac1 + fac2);
if (vtd1->max && (*vtd1->max < 0 || max < *vtd1->max)) {
*vtd1->max = max;
}
@@ -876,7 +886,7 @@
}
BMesh *BME_bevel(BMMesh *bm, float value, int res, int options, int defgrp_index, float angle, BME_TransData_Head **rtd)
- {
+{
BMVert *v;
BMEdge *e;
BMIter *verts;
Modified: branches/bmesh/blender/source/blender/editors/mesh/bmesh_tools.c
===================================================================
--- branches/bmesh/blender/source/blender/editors/mesh/bmesh_tools.c 2011-11-20 16:19:56 UTC (rev 42014)
+++ branches/bmesh/blender/source/blender/editors/mesh/bmesh_tools.c 2011-11-20 16:21:13 UTC (rev 42015)
@@ -1871,9 +1871,7 @@
/* dependencies graph and notification stuff */
DAG_id_tag_update(ob->data, OB_RECALC_DATA);
WM_event_add_notifier(C, NC_GEOM|ND_DATA, ob->data);
-/* DAG_id_tag_update(ob->data, OB_RECALC_DATA);
- WM_event_add_notifier(C, NC_GEOM|ND_DATA, ob->data);
-*/
+
/* we succeeded */
return OPERATOR_FINISHED;
}
@@ -1897,9 +1895,7 @@
/* dependencies graph and notification stuff */
DAG_id_tag_update(ob->data, OB_RECALC_DATA);
WM_event_add_notifier(C, NC_GEOM|ND_DATA, ob->data);
-/* DAG_id_tag_update(ob->data, OB_RECALC_DATA);
- WM_event_add_notifier(C, NC_GEOM|ND_DATA, ob->data);
-*/
+
/* we succeeded */
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list