From eef741b0959e22ccdecda2786b986b249892b0fa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Hejl?= Date: Wed, 16 Feb 2022 09:04:06 +0100 Subject: Fixed a copy-paste typo that could lead to duplicate edges in MMU_Graph in the multi-material segmentation. --- src/libslic3r/MultiMaterialSegmentation.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/libslic3r/MultiMaterialSegmentation.cpp b/src/libslic3r/MultiMaterialSegmentation.cpp index 922f28a63..f9a53b51d 100644 --- a/src/libslic3r/MultiMaterialSegmentation.cpp +++ b/src/libslic3r/MultiMaterialSegmentation.cpp @@ -605,7 +605,7 @@ struct MMU_Graph if (arcs[arc_idx].to_idx == to_idx) return; for (const size_t &arc_idx : this->nodes[to_idx].arc_idxs) - if (arcs[arc_idx].to_idx == to_idx) + if (arcs[arc_idx].to_idx == from_idx) return; this->nodes[from_idx].arc_idxs.push_back(this->arcs.size()); @@ -1201,7 +1201,7 @@ static inline double compute_edge_length(const MMU_Graph &graph, const size_t st used_arcs[start_arc_idx] = true; const MMU_Graph::Arc *arc = &graph.arcs[start_arc_idx]; size_t idx = start_idx; - double line_total_length = (graph.nodes[arc->to_idx].point - graph.nodes[idx].point).norm();; + double line_total_length = (graph.nodes[arc->to_idx].point - graph.nodes[idx].point).norm(); while (graph.nodes[arc->to_idx].arc_idxs.size() == 2) { bool found = false; for (const size_t &arc_idx : graph.nodes[arc->to_idx].arc_idxs) { -- cgit v1.2.3