From a83f55ec09411c2805b717b9c92559546bb28109 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Thu, 25 Oct 2018 13:57:21 +0100
Subject: [PATCH 1/1] lock-inframe-bracket: wip new divide

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
---
 lock-inframe-bracket.scad | 25 ++++++++++++++++---------
 1 file changed, 16 insertions(+), 9 deletions(-)

diff --git a/lock-inframe-bracket.scad b/lock-inframe-bracket.scad
index 12ff949..618be89 100644
--- a/lock-inframe-bracket.scad
+++ b/lock-inframe-bracket.scad
@@ -157,6 +157,18 @@ module DivideInPlace(){
     children();
 }
 
+module DivideSurround(){
+  w = tube_th/2;
+  d = divide_gap;
+
+  offset(r= w*2) {
+    hull() {
+      DivideCut();
+      translate([-(4*w + 2*d), 8*w]) circle(r=w);
+    }
+  }
+}
+
 module MainPlan(){ ////toplevel
   difference(){
     union(){
@@ -188,12 +200,7 @@ module MainPlan(){ ////toplevel
 	JoinCircs(join_cr);
       }
 
-      hull(){
-	minkowski(){
-	  DividePlanInPlace(0.1);
-	  circle(divide_around);
-	}
-      }
+      DivideInPlace() DivideSurround();
     }
     translate([0, lock_0y]){
       oval([lock_w/2, lock_d/2]);
@@ -201,7 +208,7 @@ module MainPlan(){ ////toplevel
 
     circle(r = tube_dia/2);
 
-    DividePlanInPlace();
+    DivideInPlace() DivideCut();
   }
 }
 
@@ -323,8 +330,8 @@ module DividePlanDemo(){ ////toplevel
 
 module DivideDemo(){ ////toplevel
   color("black") translate([0,0,-2]) MainPlan();
-  DivideInPlace() DivideHook();
-  color("blue") translate([0,0,2]) DivideInPlace() DivideCut();
+  color("grey") DivideInPlace() DivideHook();
+  color("blue") translate([0,0,-4]) DivideInPlace() DivideCut();
 }
 
 //MainPlan();
-- 
2.30.2