From 617e63bc23413ea04632f03b03404d98c5aeb152 Mon Sep 17 00:00:00 2001
From: bgbsww <bgbsww@gmail.com>
Date: Mon, 3 Jun 2024 14:50:16 -0400
Subject: [PATCH] Toponaming: reformat code

---
 src/Mod/PartDesign/App/FeatureDressUp.cpp     |  2 +-
 .../PartDesign/Gui/TaskDressUpParameters.cpp  | 22 ++++++++++---------
 2 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/src/Mod/PartDesign/App/FeatureDressUp.cpp b/src/Mod/PartDesign/App/FeatureDressUp.cpp
index e91e8eb249..57bbc03637 100644
--- a/src/Mod/PartDesign/App/FeatureDressUp.cpp
+++ b/src/Mod/PartDesign/App/FeatureDressUp.cpp
@@ -206,7 +206,7 @@ std::vector<TopoShape> DressUp::getContinuousEdges(const TopoShape& shape)
         const auto& ref = v.first.size() ? v.first : v.second;
         subshape = shape.getSubShape(ref.c_str(), true);
         if (subshape.IsNull()) {
-            FC_THROWM(Base::CADKernelError, "Invalid edge link: " << v.second);
+            FC_THROWM(Base::CADKernelError, "Invalid edge link: " << ref);
         }
 
         if (subshape.ShapeType() == TopAbs_EDGE) {
diff --git a/src/Mod/PartDesign/Gui/TaskDressUpParameters.cpp b/src/Mod/PartDesign/Gui/TaskDressUpParameters.cpp
index 0692b303fd..652805afa8 100644
--- a/src/Mod/PartDesign/Gui/TaskDressUpParameters.cpp
+++ b/src/Mod/PartDesign/Gui/TaskDressUpParameters.cpp
@@ -141,32 +141,34 @@ void TaskDressUpParameters::referenceSelected(const Gui::SelectionChanges& msg,
 void TaskDressUpParameters::addAllEdges(QListWidget* widget)
 {
 #ifdef FC_USE_TNP_FIX
-    if (!DressUpView)
+    if (!DressUpView) {
         return;
+    }
 
     PartDesign::DressUp* pcDressUp = static_cast<PartDesign::DressUp*>(DressUpView->getObject());
     App::DocumentObject* base = pcDressUp->Base.getValue();
-    if (!base)
+    if (!base) {
         return;
+    }
     int count = Part::Feature::getTopoShape(base).countSubShapes(TopAbs_EDGE);
     auto subValues = pcDressUp->Base.getSubValues(false);
     std::size_t len = subValues.size();
     std::string name("Edge");
-    for (int i=0; i<count; ++i) {
-        name.resize(4);
-        name += std::to_string(i+1);
-        if (std::find(subValues.begin(), subValues.begin()+len, name) == subValues.begin()+len)
+    for (int i = 0; i < count; ++i) {
+        std::string name = "Edge" + std::to_string(i+1);
+        if (std::find(subValues.begin(), subValues.begin() + len, name)
+            == subValues.begin() + len) {
             subValues.push_back(name);
+        }
     }
-    if (subValues.size() == len)
+    if (subValues.size() == len) {
         return;
+    }
     try {
         setupTransaction();
         pcDressUp->Base.setValue(base, subValues);
-//        recompute();
-//        populate(true);
     }
-    catch (Base::Exception &e) {
+    catch (Base::Exception& e) {
         e.ReportException();
     }
 #else