ofxTriangleMesh icon indicating copy to clipboard operation
ofxTriangleMesh copied to clipboard

Error with current OF master due to the new glm::tvec3 format in ofPolygon

Open frauzufall opened this issue 8 years ago • 0 comments

Hey, I was switching over from ofxTriangle and had a small issue with the current master due to the new vector formats:

addons/ofxTriangleMesh/src/ofxTriangleMesh.cpp:121: error: no matching function for call to 'ofxTriangleMesh::isPointInsidePolygon(glm::tvec3<float, (glm::precision)0u>*, size_t, ofPoint)'
         if( isPointInsidePolygon(&contour[0], contour.size(), getTriangleCenter(tr) ) ) {
                                                                                     ^

I was able to resolve it by changing the isPointInsidePolygon function to look like this:

bool isPointInsidePolygon(ofPolyline polygon, int N, ofPoint p);

I can file a PR, but maybe there is another way of resolving this without changing the function parameters..?

Thanks for the addon! :)

frauzufall avatar Aug 11 '16 23:08 frauzufall