DataMouseListener: Don't set an empty highlight on mouse move -- this causes some paintstyles to flicker

tsmock · tsmock · commit d10a7055b6f3 · 2023-02-01T12:22:07.000-07:00
Signed-off-by: Taylor Smock &lt;tsmock@meta.com&gt;
diff --git a/src/main/java/org/openstreetmap/josm/plugins/mapillary/gui/DataMouseListener.java b/src/main/java/org/openstreetmap/josm/plugins/mapillary/gui/DataMouseListener.java
@@ -145,7 +145,7 @@ public void mouseMoved(MouseEvent e) {
                             layer.getData().setHighlighted(
                                 ways.stream().map(IPrimitive::getPrimitiveId).collect(Collectors.toSet()));
                             layer.invalidate();
-                        } else {
+                        } else if (!layer.getData().getHighlighted().isEmpty()) {
                             layer.getData().setHighlighted(Collections.emptyList());
                         }
                     } finally {

Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ public void mouseMoved(MouseEvent e) {`
`145`	`145`	`layer.getData().setHighlighted(`
`146`	`146`	`ways.stream().map(IPrimitive::getPrimitiveId).collect(Collectors.toSet()));`
`147`	`147`	`layer.invalidate();`
`148`		`- } else {`
	`148`	`+ } else if (!layer.getData().getHighlighted().isEmpty()) {`
`149`	`149`	`layer.getData().setHighlighted(Collections.emptyList());`
`150`	`150`	`}`
`151`	`151`	`} finally {`