[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [45142] trunk/blender/source/blender/bmesh /intern/bmesh_polygon.c: Comment out the projection to ngon normal plane when calculating ngon area .

[Tom M]

Probably put a comment with the commented out code so that when
someone stumbles across it they know why you have it commented out.

LetterRip

On Sun, Mar 25, 2012 at 12:13 PM, Antony Riakiotakis <kalast at gmail.com> wrote:
> Revision: 45142
>          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=45142
> Author:   psy-fi
> Date:     2012-03-25 19:13:50 +0000 (Sun, 25 Mar 2012)
> Log Message:
> -----------
> Comment out the projection to ngon normal plane when calculating ngon area. This is hardly worth the cost since we use an approximation anyway.
>
> 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-03-25 19:02:28 UTC (rev 45141)
> +++ trunk/blender/source/blender/bmesh/intern/bmesh_polygon.c   2012-03-25 19:13:50 UTC (rev 45142)
> @@ -181,14 +181,14 @@
>        }
>        mul_v3_fl(center, 1.0/nverts);
>
> +#if 0
>        /* project vertices to the normal plane */
>        for (j = 0; j < nverts; j++){
>                project_v3_plane(verts[j], normal, center);
>        }
> +#endif
>
> -       /* add triangle area of triangles formed by polygon edges and center.
> -        * For each triangle Sum(Base * Height / 2)
> -        * The expression for closed polygons is simplified */
> +       /* add triangle area of triangles formed by polygon edges and center. */
>        for (j = 0; j < nverts; j++) {
>                area += area_tri_v3(verts[j], verts[i], center);
>                i = j;
>
> _______________________________________________
> Bf-blender-cvs mailing list
> Bf-blender-cvs at blender.org
> http://lists.blender.org/mailman/listinfo/bf-blender-cvs