toolbox-inserts: break out AtFixings, fix

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/toolbox-inserts.scad b/toolbox-inserts.scad
index 440b2f1ce80673498080f0e5d5cdffa5114f6e21..4f285a99915488328e02a234c3726bc62cc9eaac 100644 (file)
--- a/toolbox-inserts.scad
+++ b/toolbox-inserts.scad
@@ -192,35 +192,34 @@ module AtFixings() {
   n_x = ceil(box_x / fixing_pitch / 2) + 1;
   n_y = ceil(box_y / fixing_pitch / 2) + 1;
 
-  for (xi=[ for (xi = [-n_x : n_x])
-           if (xi != 0)
-           xi ])
+  for (xi=[-n_x : n_x])
+    if (xi != 0)
     for (yi=[-n_y : n_y + 1])
-      translate(fixing_pitch * [xi, yi - 0.5, 0])
-       children(0);
+      translate(fixing_pitch * [xi, yi - 0.5, 0]) {
+       children();
+      }
 }
 
 module FullPlate() { ////toplevel
   render() difference(){
-    render() intersection(){
+    intersection(){
       BoxCavityModel();
       linextr(-20, 0) square(1000, center=true);
     }
 
-    intersection(){
-      union(){
-       AtFixings() {
-         linextr(-20, -below_plane_z + fixing_head_th)
-           circle(r = fixing_head_d/2);
-         linextr(-20, 20)
-           circle(r = fixing_nom_d/2);
-       }
+    union(){
+      AtFixings() {
+       linextr(-20, -below_plane_z + fixing_head_th)
+         circle(r = fixing_head_d/2);
+       linextr(-20, 20)
+         circle(r = fixing_nom_d/2);
       }
+    }
 
 //      linextr(-100, 100)
 //     offset(r = -plate_unholed_edge_w)
 //     BoxCavityModelMainPlan();
-    }
+//    }
   }
 }