[GTK] Fix MenuItem/ToolItem setImage(null)

akurtakov · akurtakov · commit 1adca2e55dee · 2025-08-27T16:53:58.000+03:00
Fix for https://bugs.eclipse.org/bugs/show_bug.cgi?id=568771 had the wrong assumption that _setImage should be called only when non null image. Fixes #2457
diff --git a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/MenuItem.java b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/MenuItem.java
@@ -987,7 +987,7 @@ private void _setEnabledOrDisabledImage() {
 		}
 		_setImage(defaultDisableImage);
 	}
-	if (enabled && image != null) _setImage(image);
+	if (enabled) _setImage(image);
 }
 
 /**
diff --git a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/ToolItem.java b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/ToolItem.java
@@ -1229,7 +1229,7 @@ private void _setEnabledOrDisabledImage() {
 			_setImage(disabledImage);
 		}
 	}
-	if (enabled && image != null) _setImage(image);
+	if (enabled) _setImage(image);
 }
 
 boolean setFocus () {

Original file line number	Diff line number	Diff line change
`@@ -987,7 +987,7 @@ private void _setEnabledOrDisabledImage() {`
`987`	`987`	`}`
`988`	`988`	`_setImage(defaultDisableImage);`
`989`	`989`	`}`
`990`		`- if (enabled && image != null) _setImage(image);`
	`990`	`+ if (enabled) _setImage(image);`
`991`	`991`	`}`
`992`	`992`
`993`	`993`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -1229,7 +1229,7 @@ private void _setEnabledOrDisabledImage() {`
`1229`	`1229`	`_setImage(disabledImage);`
`1230`	`1230`	`}`
`1231`	`1231`	`}`
`1232`		`- if (enabled && image != null) _setImage(image);`
	`1232`	`+ if (enabled) _setImage(image);`
`1233`	`1233`	`}`
`1234`	`1234`
`1235`	`1235`	`boolean setFocus () {`