correct edit some comments

3 files changed, 4 insertions, 6 deletions

diff --git a/source/blender/blenkernel/intern/editderivedbmesh.c b/source/blender/blenkernel/intern/editderivedbmesh.c
index 6a624a5a0a0..5415372e6d2 100644
--- a/source/blender/blenkernel/intern/editderivedbmesh.c
+++ b/source/blender/blenkernel/intern/editderivedbmesh.c
@@ -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);
diff --git a/source/blender/blenkernel/intern/mesh.c b/source/blender/blenkernel/intern/mesh.c
index 4bd77e285de..2c2e72a68eb 100644
--- a/source/blender/blenkernel/intern/mesh.c
+++ b/source/blender/blenkernel/intern/mesh.c
@@ -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);
 }
diff --git a/source/blender/blenlib/intern/scanfill.c b/source/blender/blenlib/intern/scanfill.c
index dcfb8936c4c..b6c71bd9ad0 100644
--- a/source/blender/blenlib/intern/scanfill.c
+++ b/source/blender/blenlib/intern/scanfill.c
@@ -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);