Skip to content

Commit

Permalink
Merge pull request #374 from sbriseid/core_dump_fix
Browse files Browse the repository at this point in the history
Fixed issue with core dump for geom object with missing implementatio…
  • Loading branch information
sbriseid authored Oct 10, 2024
2 parents 8ac1970 + 202a855 commit 4ba4229
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions compositemodel/src/SurfaceModelUtils.C
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,11 @@ SurfaceModelUtils::checkClosedFaces(shared_ptr<ParamSurface> surface, double tol
vector<shared_ptr<ParamCurve> > cvs_v1 = sf->constParamCurves(dom.vmin(), true);
vector<shared_ptr<ParamCurve> > cvs_v2 = sf->constParamCurves(dom.vmax(), true);

if ((cvs_u1.size() == 0) || (cvs_u2.size() == 0) || (cvs_v1.size() == 0) || (cvs_v2.size() == 0))
{
throw std::runtime_error("Seems like the object is missing the implementation of constParamCurves().");
}

// Compare also with middle curve
vector<shared_ptr<ParamCurve> > cvs_u3 =
sf->constParamCurves(0.5*(dom.umin()+dom.umax()), false);
Expand Down

0 comments on commit 4ba4229

Please sign in to comment.