kindred/create
1
1
Fork 0
Code Issues 44 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

Surface: modernize C++: use range-based for loop

Browse Source
This commit is contained in:
wmayer
2023-08-15 15:28:49 +02:00
committed by wwmayer
parent 96f27bf43f
commit 30d98dd417
2 changed files with 6 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/Mod/Surface/App/FeatureGeomFillSurface.cpp
View File

@@ -210,11 +210,9 @@ bool GeomFillSurface::getWire(TopoDS_Wire& aWire)
}
ShapeValidator validator;
for(std::size_t i = 0; i < boundary.size(); i++) {
App::PropertyLinkSubList::SubSet set = boundary[i];
for(const auto& set : boundary) {
if (set.first->getTypeId().isDerivedFrom(Part::Feature::getClassTypeId())) {
for (const auto& jt: set.second) {
for (const auto& jt : set.second) {
const Part::TopoShape &ts = static_cast<Part::Feature*>(set.first)->Shape.getShape();
validator.checkAndAdd(ts, jt.c_str(), &aWD);
}

8
src/Mod/Surface/App/FeatureSewing.cpp
View File

@@ -73,15 +73,15 @@ App::DocumentObjectExecReturn *Sewing::execute()
BRepBuilderAPI_Sewing builder(atol,opt1,opt2,opt3,opt4);
std::vector<App::PropertyLinkSubList::SubSet> subset = ShapeList.getSubListValues();
for(std::vector<App::PropertyLinkSubList::SubSet>::iterator it = subset.begin(); it != subset.end(); ++it) {
for(const auto& it : subset) {
// the subset has the documentobject and the element name which belongs to it,
// in our case for example the cube object and the "Edge1" string
if (it->first->getTypeId().isDerivedFrom(Part::Feature::getClassTypeId())) {
if (it.first->getTypeId().isDerivedFrom(Part::Feature::getClassTypeId())) {
//we get the shape of the document object which resemble the whole box
Part::TopoShape ts = static_cast<Part::Feature*>(it->first)->Shape.getShape();
Part::TopoShape ts = static_cast<Part::Feature*>(it.first)->Shape.getShape();
//we want only the subshape which is linked
for (const auto& jt: it->second) {
for (const auto& jt : it.second) {
TopoDS_Shape sub = ts.getSubShape(jt.c_str());
builder.Add(sub);
}