From ec10b5e62ef0e61b113f4c8b7b0f26ca28dde63b Mon Sep 17 00:00:00 2001
From: "Zheng, Lei" <realthunder.dev@gmail.com>
Date: Fri, 24 Jan 2020 07:58:57 +0800
Subject: [PATCH] Path: add some diagnostic trace in Area

---
 src/Mod/Path/App/Area.cpp | 24 +++++++++++++++++++++++-
 src/Mod/Path/App/Area.h   |  2 ++
 2 files changed, 25 insertions(+), 1 deletion(-)

diff --git a/src/Mod/Path/App/Area.cpp b/src/Mod/Path/App/Area.cpp
index 6ab2c1dd66..fe552ecbd9 100644
--- a/src/Mod/Path/App/Area.cpp
+++ b/src/Mod/Path/App/Area.cpp
@@ -135,6 +135,20 @@ AreaParams::AreaParams()
     :PARAM_INIT(PARAM_FNAME,AREA_PARAMS_AREA)
 {}
 
+void AreaParams::dump(const char *msg) const {
+
+#define AREA_PARAM_PRINT(_param) \
+    ss << PARAM_FNAME_STR(_param) << " = " << PARAM_FNAME(_param) << '\n';
+
+    if(FC_LOG_INSTANCE.level()>FC_LOGLEVEL_TRACE) {
+        std::ostringstream ss;
+        ss << msg << '\n';
+        PARAM_FOREACH(AREA_PARAM_PRINT, AREA_PARAMS_AREA)
+
+        FC_MSG(ss.str());
+    }
+}
+
 CAreaConfig::CAreaConfig(const CAreaParams &p, bool noFitArcs)
 {
 #define AREA_CONF_SAVE_AND_APPLY(_param) \
@@ -877,7 +891,9 @@ struct WireJoiner {
             if(info.p1.SquareDistance(info.p2)<tol)
 #endif
             {
-                builder.Add(comp,BRepBuilderAPI_MakeWire(info.edge).Wire());
+                auto wire = BRepBuilderAPI_MakeWire(info.edge).Wire();
+                Area::showShape(wire,"closed");
+                builder.Add(comp,wire);
                 ++count;
                 continue;
             }
@@ -1002,6 +1018,7 @@ struct WireJoiner {
                 TopoDS_Wire wire = makeCleanWire(wireData,0.01);
                 if(!BRep_Tool::IsClosed(wire)) {
                     FC_WARN("failed to close some projection wire");
+                    Area::showShape(wire,"failed");
                     ++skips;
                 }else{
                     for(auto &r : stack) {
@@ -1271,6 +1288,8 @@ int Area::project(TopoDS_Shape &shape_out,
     int skips = joiner.findClosedWires();
     FC_TIME_LOG(t1,"WireJoiner findClosedWires");
 
+    showShape(joiner.comp,"pre_project");
+
     Area area(params);
     area.myParams.SectionCount = 0;
     area.myParams.Offset = 0.0;
@@ -1284,6 +1303,9 @@ int Area::project(TopoDS_Shape &shape_out,
     area.myProjecting = true;
     area.add(joiner.comp, OperationUnion);
     const TopoDS_Shape &shape = area.getShape();
+
+    area.myParams.dump("project");
+
     showShape(shape,"projected");
 
     FC_TIME_LOG(t1,"Clipper wire union");
diff --git a/src/Mod/Path/App/Area.h b/src/Mod/Path/App/Area.h
index af0cca4ec8..3e0c70f5f0 100644
--- a/src/Mod/Path/App/Area.h
+++ b/src/Mod/Path/App/Area.h
@@ -68,6 +68,8 @@ struct PathExport AreaParams: CAreaParams {
         return !(*this == other);
     }
 
+    void dump(const char *) const;
+
     AreaParams();
 };