CGAL users discussion list

["J. Scheurich" <>]

> There must be something fishy here. See for example our test results here:
>
> https://cgal.geometryfactory.com/CGAL/testsuite/results-4.13-I-34.shtml
>
> You'll see that the line Surface_mesh_Examples where sm_circulators.cpp
> is tested can be compiled and ran successfully on all our test platforms
> (that includes some of yours).

Sorry for the table, it is misleading. sm_circulators.cpp failes only
under fedora 28 (not in the list of the tested systems) but i get
the exception of it under also under Debian and Arch Linux when i do

...
#include <CGAL/Surface_mesh.h>
typedef Kernel::Point_3 Point;
typedef CGAL::Surface_mesh<Point> Surface;
typedef Surface::Vertex_index vertex_index;
...

static void
build_mesh(Surface *meshOut, MyMesh *mesh, Matrix matrix)
{
    Array<vertex_index> u;
    for (int i = 0; i < mesh->getVertices()->getSFSize(); i++) {
        Vec3f v = mesh->getVertices()->getVec(i);
        v = matrix * v;
        u.append(meshOut->add_vertex(Kernel::Point_3(v.x, v.y, v.z)));
    }
    MFInt32 *meshCoords = mesh->getCoordIndex();
    int vert = 0;
    vertex_index v1;
    vertex_index v2;
    vertex_index v3;

    for (int i = 0; i < mesh->getSFSize(); i++) {
         int face = meshCoords->getValue(i);
         Vec3f v = mesh->getVertices()->getVec(face);
         if (face > -1) {
             switch (vert) {
               case 0:
                 v1 = u[face];
                 break;
               case 1:
                 v2 = u[face];
                break;
               case 2:
                 v3 = u[face];
                 break;
             }
             if (vert++ == 2) {
                 vert = 0;
                 meshOut->add_face(v1, v2, v3);
             }
         }
    }
}

Array is a std::vector like, mesh->getVertices() are the vertices of the
mesh,  meshCoords is a numVertex1, numVertex2, numVertex2, -1 List as in
X3D.
Is this a valid way to fill a CGAL::Surface_mesh ?
It works on Ubuntu and OpenSuse.

> Note also that in your compilation line with gcc you are missing 
> -frounding-math but that should not be responsible for the issue you have.

This is often missing in CGAL compiled by Linux distributors.
This fails valgrind to run on programs with CGAL linked, it throws a
other exception 8-(
Can't you add -frounding-math to the default cmake ?

> Could you indicate the gcc version with which sm_circulators.cpp is
> failing?

$ g++ --version
g++ (GCC) 8.1.1 20180502 (Red Hat 8.1.1-1)
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

so long
MUFTI