alecjacobson / gptoolbox

Matlab toolbox for Geometry Processing.
MIT License
628 stars 166 forks source link

dual_laplacian crashes matlab if I (incorrectly) give it triangle mesh #110

Open otmanon opened 3 years ago

otmanon commented 3 years ago

dual_laplacian expects a tet-mesh defined by V and T as input. If you pass in a triangle mesh instead all of matlab crashes (at least on windows 10). I know it doesn't expect a triangle mesh, and works fine if you give it the expected input... but crashing all of matlab is a harsh punishment for forgetting one measly column :( .

alecjacobson commented 3 years ago

thanks. it should use mexErrMsgTxt([condition],"message"); if you're keen, you could submit a PR and I'll merge right away.