[Bf-blender-cvs] [59c59cc] master: Cleanup: remove redundant check

Campbell Barton noreply at git.blender.org
Tue Jul 12 09:59:21 CEST 2016


Commit: 59c59cc402d1000bf1aabef1c01a708ddee293ca
Author: Campbell Barton
Date:   Tue Jul 12 17:59:25 2016 +1000
Branches: master
https://developer.blender.org/rB59c59cc402d1000bf1aabef1c01a708ddee293ca

Cleanup: remove redundant check

===================================================================

M	intern/opensubdiv/opensubdiv_converter.cc

===================================================================

diff --git a/intern/opensubdiv/opensubdiv_converter.cc b/intern/opensubdiv/opensubdiv_converter.cc
index 3fadde6..0ad72c8 100644
--- a/intern/opensubdiv/opensubdiv_converter.cc
+++ b/intern/opensubdiv/opensubdiv_converter.cc
@@ -332,7 +332,6 @@ inline bool TopologyRefinerFactory<OpenSubdiv_Converter>::assignComponentTopolog
 			else {
 				/* Special handle of non-manifold vertex. */
 				for (int i = 0; i < num_vert_edges; ++i) {
-					bool start_found = false;
 					edge_start = vert_edges[i];
 					IndexArray edge_faces = getBaseEdgeFaces(refiner, edge_start);
 					if (edge_faces.size() == 1) {
@@ -343,14 +342,10 @@ inline bool TopologyRefinerFactory<OpenSubdiv_Converter>::assignComponentTopolog
 							    face_edges = getBaseFaceEdges(refiner, face_start);
 							face_vert_start = findInArray(face_verts, vert);
 							if (edge_start == face_edges[face_vert_start]) {
-								start_found = true;
 								break;
 							}
 						}
 					}
-					if (start_found) {
-						break;
-					}
 					/* Reset indices for sanity check below. */
 					face_start = edge_start = face_vert_start =  -1;
 				}




More information about the Bf-blender-cvs mailing list