From dfa62af40f2955d73c6ff48ee73f238bce7d0db2 Mon Sep 17 00:00:00 2001 From: Uwe Date: Wed, 5 Jan 2022 03:16:00 +0100 Subject: [PATCH] [PD] avoid a single-letter variable (single letter variables make it hard to read and understand code quickly and are error-prone for typos) --- src/Mod/PartDesign/App/FeatureTransformed.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Mod/PartDesign/App/FeatureTransformed.cpp b/src/Mod/PartDesign/App/FeatureTransformed.cpp index 7de5640a13..65b6270b6e 100644 --- a/src/Mod/PartDesign/App/FeatureTransformed.cpp +++ b/src/Mod/PartDesign/App/FeatureTransformed.cpp @@ -265,19 +265,19 @@ App::DocumentObjectExecReturn *Transformed::execute(void) std::vector shapes; bool overlapping = false; - std::vector::const_iterator t = transformations.begin(); + std::vector::const_iterator transformIter = transformations.begin(); // First transformation is skipped since it should not be part of the toolShape. - t++; + transformIter++; - for (; t != transformations.end(); ++t) { + for (; transformIter != transformations.end(); ++transformIter) { // Make an explicit copy of the shape because the "true" parameter to BRepBuilderAPI_Transform // seems to be pretty broken BRepBuilderAPI_Copy copy(origShape); shape = copy.Shape(); - BRepBuilderAPI_Transform mkTrf(shape, *t, false); // No need to copy, now + BRepBuilderAPI_Transform mkTrf(shape, *transformIter, false); // No need to copy, now if (!mkTrf.IsDone()) return new App::DocumentObjectExecReturn("Transformation failed", (*o)); shape = mkTrf.Shape();