diff --git a/src/ifcgeom/AbstractKernel.cpp b/src/ifcgeom/AbstractKernel.cpp index 9954b86531..96b5617b8e 100644 --- a/src/ifcgeom/AbstractKernel.cpp +++ b/src/ifcgeom/AbstractKernel.cpp @@ -206,8 +206,8 @@ ifcopenshell::geometry::kernels::AbstractKernel* ifcopenshell::geometry::kernels } #endif if (kernels.size() != n + 1) { - for (auto k : kernels) - delete k; + for (auto k : kernels) + delete k; throw IfcParse::IfcException("Invalid hybrid kernel " + geometry_library); } } diff --git a/src/ifcgeom/Converter.cpp b/src/ifcgeom/Converter.cpp index 15c06aef47..9c6739292d 100644 --- a/src/ifcgeom/Converter.cpp +++ b/src/ifcgeom/Converter.cpp @@ -15,12 +15,12 @@ ifcopenshell::geometry::Converter::Converter(const std::string& geometry_library ifcopenshell::geometry::Converter::~Converter() { - if (kernel_ != nullptr) { - delete kernel_; - } - if (mapping_ != nullptr) { - delete mapping_; - } + if (kernel_ != nullptr) { + delete kernel_; + } + if (mapping_ != nullptr) { + delete mapping_; + } } namespace { diff --git a/src/ifcgeom/kernels/opencascade/boolean_utils.cpp b/src/ifcgeom/kernels/opencascade/boolean_utils.cpp index 05d983f22d..2308ff869c 100644 --- a/src/ifcgeom/kernels/opencascade/boolean_utils.cpp +++ b/src/ifcgeom/kernels/opencascade/boolean_utils.cpp @@ -978,7 +978,7 @@ bool IfcGeom::util::boolean_operation(const boolean_settings& settings, const To if (b.Extent() == 0) { Logger::Warning("No other operands remaining, using first operand"); result = a; - delete builder; + delete builder; return true; } @@ -1131,7 +1131,7 @@ bool IfcGeom::util::boolean_operation(const boolean_settings& settings, const To } else { Logger::Notice("Processed fully in 2D"); result = mp.Shape(); - delete builder; + delete builder; return true; } } else {