star-cad

commit 9ca9cad1b7848ba0dccdc4157a3fe31a430a3ed9
parent 98b7aa2374175859f7ec24cae1ebeb975bc59899
Author: Christophe Coustet <christophe.coustet@meso-star.com>
Date:   Fri,  6 Jun 2025 11:54:35 +0200

Fix test_periodic

Check API calls return

Diffstat:
M
src/test_periodic.c
 | 
42
+++++++++++++++++++++---------------------

1 file changed, 21 insertions(+), 21 deletions(-)
diff --git a/src/test_periodic.c b/src/test_periodic.c
@@ -41,60 +41,60 @@ main(int argc, char* argv[])
 
   OK(mem_init_proxy_allocator(&allocator, &mem_default_allocator));
 
-  scad_initialize(NULL, &allocator, 2);
+  OK(scad_initialize(NULL, &allocator, 2));
   r2 = r1 * XR;
-  scad_add_cylinder(NULL, p1, d2, r1, 2*PI, &cyl1);
-  scad_add_cylinder(NULL, p1, d2, r2, 2*PI, &cyl2);
-  scad_geometries_cut("cylinder", &cyl2, 1, &cyl1, 1, &cyl);
-  scad_geometry_ref_put(cyl1);
-  scad_geometry_ref_put(cyl2);
-  scad_geometries_boundary(NULL, &cyl, 1, &bound);
-  scad_geometry_explode(bound, NULL, &faces, &facesn);
-  scad_geometry_ref_put(bound);
+  OK(scad_add_cylinder(NULL, p1, d2, r1, 2*PI, &cyl1));
+  OK(scad_add_cylinder(NULL, p1, d2, r2, 2*PI, &cyl2));
+  OK(scad_geometries_cut("cylinder", &cyl2, 1, &cyl1, 1, &cyl));
+  OK(scad_geometry_ref_put(cyl1));
+  OK(scad_geometry_ref_put(cyl2));
+  OK(scad_geometries_boundary(NULL, &cyl, 1, &bound));
+  OK(scad_geometry_explode(bound, NULL, &faces, &facesn));
+  OK(scad_geometry_ref_put(bound));
   ASSERT(facesn == 4);
   d3_add(p2, p1, d2);
   len = d3_len(d2);
   for(i = 0; i < facesn; i++) {
     struct scad_geometry* f = faces[i];
     double center[3], m;
-    scad_geometry_get_centerofmass(f, center);
+    OK(scad_geometry_get_centerofmass(f, center));
     if(fabs(center[0] - p1[0]) < FLT_EPSILON) {
       ASSERT(lat[0] == NULL);
       lat[0] = f;
-      scad_geometry_rename(f, "left_side");
+      OK(scad_geometry_rename(f, "left_side"));
       continue;
     }
     if(fabs(center[0] - p2[0]) < FLT_EPSILON) {
       ASSERT(lat[1] == NULL);
       lat[1] = f;
-      scad_geometry_rename(f, "right_side");
+      OK(scad_geometry_rename(f, "right_side"));
       continue;
     }
     scad_geometry_get_mass(f, &m);
     if(fabs(m - len*2*PI*r1) < FLT_EPSILON) {
       ASSERT(internal == NULL);
       internal = f;
-      scad_geometry_rename(f, "internal");
+      OK(scad_geometry_rename(f, "internal"));
       continue;
     }
     if(fabs(m - len*2*PI*r2) < FLT_EPSILON) {
       ASSERT(external == NULL);
       external = f;
-      scad_geometry_rename(f, "external");
+      OK(scad_geometry_rename(f, "external"));
       continue;
     }
   }
   ASSERT(lat[0] && lat[1] && internal && external);
-  scad_geometries_set_periodic(&internal, 1, &external, 1, cyl_affine);
-  scad_geometries_set_mesh_algorithm(lat, 1, SCAD_INITIAL_MESH_ONLY);
+  OK(scad_geometries_set_periodic(&internal, 1, &external, 1, cyl_affine));
+  OK(scad_geometries_set_mesh_algorithm(lat, 1, SCAD_INITIAL_MESH_ONLY));
   for(i = 0; i < facesn; i++) {
-    scad_geometry_ref_put(faces[i]);
+    OK(scad_geometry_ref_put(faces[i]));
   }
   MEM_RM(&allocator, faces);
-  scad_scene_mesh();
-  scad_stl_export(cyl, "periodic", SCAD_FORCE_NORMALS_OUTWARD, 1);
-  scad_geometry_ref_put(cyl);
-  scad_finalize();
+  OK(scad_scene_mesh());
+  OK(scad_stl_export(cyl, "periodic", SCAD_FORCE_NORMALS_OUTWARD, 1));
+  OK(scad_geometry_ref_put(cyl));
+  OK(scad_finalize());
 #undef XR
 
   check_memory_allocator(&allocator);