[Bf-codereview] modeler normals don't match render normals (issue4280049)

brechtvanlommel at gmail.com brechtvanlommel at gmail.com
Tue Mar 15 14:59:52 CET 2011


To be honest, I don't like this SCalcNormalInterface at all, way too
much code and abstraction.

Why not simply keep the loops per mesh type, to clear vertex normals and
normalize them, and then have one utility function that is used from all
the face loop? Something like:

void accumulate_vertex_normals(int nverts, const float verts[][3], float
vnormals[][3], float fnormal[3]);

http://codereview.appspot.com/4280049/


More information about the Bf-codereview mailing list