From 735fb533d3b456169f43be776ffcec1d37e350b6 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Tue, 9 Feb 2016 02:04:46 +0000
Subject: [PATCH] cable-splice-clamp: fix roundoff_rad usage etc.

---
 cable-splice-clamp.scad | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/cable-splice-clamp.scad b/cable-splice-clamp.scad
index 851b5f6..0a78250 100644
--- a/cable-splice-clamp.scad
+++ b/cable-splice-clamp.scad
@@ -16,9 +16,13 @@ delta_ellipse = 0.45;
 
 // calculated
 
-outerdia = cable_maxdia / cos(30) + wallmin*2;
 delta_shift = 1 - delta_ellipse;
 
+roundoff_rad = cable_maxdia/2 * sqrt(delta_shift)
+	      + cable_mindia/2 * (1-sqrt(delta_shift));
+
+outerdia = 2*(roundoff_rad / cos(30) + wallmin);
+
 delta_shift_gap = (cable_maxdia-cable_mindia) * delta_shift;
 
 module CrossSection(plus=0) {
@@ -31,9 +35,7 @@ module CrossSection(plus=0) {
       scale([cable_maxdia,
 	     cable_maxdia * delta_shift + cable_mindia * delta_ellipse])
 	circle(r= sqrt(0.5) * delta_shift + 0.5 * delta_ellipse, $fn=4);
-      circle(r= cable_maxdia/2 * sqrt(delta_shift)
-	      + cable_mindia/2 * (1-sqrt(delta_shift)),
-	     $fn=20);
+      circle(r= roundoff_rad, $fn=20);
     }
 
     square(center=true, [40, delta_shift_gap]);
@@ -76,5 +78,5 @@ module CompleteClamp(){ ////toplevel
   }
 }
 
-CrossSectionDemo();
-//CompleteClamp();
+//CrossSectionDemo();
+CompleteClamp();
-- 
2.30.2