From 7205f0165eeff6186a0c4dfc6ab57f2d418bb68a Mon Sep 17 00:00:00 2001
From: Thomas Krijnen <t.krijnen@gmail.com>
Date: Fri, 21 Sep 2018 11:21:51 +0200
Subject: [PATCH] Work around private copy constructor on older versions of
 occt

---
 src/ifcgeom/IfcGeomFunctions.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/ifcgeom/IfcGeomFunctions.cpp b/src/ifcgeom/IfcGeomFunctions.cpp
index 1fd1b78d1a..8c8060eda4 100644
--- a/src/ifcgeom/IfcGeomFunctions.cpp
+++ b/src/ifcgeom/IfcGeomFunctions.cpp
@@ -157,7 +157,7 @@
 #endif
 
 namespace {
-	TopTools_ListOfShape copy_operand(const TopTools_ListOfShape& l) {
+	void copy_operand(const TopTools_ListOfShape& l, TopTools_ListOfShape& r) {
 #if OCC_VERSION_HEX < 0x70000
 		TopTools_ListOfShape r;
 		TopTools_ListIteratorOfListOfShape it(l);
@@ -169,7 +169,7 @@ namespace {
 		// On OCCT 7.0 and higher BRepAlgoAPI_BuilderAlgo::SetNonDestructive(true) is
 		// called. Not entirely sure on the behaviour before 7.0, so overcautiously
 		// create copies.
-		return l;
+		r.Assign(l);
 #endif
 	}
 
@@ -3381,7 +3381,7 @@ bool IfcGeom::Kernel::boolean_operation(const TopoDS_Shape& a, const TopoDS_Shap
 bool IfcGeom::Kernel::boolean_operation(const TopoDS_Shape& a, const TopTools_ListOfShape& b_, BOPAlgo_Operation op, TopoDS_Shape& result, double fuzziness) {
 	bool success = false;
 	BRepAlgoAPI_BooleanOperation* builder;
-	TopTools_ListOfShape b;
+	TopTools_ListOfShape B, b;
 	if (op == BOPAlgo_CUT) {
 		builder = new BRepAlgoAPI_Cut();
 		bounding_box_overlap(getValue(GV_PRECISION), a, b_, b);
@@ -3420,7 +3420,8 @@ bool IfcGeom::Kernel::boolean_operation(const TopoDS_Shape& a, const TopTools_Li
 #endif
 	builder->SetFuzzyValue(fuzz);
 	builder->SetArguments(s1s);
-	builder->SetTools(copy_operand(b));
+	copy_operand(b, B);
+	builder->SetTools(B);
 	builder->Build();
 	if (builder->IsDone()) {
 		TopoDS_Shape r = *builder;