Revert "BUG: Fix GW_Face::GetEdgeNumber to return -1 if edge is not found"

jcfr · jcfr · commit 3246cea86b16 · 2021-11-02T13:13:32.000-04:00
This reverts commit f9494db. Following discussion with @lassoan during Slicer hangout of Nov 2nd 2021, we decided not fix this as it could potentially introduce crashes. Further investigations are required. See #49
diff --git a/DynamicModeler/Logic/FastMarching/gw_core/GW_Face.inl b/DynamicModeler/Logic/FastMarching/gw_core/GW_Face.inl
@@ -356,7 +356,7 @@ GW_I32 GW_Face::GetEdgeNumber( const GW_Vertex& Vert1, const GW_Vertex& Vert2 )
 				return (i+1)%3;
 		}
 	}
-  return -1;
+	return NULL;
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -356,7 +356,7 @@ GW_I32 GW_Face::GetEdgeNumber( const GW_Vertex& Vert1, const GW_Vertex& Vert2 )`
`356`	`356`	`return (i+1)%3;`
`357`	`357`	`}`
`358`	`358`	`}`
`359`		`- return -1;`
	`359`	`+ return NULL;`
`360`	`360`	`}`
`361`	`361`
`362`	`362`