libtess2 icon indicating copy to clipboard operation
libtess2 copied to clipboard

Just an user feedback.

Open rafirafi opened this issue 10 years ago • 1 comments

I've tried a lot of libraries to replace the glu tessselator and this is the first one to give me consistant (and correct) results.

As a thank you, I want to offer you my first impressions on the api as an user :

Globally the library is easy to use and more expressive than glu (no need to define edgeCallback to get only triangles, no need to use gluTessNormal to inform that only x and y are used...).

For tessTesselate I've found 'polySize' and 'vertexSize' not to be enought expressive names : it was not immediatly clear that this was only for defining the output format.

For tessAddContour, it's specified that only tessReal is accepted but :

  • reading the function signature it's not that clear that this means 'only float'
  • it will be nice to have a parameter to specify the type, like it's done in the opengl api, so libtess2 could do the conversion internally if necessary

After tessTesselate, the result is accessible throught getter like tessGetVertices. I've not found (in tesselator.h) how much time the returned arrays are valid. I assume this is till the next call to tessTesselate, but maybe it's till the next call to tessAddContour, or else ?

If the library segfault, it's not obvious what went wrong even with NDEBUG not defined, not a big issue as the interface is still simple it's relatively easy to check everything.

Again, theses are just my personnal impressions so feel free to ignore them if you find them not relevant.

rafirafi avatar Feb 02 '15 22:02 rafirafi

Thanks for the feedback, good points!

memononen avatar Feb 03 '15 07:02 memononen