[Bf-codereview] Skin modifier (issue 6220055)

[ideasman42 at gmail.com]

1) --- thats bad!, fixed in svn.
2) --- no, but may as well add one.

On 2012/05/20 22:47:03, nicholasbishop wrote:
> Thanks for reviewing :)

> I'm working my way through the changes now, couple quick questions:

> 1) BM_edge_face_pair() seems to return true for edges with just one
face. The
> function doc says it "returns TRUE when only 2 faces are found".
Guessing it
> needs an extra check?

> 2) Is there an equivalent to BM_edge_other_vert for MEdge? Some of the
marked
> places aren't BMesh.



http://codereview.appspot.com/6220055/