Page MenuHome

branches/bmesh/blender face selection - DONE - similar material/normal/perimeter/coplanar/area/texture
Closed, ArchivedPublicPATCH

Description

This patch will complete the similar face selection for bmesh. Now everything should work.

There was a minor glitch in one test and is now fixed.

Event Timeline

Committed at r22695. Couple of notes:

* We don't use tabs in variable assignments (e.g. a = b).
* &(struct->member) isn't necessary, &struct->member works. It's not a
big deal though; we all tend to overuse params at times.

Other then that, great job! Much better then I expected. I'm also curious as to why you had fs and fm variables instead of just f. The current "standard" is to use single-
letter variable names for topological elements, but multiple two-letter variables might
be a better idea (at the very least, they differentiate different uses of data, even if
the actual meaning of the second letter isn't obvious).

Joseph Eagar (joeedh) changed the task status from Unknown Status to Unknown Status.Aug 22 2009, 6:55 AM