correct edit some comments
authorCampbell Barton <ideasman42@gmail.com>
Fri, 9 Sep 2011 14:51:30 +0000 (14:51 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 9 Sep 2011 14:51:30 +0000 (14:51 +0000)
source/blender/blenkernel/intern/editderivedbmesh.c
source/blender/blenkernel/intern/mesh.c
source/blender/blenlib/intern/scanfill.c

index 6a624a5a0a080d9fd9d05d1fc6a76d00ab638400..5415372e6d2cc5df6ba70c3be15778fa5c4f87bd 100644 (file)
@@ -1289,9 +1289,6 @@ static void bmDM_copyFaceArray(DerivedMesh *dm, MFace *face_r)
                face_r->v1 = BM_GetIndex(l[0]->v);
                face_r->v2 = BM_GetIndex(l[1]->v);
                face_r->v3 = BM_GetIndex(l[2]->v);
-               /*BMESH_TODO: zero is not the best way to signal that
-                 the face is a triangle and not a quad, since 0 is
-                 also the index assigned to a real vertex*/
                face_r->v4 = 0;
 
                test_index_face(face_r, NULL, 0, 3);
index 4bd77e285de49b5f8d4538146d402c783450574c..2c2e72a68eb4defccdc50f1f08f659e4a2299d40 100644 (file)
@@ -1961,7 +1961,8 @@ void convert_mfaces_to_mpolys(Mesh *mesh)
                bmesh_corners_to_loops(mesh, i, mp->loopstart, numTex, numCol);
        }
 
-       /*BMESH_TODO now to deal with fgons*/
+       /* note, we dont convert FGons at all, these are not even real ngons,
+        * they have their own UV's, colors etc - its more an editing feature. */
 
        BLI_edgehash_free(eh, NULL);
 }
index dcfb8936c4ca8680389af468848ea52333140f70..b6c71bd9ad0cc365bd96645d7a95dd449b51ed10 100644 (file)
@@ -818,8 +818,8 @@ int BLI_edgefill(short mat_nr)
                float vec1[3], vec2[3];
 
                eve = fillvertbase.first;
-               
-               if (1 && eve->next && eve->next->next && eve->next->next->next) { //BMESH_TODO) {
+               /* no need to check 'eve->next->next->next' is valid, already counted */
+               if (1) { //BMESH_TODO) {
                        /*use shortest diagonal for quad*/
                        sub_v3_v3v3(vec1, eve->co, eve->next->next->co);
                        sub_v3_v3v3(vec2, eve->next->co, eve->next->next->next->co);