[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [44398] trunk/blender/source/blender/bmesh /intern/bmesh_polygon.c: bmesh: minor speedup, use 1 iterator rather then 2 when normalizing verts.

[Campbell Barton]

Revision: 44398
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=44398
Author:   campbellbarton
Date:     2012-02-24 10:40:32 +0000 (Fri, 24 Feb 2012)
Log Message:
-----------
bmesh: minor speedup, use 1 iterator rather then 2 when normalizing verts.

Modified Paths:
--------------
    trunk/blender/source/blender/bmesh/intern/bmesh_polygon.c

Modified: trunk/blender/source/blender/bmesh/intern/bmesh_polygon.c
===================================================================
--- trunk/blender/source/blender/bmesh/intern/bmesh_polygon.c   2012-02-24 
10:35:52 UTC (rev 44397)
+++ trunk/blender/source/blender/bmesh/intern/bmesh_polygon.c   2012-02-24 
10:40:32 UTC (rev 44398)
@@ -459,30 +459,25 @@
 {
        /* TODO, we can normalize each edge only once, then compare with 
previous edge */
 
-       BMIter eiter, liter;
-       BMEdge *e;
+       BMIter liter;
        BMLoop *l;
        float vec1[3], vec2[3], fac;
        int len = 0;
 
        zero_v3(v->no);
 
-       BM_ITER(e, &eiter, bm, BM_EDGES_OF_VERT, v) {
-               BM_ITER(l, &liter, bm, BM_LOOPS_OF_EDGE, e) {
-                       if (l->v == v) {
-                               /* Same calculation used in 
BM_mesh_normals_update */
-                               sub_v3_v3v3(vec1, l->v->co, l->prev->v->co);
-                               sub_v3_v3v3(vec2, l->next->v->co, l->v->co);
-                               normalize_v3(vec1);
-                               normalize_v3(vec2);
+       BM_ITER(l, &liter, bm, BM_LOOPS_OF_VERT, v) {
+               /* Same calculation used in BM_mesh_normals_update */
+               sub_v3_v3v3(vec1, l->v->co, l->prev->v->co);
+               sub_v3_v3v3(vec2, l->next->v->co, l->v->co);
+               normalize_v3(vec1);
+               normalize_v3(vec2);
 
-                               fac = saacos(-dot_v3v3(vec1, vec2));
-                               
-                               madd_v3_v3fl(v->no, l->f->no, fac);
+               fac = saacos(-dot_v3v3(vec1, vec2));
 
-                               len++;
-                       }
-               }
+               madd_v3_v3fl(v->no, l->f->no, fac);
+
+               len++;
        }
 
        if (len) {

_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
http://lists.blender.org/mailman/listinfo/bf-blender-cvs