fix(swingui): replace invalid Optional.set() with field reassignment

Copilot · DanySK · web-flow · commit 6aecbcbf1acb · 2026-04-30T09:30:21.000Z
Agent-Logs-Url: https://github.com/AlchemistSimulator/Alchemist/sessions/81d95530-b420-47ed-a40d-0b662f7526fa Co-authored-by: DanySK <1991673+DanySK@users.noreply.github.com>
diff --git a/alchemist-swingui/src/main/java/it/unibo/alchemist/boundary/swingui/monitor/impl/Generic2DDisplay.java b/alchemist-swingui/src/main/java/it/unibo/alchemist/boundary/swingui/monitor/impl/Generic2DDisplay.java
@@ -373,7 +373,7 @@ private void updateHookedNodeView() {
         final Node<T> hookedNode = hooked.get();
         final P hookedCoordinates = positions.get(hookedNode);
         if (hookedCoordinates == null) {
-            hooked.set(Optional.empty());
+            hooked = Optional.empty();
             return;
         }
         final Point hookedPoint = wormhole.getViewPoint(hookedCoordinates);

Original file line number	Diff line number	Diff line change
`@@ -373,7 +373,7 @@ private void updateHookedNodeView() {`
`373`	`373`	`final Node<T> hookedNode = hooked.get();`
`374`	`374`	`final P hookedCoordinates = positions.get(hookedNode);`
`375`	`375`	`if (hookedCoordinates == null) {`
`376`		`- hooked.set(Optional.empty());`
	`376`	`+ hooked = Optional.empty();`
`377`	`377`	`return;`
`378`	`378`	`}`
`379`	`379`	`final Point hookedPoint = wormhole.getViewPoint(hookedCoordinates);`