This pr marks several unused deprecated methods/constants/classes.

deepika-u · deepika-u · commit 614510aa284e · 2025-10-30T23:34:51.000-07:00
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/action/MenuManager.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/action/MenuManager.java
@@ -218,7 +218,7 @@ public Menu createMenuBar(Decorations parent) {
 	 * @return the menu control
 	 * @deprecated use <code>createMenuBar(Decorations)</code> instead.
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public Menu createMenuBar(Shell parent) {
 		return createMenuBar((Decorations) parent);
 	}
@@ -535,7 +535,7 @@ public boolean isSeparator() {
 	 * @deprecated this method is no longer a part of the
 	 *             {@link org.eclipse.jface.action.IContributionItem} API.
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public boolean isSubstituteFor(IContributionItem item) {
 		return this.equals(item);
 	}
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/action/SubContributionManager.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/action/SubContributionManager.java
@@ -205,7 +205,7 @@ protected void itemRemoved(IContributionItem item) {
 	 * @return fetch all enumeration of wrappers for the item
 	 * @deprecated Use getItems(String value) instead.
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public Enumeration<SubContributionItem> items() {
 		final Iterator<SubContributionItem> i = mapItemToWrapper.values().iterator();
 		return new Enumeration<>() {
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/IDialogConstants.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/IDialogConstants.java
@@ -411,7 +411,7 @@ public interface IDialogConstants {
 	 * @deprecated This constant is no longer in use.
 	 * The button heights are now determined by the layout.
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	int BUTTON_HEIGHT = 14;
 
 	/**
@@ -425,7 +425,7 @@ public interface IDialogConstants {
 	 * @deprecated As of 3.9, this is deprecated as it is too big as DLU and got accidentally used
 	 *             when setting pixel values
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	int INDENT = 21;
 
 	/**
@@ -434,7 +434,7 @@ public interface IDialogConstants {
 	 * @deprecated As of 3.9, this is deprecated as it is not clear what the meaning of a
 	 *             "small indent" would be
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	int SMALL_INDENT = 7;
 
 	/**
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/InputDialog.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/InputDialog.java
@@ -171,7 +171,7 @@ protected Control createDialogArea(Composite parent) {
 	 * @return the error message label
 	 * @deprecated use setErrorMessage(String) instead
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	protected Label getErrorMessageLabel() {
 		return null;
 	}
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/TitleAreaDialog.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/dialogs/TitleAreaDialog.java
@@ -69,7 +69,7 @@ public class TitleAreaDialog extends TrayDialog {
 	 * @since 2.0
 	 * @deprecated
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static final String INFO_MESSAGE = "INFO_MESSAGE"; //$NON-NLS-1$
 
 	/**
@@ -78,7 +78,7 @@ public class TitleAreaDialog extends TrayDialog {
 	 * @since 2.0
 	 * @deprecated
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static final String WARNING_MESSAGE = "WARNING_MESSAGE"; //$NON-NLS-1$
 
 	// Space between an image and a label
@@ -382,7 +382,7 @@ protected Point getInitialSize() {
 	 * @return Composite
 	 * @deprecated
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	protected Composite getTitleArea() {
 		return getShell();
 	}
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/fieldassist/TextControlCreator.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/fieldassist/TextControlCreator.java
@@ -25,7 +25,7 @@
  * @deprecated As of 3.3, clients should use {@link ControlDecoration} instead
  *             of {@link DecoratedField}.
  */
-@Deprecated
+@Deprecated(forRemoval = true, since = "2025-12")
 public class TextControlCreator implements IControlCreator {
 
 	@Override
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/operation/ProgressMonitorUtil.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/operation/ProgressMonitorUtil.java
@@ -45,7 +45,7 @@ public final class ProgressMonitorUtil {
 	 *
 	 * @since 3.13
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static org.eclipse.core.runtime.IProgressMonitorWithBlocking createAccumulatingProgressMonitor(
 			IProgressMonitor monitor,
 			Display display) {
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/ImageDataImageDescriptor.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/ImageDataImageDescriptor.java
@@ -46,7 +46,7 @@ class ImageDataImageDescriptor extends ImageDescriptor {
 	 * @param data describing the image
 	 * @deprecated use {@link #ImageDataImageDescriptor(ImageDataProvider)}
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	ImageDataImageDescriptor(ImageData data) {
 		this(zoom -> zoom == 100 ? data : null);
 	}
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/ImageDescriptor.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/ImageDescriptor.java
@@ -183,7 +183,7 @@ public static ImageDescriptor createWithFlags(ImageDescriptor originalImage, int
 	 * @param theDevice the device that was used to create the Image
 	 * @return a newly created image descriptor
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static ImageDescriptor createFromImage(Image img, Device theDevice) {
 		return new ImageDataImageDescriptor(img);
 	}
@@ -261,7 +261,7 @@ public static ImageDescriptor createFromURI(URI uriIconPath) {
 	 * @since 3.19
 	 * @deprecated Use {@link #createFromURI(URI)} instead.
 	 */
-	@Deprecated(since = "3.36", forRemoval = true)
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public ImageDescriptor imageDescriptorFromURI(URI uriIconPath) {
 		return createFromURI(uriIconPath);
 	}
@@ -451,7 +451,7 @@ public ImageData getImageData(int zoom) {
 	 *             {@link #getImageData(int)} to prevent endless cycle between the 2
 	 *             implementation in this class since 2017.</b>
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public ImageData getImageData() {
 		return getImageData(100);
 	}
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/JFaceColors.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/JFaceColors.java
@@ -176,7 +176,7 @@ public static Color getInformationViewerForegroundColor(Display display) {
 	 * @deprecated JFaceColors no longer maintains a cache of colors.  This job
 	 * is now handled by the ColorRegistry.
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static void clearColor(String colorName) {
 		//no-op
 	}
@@ -188,7 +188,7 @@ public static void clearColor(String colorName) {
 	 * @deprecated JFaceColors no longer maintains a cache of colors.  This job
 	 * is now handled by the ColorRegistry.
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static void disposeColors() {
 		//no-op
 	}
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/JFaceResources.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/JFaceResources.java
@@ -129,7 +129,7 @@ public class JFaceResources {
 	 *
 	 * @deprecated This font is not in use
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static final String VIEWER_FONT = "org.eclipse.jface.viewerfont"; //$NON-NLS-1$
 
 	/**
@@ -138,7 +138,7 @@ public class JFaceResources {
 	 *
 	 * @deprecated This font is not in use
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static final String WINDOW_FONT = "org.eclipse.jface.windowfont"; //$NON-NLS-1$
 
 	/**
@@ -564,7 +564,7 @@ public static FontDescriptor getTextFontDescriptor() {
 	 * @return the font
 	 * @deprecated This font is not in use
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static Font getViewerFont() {
 		return getFontRegistry().get(VIEWER_FONT);
 	}
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/StringConverter.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/resource/StringConverter.java
@@ -861,7 +861,7 @@ public static String removeWhiteSpaces(String s) {
 	 * @return The string representation of the font data object.
 	 * @deprecated use asString(FontData)
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static String asDisplayableString(FontData value) {
 		Assert.isNotNull(value);
 		StringBuilder buffer = new StringBuilder();
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/util/OpenStrategy.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/util/OpenStrategy.java
@@ -71,7 +71,7 @@ public class OpenStrategy {
 	 *
 	 * @deprecated
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static final int NO_TIMER = SINGLE_CLICK;
 
 	/** A single click will generate an open
@@ -80,15 +80,15 @@ public class OpenStrategy {
 	 *
 	 * @deprecated
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static final int FILE_EXPLORER = SINGLE_CLICK | ARROW_KEYS_OPEN;
 
 	/** Pointing to an item will change the selection
 	 * and a single click will gererate an open event
 	 *
 	 * @deprecated
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static final int ACTIVE_DESKTOP = SINGLE_CLICK | SELECT_ON_HOVER;
 
 	// Time used in FILE_EXPLORER and ACTIVE_DESKTOP
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/util/SafeRunnable.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/util/SafeRunnable.java
@@ -74,7 +74,7 @@ public void handleException(Throwable e) {
 	 * @return true if errors should be ignored
 	 * @deprecated use getIgnoreErrors()
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static boolean getIgnoreErrors(boolean flag) {
 		return ignoreErrors;
 	}
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/util/Util.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/util/Util.java
@@ -271,7 +271,7 @@ public static boolean endsWith(final Object[] left,
 	 *         <code>false</code> otherwise.
 	 * @deprecated Use {@link Objects#equals(Object, Object)}
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static boolean equals(final Object left, final Object right) {
 		return Objects.equals(left, right);
 	}
@@ -290,7 +290,7 @@ public static boolean equals(final Object left, final Object right) {
 	 *         the same position are equal; <code>false</code> otherwise.
 	 * @deprecated Use {@link Arrays#equals(Object[], Object[])}
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static boolean equals(final Object[] leftArray,
 			final Object[] rightArray) {
 		return Arrays.equals(leftArray, rightArray);
@@ -303,7 +303,7 @@ public static boolean equals(final Object[] leftArray,
 	 * @return <code>i</code>
 	 * @deprecated return directly value, or use {@link Integer#hashCode(int)}
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static int hashCode(final int i) {
 		return i;
 	}
@@ -316,7 +316,7 @@ public static int hashCode(final int i) {
 	 *         if <code>null</code>.
 	 * @deprecated use {@link Objects#hashCode(Object)}
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static int hashCode(final Object object) {
 		return object != null ? object.hashCode() : 0;
 	}
@@ -331,7 +331,7 @@ public static int hashCode(final Object object) {
 	 *         <code>objects</code> is <code>null</code>.
 	 * @deprecated use {@link Arrays#hashCode(Object[])}
 	 */
-	@Deprecated
+	@Deprecated(forRemoval = true, since = "2025-12")
 	public static int hashCode(final Object[] objects) {
 		if (objects == null) {
 			return 0;

Original file line number	Diff line number	Diff line change
`@@ -171,7 +171,7 @@ protected Control createDialogArea(Composite parent) {`
`171`	`171`	`* @return the error message label`
`172`	`172`	`* @deprecated use setErrorMessage(String) instead`
`173`	`173`	`*/`
`174`		`- @Deprecated`
	`174`	`+ @Deprecated(forRemoval = true, since = "2025-12")`
`175`	`175`	`protected Label getErrorMessageLabel() {`
`176`	`176`	`return null;`
`177`	`177`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ public final class ProgressMonitorUtil {`
`45`	`45`	`*`
`46`	`46`	`* @since 3.13`
`47`	`47`	`*/`
`48`		`- @Deprecated`
	`48`	`+ @Deprecated(forRemoval = true, since = "2025-12")`
`49`	`49`	`public static org.eclipse.core.runtime.IProgressMonitorWithBlocking createAccumulatingProgressMonitor(`
`50`	`50`	`IProgressMonitor monitor,`
`51`	`51`	`Display display) {`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ class ImageDataImageDescriptor extends ImageDescriptor {`
`46`	`46`	`* @param data describing the image`
`47`	`47`	`* @deprecated use {@link #ImageDataImageDescriptor(ImageDataProvider)}`
`48`	`48`	`*/`
`49`		`- @Deprecated`
	`49`	`+ @Deprecated(forRemoval = true, since = "2025-12")`
`50`	`50`	`ImageDataImageDescriptor(ImageData data) {`
`51`	`51`	`this(zoom -> zoom == 100 ? data : null);`
`52`	`52`	`}`