Fix Position handling for IfcSurfaceCurveSweptAreaSolid

dleverton · dleverton · commit d12cacbd2ed9 · 2019-02-13T15:34:50.000Z
diff --git a/src/ifcgeom/IfcGeomShapes.cpp b/src/ifcgeom/IfcGeomShapes.cpp
@@ -885,7 +885,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcRectangularTrimmedSurface* l,
 }
 
 bool IfcGeom::Kernel::convert(const IfcSchema::IfcSurfaceCurveSweptAreaSolid* l, TopoDS_Shape& shape) {
-	gp_Trsf directrix, position;
+	gp_Trsf directrix;
 	TopoDS_Shape face;
 	TopoDS_Wire wire, section;
 
@@ -964,7 +964,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcSurfaceCurveSweptAreaSolid* l,
 	if (has_position) {
 		// IfcSweptAreaSolid.Position (trsf) is an IfcAxis2Placement3D
 		// and therefore has a unit scale factor
-		shape.Move(position);
+		shape.Move(trsf);
 	}
 
 	return true;

Original file line number	Diff line number	Diff line change
`@@ -885,7 +885,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcRectangularTrimmedSurface* l,`
`885`	`885`	`}`
`886`	`886`
`887`	`887`	`bool IfcGeom::Kernel::convert(const IfcSchema::IfcSurfaceCurveSweptAreaSolid* l, TopoDS_Shape& shape) {`
`888`		`- gp_Trsf directrix, position;`
	`888`	`+ gp_Trsf directrix;`
`889`	`889`	`TopoDS_Shape face;`
`890`	`890`	`TopoDS_Wire wire, section;`
`891`	`891`
`@@ -964,7 +964,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcSurfaceCurveSweptAreaSolid* l,`
`964`	`964`	`if (has_position) {`
`965`	`965`	`// IfcSweptAreaSolid.Position (trsf) is an IfcAxis2Placement3D`
`966`	`966`	`// and therefore has a unit scale factor`
`967`		`- shape.Move(position);`
	`967`	`+ shape.Move(trsf);`
`968`	`968`	`}`
`969`	`969`
`970`	`970`	`return true;`