Add early return to ToolBar::handleDPIChange in case it has no items

fedejeanne · fedejeanne · commit 0af7bbcdf4de · 2024-08-08T09:49:59.000+02:00
No need to send OS events if there are no children to process in the
toolbar
diff --git a/bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/ToolBar.java b/bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/ToolBar.java
@@ -1754,6 +1754,10 @@ private static void handleDPIChange(Widget widget, int newZoom, float scalingFac
 	var seperatorWidth = new int[toolItems.length];
 	int itemCount = toolItems.length;
 
+	if (itemCount == 0) {
+		return;
+	}
+
 	record ToolItemData(ToolItem toolItem, TBBUTTON button) {
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1754,6 +1754,10 @@ private static void handleDPIChange(Widget widget, int newZoom, float scalingFac`
`1754`	`1754`	`var seperatorWidth = new int[toolItems.length];`
`1755`	`1755`	`int itemCount = toolItems.length;`
`1756`	`1756`
	`1757`	`+ if (itemCount == 0) {`
	`1758`	`+ return;`
	`1759`	`+ }`
	`1760`	`+`
`1757`	`1761`	`record ToolItemData(ToolItem toolItem, TBBUTTON button) {`
`1758`	`1762`	`}`
`1759`	`1763`