style(swingui): fix Graphics2D thread-safety in highlightSelectedNodes

Copilot · DanySK · DanySK · commit 23acbbc0b468 · 2026-04-30T11:16:20.000+02:00
Agent-Logs-Url: https://github.com/AlchemistSimulator/Alchemist/sessions/4322e07a-9841-40ad-a5d1-10555384ad83 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
@@ -483,7 +483,8 @@ private void highlightSelectedNodes(final Graphics2D g, final Map<Node<T>, Point
             .map(node -> Optional.ofNullable(onView.get(node)))
             .filter(Optional::isPresent)
             .map(Optional::get)
-            .forEachOrdered(point -> drawFriedEgg(g, point.x, point.y, Color.BLUE, Color.CYAN));
+            .collect(Collectors.toList())
+            .forEach(point -> drawFriedEgg(g, point.x, point.y, Color.BLUE, Color.CYAN));
     }
 
     private void drawFriedEgg(final Graphics g, final int x, final int y, final Color c1, final Color c2) {

Original file line number	Diff line number	Diff line change
`@@ -483,7 +483,8 @@ private void highlightSelectedNodes(final Graphics2D g, final Map<Node<T>, Point`
`483`	`483`	`.map(node -> Optional.ofNullable(onView.get(node)))`
`484`	`484`	`.filter(Optional::isPresent)`
`485`	`485`	`.map(Optional::get)`
`486`		`- .forEachOrdered(point -> drawFriedEgg(g, point.x, point.y, Color.BLUE, Color.CYAN));`
	`486`	`+ .collect(Collectors.toList())`
	`487`	`+ .forEach(point -> drawFriedEgg(g, point.x, point.y, Color.BLUE, Color.CYAN));`
`487`	`488`	`}`
`488`	`489`
`489`	`490`	`private void drawFriedEgg(final Graphics g, final int x, final int y, final Color c1, final Color c2) {`