[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [43557] branches/bmesh/blender/source/ blender/blenkernel/intern/DerivedMesh.c: after all modifier are applied calculate normals, then tessfaces, this works so tesselating copies normals into the the newly creates tesselated faces .

[Campbell Barton]

Revision: 43557
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=43557
Author:   campbellbarton
Date:     2012-01-20 13:14:14 +0000 (Fri, 20 Jan 2012)
Log Message:
-----------
after all modifier are applied calculate normals, then tessfaces, this works so 
tesselating copies normals into the the newly creates tesselated faces.

Modified Paths:
--------------
    branches/bmesh/blender/source/blender/blenkernel/intern/DerivedMesh.c

Modified: branches/bmesh/blender/source/blender/blenkernel/intern/DerivedMesh.c
===================================================================
--- branches/bmesh/blender/source/blender/blenkernel/intern/DerivedMesh.c       
2012-01-20 12:34:00 UTC (rev 43556)
+++ branches/bmesh/blender/source/blender/blenkernel/intern/DerivedMesh.c       
2012-01-20 13:14:14 UTC (rev 43557)
@@ -1539,18 +1539,16 @@
        }
 #endif /* WITH_GAMEENGINE */
 
-
        /* 
--------------------------------------------------------------------- */
+       /* First calculate the polygon and vertex normals, re-tesselation
+        * copies these into the tessface's normal layer */
+       finaldm->calcNormals(finaldm);
        /* Re-tesselation is necessary to push render data (uvs, textures, 
colors)
         * from loops and polys onto the tessfaces. This may be currently be
         * redundantin cases where the render mode doesn't use these inputs, but
         * ideally eventually tesselation would happen on-demand, and this is 
one
         * of the primary places it would be needed. */
        finaldm->recalcTesselation(finaldm);
-       /* if we have no modifiers applied we'lll still want the tessface 
normals
-        * to be calculated from the polygon noramals,
-        * 'CDDM_calc_normals_mapping' checks for this case - campbell */
-       finaldm->calcNormals(finaldm);
        /* Need to watch this, it can cause issues, see bug [#29338]            
 */
        /* take care with this block, we really need testing frameworks         
 */
        /* 
--------------------------------------------------------------------- */

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