Scene: Remove unnecessary call to wire_system::wire::simplify()

Tectu · Tectu · commit 700f6efc21c3 · 2025-06-20T01:48:00.000+02:00
This is unnecessary as the Commands::ItemMove command already simplifies the wire. Fixes #39
diff --git a/qschematic/scene.cpp b/qschematic/scene.cpp
@@ -639,14 +639,6 @@ Scene::mouseReleaseEvent(QGraphicsSceneMouseEvent* event)
                     if (const Items::Node* node = dynamic_cast<const Items::Node*>(item.get()); node)
                         updateNodeConnections(node);
                 }
-
-                for (auto& item : m_wire_manager->wires()) {
-                    Items::Wire* wire = dynamic_cast<Items::Wire*>(item.get());
-                    if (wire) {
-                        // wire->updatePosition();
-                        wire->simplify();
-                    }
-                }
             }
             break;
         }

Original file line number	Diff line number	Diff line change
`@@ -639,14 +639,6 @@ Scene::mouseReleaseEvent(QGraphicsSceneMouseEvent* event)`
`639`	`639`	`if (const Items::Node* node = dynamic_cast<const Items::Node*>(item.get()); node)`
`640`	`640`	`updateNodeConnections(node);`
`641`	`641`	`}`
`642`		`-`
`643`		`- for (auto& item : m_wire_manager->wires()) {`
`644`		`- Items::Wire* wire = dynamic_cast<Items::Wire*>(item.get());`
`645`		`- if (wire) {`
`646`		`- // wire->updatePosition();`
`647`		`- wire->simplify();`
`648`		`- }`
`649`		`- }`
`650`	`642`	`}`
`651`	`643`	`break;`
`652`	`644`	`}`