From b3b45893cc405c3ac201c69aca122fa97cc73bc4 Mon Sep 17 00:00:00 2001 From: luzpaz Date: Fri, 7 Jul 2023 14:36:01 +0000 Subject: [PATCH] Fix various typos in src/ifcgeom* subdirs --- src/ifcgeom/IfcBooleanResult.cpp | 2 +- src/ifcgeom/IfcCompositeCurve.cpp | 2 +- src/ifcgeom/IfcCurveBoundedPlane.cpp | 2 +- src/ifcgeom/IfcFace.cpp | 2 +- src/ifcgeom/IfcGeom.cpp | 6 +++--- src/ifcgeom/IfcGeomIteratorImplementation.h | 2 +- src/ifcgeom/IfcTrimmedCurve.cpp | 4 ++-- src/ifcgeom/faceset_helper.cpp | 2 +- src/ifcgeom_schema_agnostic/base_utils.cpp | 2 +- src/ifcgeom_schema_agnostic/boolean_utils.cpp | 6 +++--- 10 files changed, 15 insertions(+), 15 deletions(-) diff --git a/src/ifcgeom/IfcBooleanResult.cpp b/src/ifcgeom/IfcBooleanResult.cpp index c342d9be78..8e076511ce 100644 --- a/src/ifcgeom/IfcBooleanResult.cpp +++ b/src/ifcgeom/IfcBooleanResult.cpp @@ -175,7 +175,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcBooleanResult* l, TopoDS_Shape TopoDS_Shape temp; double d; if (util::fit_halfspace(s1, s2, temp, d, getValue(GV_PRECISION) * 1000.)) { - // #2665 we also set a precision-independent treshold, because in the boolean op routine + // #2665 we also set a precision-independent threshold, because in the boolean op routine // the working fuzziness might still be increased. if (d < getValue(GV_PRECISION) * 20. || d < 0.00002) { Logger::Message(Logger::LOG_WARNING, "Halfspace subtraction yields unchanged volume:", l); diff --git a/src/ifcgeom/IfcCompositeCurve.cpp b/src/ifcgeom/IfcCompositeCurve.cpp index 1986ac80cc..13837b9382 100644 --- a/src/ifcgeom/IfcCompositeCurve.cpp +++ b/src/ifcgeom/IfcCompositeCurve.cpp @@ -169,7 +169,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcCompositeCurve* l, TopoDS_Wire } if (converted_segments.Extent() == 0) { - Logger::Message(Logger::LOG_ERROR, "No segment succesfully converted:", l); + Logger::Message(Logger::LOG_ERROR, "No segment successfully converted:", l); return false; } diff --git a/src/ifcgeom/IfcCurveBoundedPlane.cpp b/src/ifcgeom/IfcCurveBoundedPlane.cpp index 07f25981c6..1f8143e164 100644 --- a/src/ifcgeom/IfcCurveBoundedPlane.cpp +++ b/src/ifcgeom/IfcCurveBoundedPlane.cpp @@ -62,7 +62,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcCurveBoundedPlane* l, TopoDS_S ShapeFix_Shape sfs(mf.Face()); sfs.Perform(); - // `trsf` consitutes the placement of the plane and therefore has unit scale factor + // `trsf` constitutes the placement of the plane and therefore has unit scale factor face = TopoDS::Face(sfs.Shape()).Moved(trsf); return true; diff --git a/src/ifcgeom/IfcFace.cpp b/src/ifcgeom/IfcFace.cpp index c544bcd612..37a265fefb 100644 --- a/src/ifcgeom/IfcFace.cpp +++ b/src/ifcgeom/IfcFace.cpp @@ -56,7 +56,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcFace* l, TopoDS_Shape& result) TopoDS_Shape surface_shape; if (!convert_shape(fs->FaceSurface(), surface_shape)) return false; - // FIXME: Assert this obtaines the only face + // FIXME: Assert this obtains the only face TopExp_Explorer exp(surface_shape, TopAbs_FACE); if (!exp.More()) return false; diff --git a/src/ifcgeom/IfcGeom.cpp b/src/ifcgeom/IfcGeom.cpp index d0f17a46c8..d0b5ebd19f 100644 --- a/src/ifcgeom/IfcGeom.cpp +++ b/src/ifcgeom/IfcGeom.cpp @@ -1239,7 +1239,7 @@ bool IfcGeom::Kernel::fold_layers(const IfcSchema::IfcWall* wall, const IfcRepre typedef std::vector< std::vector > result_t; endpoint_connections_t endpoint_connections; - // Find the semantic connections ot other wall elements when they are not connected 'AT_PATH' because + // Find the semantic connections to other wall elements when they are not connected 'AT_PATH' because // in that latter case no folds need to be made. for (IfcSchema::IfcRelConnectsPathElements::list::it it = connections->begin(); it != connections->end(); ++it) { IfcSchema::IfcRelConnectsPathElements* connection = *it; @@ -1360,7 +1360,7 @@ bool IfcGeom::Kernel::fold_layers(const IfcSchema::IfcWall* wall, const IfcRepre // range. It's only a safeguard though, so can probably be approximated. const double axis_length = own_axis_start.Distance(own_axis_end); if (length_required > axis_length) { - Logger::Warning("The wall axis is not long enough to accomodate the fold points"); + Logger::Warning("The wall axis is not long enough to accommodate the fold points"); return false; } @@ -1455,7 +1455,7 @@ bool IfcGeom::Kernel::fold_layers(const IfcSchema::IfcWall* wall, const IfcRepre result_t::iterator result_vector = result.begin() + 1; // nb The first layer is never folded, because it corresponds - // to one of the longitudonal faces of the wall. Hence the +1 + // to one of the longitudinal faces of the wall. Hence the +1 for (surfaces_t::const_iterator jt = surfaces.begin() + 1; jt != surfaces.end() - 1; ++jt, ++result_vector) { layer_offset += *thickness++; diff --git a/src/ifcgeom/IfcGeomIteratorImplementation.h b/src/ifcgeom/IfcGeomIteratorImplementation.h index ea1d93bafe..5cf70522f9 100644 --- a/src/ifcgeom/IfcGeomIteratorImplementation.h +++ b/src/ifcgeom/IfcGeomIteratorImplementation.h @@ -700,7 +700,7 @@ namespace IfcGeom { } } - // Check if this represenation has (or will be) processed as part its mapped representation + // Check if this representation has (or will be) processed as part its mapped representation bool representation_processed_as_mapped_item = false; IfcSchema::IfcRepresentation* representation_mapped_to = kernel.representation_mapped_to(representation); if (representation_mapped_to) { diff --git a/src/ifcgeom/IfcTrimmedCurve.cpp b/src/ifcgeom/IfcTrimmedCurve.cpp index 77140a1e2a..76a8760afc 100644 --- a/src/ifcgeom/IfcTrimmedCurve.cpp +++ b/src/ifcgeom/IfcTrimmedCurve.cpp @@ -172,7 +172,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcTrimmedCurve* l, TopoDS_Wire& // Fix from @sanderboer to compare using model tolerance, see #744 // Made dependent on radius, see #928 - // A good critereon for determining whether to take full curve + // A good criterion for determining whether to take full curve // or trimmed segment would be whether there are other curve segments or this // is the only one. boost::optional num_segments; @@ -227,7 +227,7 @@ bool IfcGeom::Kernel::convert(const IfcSchema::IfcTrimmedCurve* l, TopoDS_Wire& TopoDS_Vertex v0, v1; TopExp::Vertices(e, v0, v1); e = TopoDS::Edge(BRepBuilderAPI_MakeEdge(v0, v1).Edge().Oriented(e.Orientation())); - Logger::Warning("Subsituted edge with linear approximation", l); + Logger::Warning("Substituted edge with linear approximation", l); } } diff --git a/src/ifcgeom/faceset_helper.cpp b/src/ifcgeom/faceset_helper.cpp index 0ee89a30a7..e4e1228343 100644 --- a/src/ifcgeom/faceset_helper.cpp +++ b/src/ifcgeom/faceset_helper.cpp @@ -72,7 +72,7 @@ IfcGeom::Kernel::faceset_helper::faceset_helper( // double bdiff = std::sqrt(box.SquareExtent()); // @todo the bounding box diagonal is not used (see above) - // because we're explicitly interested in the miminal + // because we're explicitly interested in the minimal // dimension of the element to limit the tolerance (for sheet- // like elements for example). But the way below is very // dependent on orientation due to the usage of the diff --git a/src/ifcgeom_schema_agnostic/base_utils.cpp b/src/ifcgeom_schema_agnostic/base_utils.cpp index f656fb45a9..e9eff396d4 100644 --- a/src/ifcgeom_schema_agnostic/base_utils.cpp +++ b/src/ifcgeom_schema_agnostic/base_utils.cpp @@ -589,7 +589,7 @@ bool IfcGeom::util::create_solid_from_faces(const TopTools_ListOfShape& face_lis bool has_shared_edges = false; TopTools_MapOfShape edge_set; - // In case there are wire interesections or failures in non-planar wire triangulations + // In case there are wire intersections or failures in non-planar wire triangulations // the idea is to let occt do an exhaustive search of edge partners. But we have not // found a case where this actually improves boolean ops later on. // if (!faceset_helper_ || !faceset_helper_->non_manifold()) { diff --git a/src/ifcgeom_schema_agnostic/boolean_utils.cpp b/src/ifcgeom_schema_agnostic/boolean_utils.cpp index f8815523ab..e7548b0d82 100644 --- a/src/ifcgeom_schema_agnostic/boolean_utils.cpp +++ b/src/ifcgeom_schema_agnostic/boolean_utils.cpp @@ -1183,9 +1183,9 @@ bool IfcGeom::util::boolean_operation(const boolean_settings& settings, const To } if (!success) { - PERF("boolean operation: manifoldness check excemption"); + PERF("boolean operation: manifoldness check exemption"); - // An excemption for the requirement to be manifold: When the cut operands have overlapping edge belonging to faces that do not overlap. + // An exemption for the requirement to be manifold: When the cut operands have overlapping edge belonging to faces that do not overlap. bool operands_nonmanifold = false; if (op == BOPAlgo_CUT) { TopTools_IndexedMapOfShape edges; @@ -1248,7 +1248,7 @@ bool IfcGeom::util::boolean_operation(const boolean_settings& settings, const To bool has_open_shells = false; if (op == BOPAlgo_CUT) { - PERF("boolean operation: open shell face adition check"); + PERF("boolean operation: open shell face addition check"); for (TopExp_Explorer exp(a, TopAbs_SHELL); exp.More(); exp.Next()) { if (!exp.Current().Closed()) {