Skip to content

Commit

Permalink
BUG: Fix GW_Face::GetEdgeNumber to return -1 if edge is not found
Browse files Browse the repository at this point in the history
This commit fixes the following warning introduced in
3e42453 (ENH: Add SelectByPoints tool to DynamicModeler)

  /path/to/SurfaceToolbox/DynamicModeler/Logic/FastMarching/gw_core/GW_Face.inl: In member function ‘GW::GW_I32 GW::GW_Face::GetEdgeNumber(const GW::GW_Vertex&, const GW::GW_Vertex&)’:
  /path/to/SurfaceToolbox/DynamicModeler/Logic/FastMarching/gw_core/GW_Face.inl:359:9: warning: converting to non-pointer type ‘long int’ from NULL [-Wconversion-null]
    return NULL;
           ^
  • Loading branch information
jcfr committed Nov 2, 2021
1 parent 8dd9ce0 commit 4a50d15
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion DynamicModeler/Logic/FastMarching/gw_core/GW_Face.inl
Original file line number Diff line number Diff line change
Expand Up @@ -356,7 +356,7 @@ GW_I32 GW_Face::GetEdgeNumber( const GW_Vertex& Vert1, const GW_Vertex& Vert2 )
return (i+1)%3;
}
}
return NULL;
return -1;
}


Expand Down

0 comments on commit 4a50d15

Please sign in to comment.