From 3090024549b8fed622f11a980729f7d6aa18a708 Mon Sep 17 00:00:00 2001 From: Alexander Kurtakov Date: Wed, 20 Mar 2019 16:31:43 +0200 Subject: Remove useless non-javadoc see comments. Handled by Override annotation. Change-Id: Icd0334c0b57bb3021693117877a9af5fcda571d7 Signed-off-by: Alexander Kurtakov --- .../ui/AbstractBreakpointOrganizerDelegate.java | 40 +++++----------------- .../debug/ui/AbstractLaunchConfigurationTab.java | 4 --- .../eclipse/debug/ui/BreakpointTypeCategory.java | 18 ---------- .../ui/org/eclipse/debug/ui/CommonTab.java | 38 -------------------- .../debug/ui/DebugElementWorkbenchAdapter.java | 15 -------- .../ui/org/eclipse/debug/ui/DebugPopup.java | 15 -------- .../ui/DeferredDebugElementWorkbenchAdapter.java | 9 ----- .../ui/org/eclipse/debug/ui/RefreshTab.java | 6 ---- .../debug/ui/StringVariableSelectionDialog.java | 12 ------- .../eclipse/debug/ui/WorkingDirectoryBlock.java | 15 -------- 10 files changed, 8 insertions(+), 164 deletions(-) (limited to 'org.eclipse.debug.ui') diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractBreakpointOrganizerDelegate.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractBreakpointOrganizerDelegate.java index a95bc2f34..2d24acb8a 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractBreakpointOrganizerDelegate.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractBreakpointOrganizerDelegate.java @@ -35,65 +35,44 @@ public abstract class AbstractBreakpointOrganizerDelegate implements IBreakpoint // property change listeners private ListenerList fListeners = new ListenerList<>(); - /* (non-Javadoc) - * @see org.eclipse.debug.ui.IBreakpointOrganizerDelegate#addBreakpoint(org.eclipse.debug.core.model.IBreakpoint, org.eclipse.core.runtime.IAdaptable) - */ @Override public void addBreakpoint(IBreakpoint breakpoint, IAdaptable category) { // do noting, not supported by default } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.IBreakpointOrganizerDelegate#addPropertyChangeListener(org.eclipse.jface.util.IPropertyChangeListener) - */ @Override public void addPropertyChangeListener(IPropertyChangeListener listener) { fListeners.add(listener); } - /* (non-Javadoc) - * - * Subclasses that override should return super.canAdd(...) when they are not able to add - * the breakpoint. - * - * @see org.eclipse.debug.ui.IBreakpointOrganizerDelegate#canAdd(org.eclipse.debug.core.model.IBreakpoint, org.eclipse.core.runtime.IAdaptable) - */ + /* + * Subclasses that override should return super.canAdd(...) when they are not + * able to add the breakpoint. + */ @Override public boolean canAdd(IBreakpoint breakpoint, IAdaptable category) { return category instanceof OtherBreakpointCategory; } - /* (non-Javadoc) - * - * Subclasses that override should return super.canRemove(...) when they are not able to remove - * the breakpoint. - * - * @see org.eclipse.debug.ui.IBreakpointOrganizerDelegate#canRemove(org.eclipse.debug.core.model.IBreakpoint, org.eclipse.core.runtime.IAdaptable) - */ + /* + * Subclasses that override should return super.canRemove(...) when they are not + * able to remove the breakpoint. + */ @Override public boolean canRemove(IBreakpoint breakpoint, IAdaptable category) { return category instanceof OtherBreakpointCategory; } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.IBreakpointOrganizerDelegate#dispose() - */ @Override public void dispose() { fListeners = new ListenerList<>(); } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.IBreakpointOrganizerDelegate#removeBreakpoint(org.eclipse.debug.core.model.IBreakpoint, org.eclipse.core.runtime.IAdaptable) - */ @Override public void removeBreakpoint(IBreakpoint breakpoint, IAdaptable category) { // do nothing, not supported by default } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.IBreakpointOrganizerDelegate#removePropertyChangeListener(org.eclipse.jface.util.IPropertyChangeListener) - */ @Override public void removePropertyChangeListener(IPropertyChangeListener listener) { fListeners.remove(listener); @@ -125,9 +104,6 @@ public abstract class AbstractBreakpointOrganizerDelegate implements IBreakpoint } } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.IBreakpointOrganizerDelegate#getCategories() - */ @Override public IAdaptable[] getCategories() { return null; diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractLaunchConfigurationTab.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractLaunchConfigurationTab.java index 51108bed3..2f5e39f73 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractLaunchConfigurationTab.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/AbstractLaunchConfigurationTab.java @@ -124,10 +124,6 @@ public abstract class AbstractLaunchConfigurationTab implements ILaunchConfigura protected void initializeAttributes() { } - /* - * (non-Javadoc) - * @see org.eclipse.debug.core.IPrototypeAttributesLabelProvider#getLabel(java.lang.String) - */ @Override public String getAttributeLabel(String attribute) { String label = fAttributesLabelsForPrototype.get(attribute); diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/BreakpointTypeCategory.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/BreakpointTypeCategory.java index 1a6c2a5e5..7a5199cd5 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/BreakpointTypeCategory.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/BreakpointTypeCategory.java @@ -64,9 +64,6 @@ public class BreakpointTypeCategory extends PlatformObject implements IBreakpoin return fName; } - /* (non-Javadoc) - * @see java.lang.Object#equals(java.lang.Object) - */ @Override public boolean equals(Object object) { if (object instanceof BreakpointTypeCategory) { @@ -76,41 +73,26 @@ public class BreakpointTypeCategory extends PlatformObject implements IBreakpoin return false; } - /* (non-Javadoc) - * @see java.lang.Object#hashCode() - */ @Override public int hashCode() { return getName().hashCode(); } - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter#getChildren(java.lang.Object) - */ @Override public Object[] getChildren(Object o) { return null; } - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter#getImageDescriptor(java.lang.Object) - */ @Override public ImageDescriptor getImageDescriptor(Object object) { return fImageDescriptor; } - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter#getLabel(java.lang.Object) - */ @Override public String getLabel(Object o) { return getName(); } - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter#getParent(java.lang.Object) - */ @Override public Object getParent(Object o) { return null; diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/CommonTab.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/CommonTab.java index 788b153a1..559197f64 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/CommonTab.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/CommonTab.java @@ -168,10 +168,6 @@ public class CommonTab extends AbstractLaunchConfigurationTab { setHelpContextId(IDebugHelpContextIds.LAUNCH_CONFIGURATION_DIALOG_COMMON_TAB); } - - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#createControl(org.eclipse.swt.widgets.Composite) - */ @Override public void createControl(Composite parent) { Composite comp = new Composite(parent, SWT.NONE); @@ -543,11 +539,6 @@ public class CommonTab extends AbstractLaunchConfigurationTab { fDefaultEncodingButton.addSelectionListener(listener); fEncodingCombo.addSelectionListener(listener); fEncodingCombo.addKeyListener(new KeyAdapter() { - /* - * (non-Javadoc) - * - * @see org.eclipse.swt.events.KeyListener#keyReleased(org.eclipse.swt.events.KeyEvent) - */ @Override public void keyReleased(KeyEvent e) { scheduleUpdateJob(); @@ -691,9 +682,6 @@ public class CommonTab extends AbstractLaunchConfigurationTab { return (IContainer) getWorkspaceRoot().findMember(containerPath); } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#initializeFrom(org.eclipse.debug.core.ILaunchConfiguration) - */ @Override public void initializeFrom(ILaunchConfiguration configuration) { boolean isShared = !configuration.isLocal(); @@ -929,9 +917,6 @@ public class CommonTab extends AbstractLaunchConfigurationTab { return ResourcesPlugin.getWorkspace().getRoot(); } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#isValid(org.eclipse.debug.core.ILaunchConfiguration) - */ @Override public boolean isValid(ILaunchConfiguration config) { setMessage(null); @@ -1006,18 +991,12 @@ public class CommonTab extends AbstractLaunchConfigurationTab { return true; } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#setDefaults(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy) - */ @Override public void setDefaults(ILaunchConfigurationWorkingCopy config) { config.setContainer(null); setAttribute(IDebugUIConstants.ATTR_LAUNCH_IN_BACKGROUND, config, true, true); } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#performApply(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy) - */ @Override public void performApply(ILaunchConfigurationWorkingCopy configuration) { updateConfigFromLocalShared(configuration); @@ -1060,17 +1039,12 @@ public class CommonTab extends AbstractLaunchConfigurationTab { } } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#getName() - */ @Override public String getName() { return LaunchConfigurationsMessages.CommonTab__Common_15; } /** - * @see org.eclipse.debug.ui.AbstractLaunchConfigurationTab#getId() - * * @since 3.3 */ @Override @@ -1078,31 +1052,19 @@ public class CommonTab extends AbstractLaunchConfigurationTab { return "org.eclipse.debug.ui.commonTab"; //$NON-NLS-1$ } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#canSave() - */ @Override public boolean canSave() { return validateLocalShared(); } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#getImage() - */ @Override public Image getImage() { return DebugUITools.getImage(IInternalDebugUIConstants.IMG_OBJS_COMMON_TAB); } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#activated(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy) - */ @Override public void activated(ILaunchConfigurationWorkingCopy workingCopy) {} - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#deactivated(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy) - */ @Override public void deactivated(ILaunchConfigurationWorkingCopy workingCopy) {} diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DebugElementWorkbenchAdapter.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DebugElementWorkbenchAdapter.java index 3480dc483..ae1c7aa60 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DebugElementWorkbenchAdapter.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DebugElementWorkbenchAdapter.java @@ -37,41 +37,26 @@ import org.eclipse.ui.model.IWorkbenchAdapter2; @Deprecated public abstract class DebugElementWorkbenchAdapter implements IWorkbenchAdapter, IWorkbenchAdapter2 { - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter#getImageDescriptor(java.lang.Object) - */ @Override public ImageDescriptor getImageDescriptor(Object object) { return DebugElementHelper.getImageDescriptor(object); } - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter#getLabel(java.lang.Object) - */ @Override public String getLabel(Object o) { return DebugElementHelper.getLabel(o); } - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter2#getForeground(java.lang.Object) - */ @Override public RGB getForeground(Object element) { return DebugElementHelper.getForeground(element); } - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter2#getBackground(java.lang.Object) - */ @Override public RGB getBackground(Object element) { return DebugElementHelper.getBackground(element); } - /* (non-Javadoc) - * @see org.eclipse.ui.model.IWorkbenchAdapter2#getFont(java.lang.Object) - */ @Override public FontData getFont(Object element) { return DebugElementHelper.getFont(element); diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DebugPopup.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DebugPopup.java index 9c9c7ec0f..134ff6bf9 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DebugPopup.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DebugPopup.java @@ -138,10 +138,6 @@ public abstract class DebugPopup extends PopupDialog { return fPersisted; } - /* - * (non-Javadoc) - * @see org.eclipse.jface.dialogs.PopupDialog#createDialogArea(org.eclipse.swt.widgets.Composite) - */ @Override protected abstract Control createDialogArea(Composite parent); @@ -173,20 +169,12 @@ public abstract class DebugPopup extends PopupDialog { return point; } - /* - * (non-Javadoc) - * @see org.eclipse.jface.dialogs.PopupDialog#getDialogSettings() - */ @Override protected IDialogSettings getDialogSettings() { IDialogSettings settings = DebugUIPlugin.getDefault().getDialogSettings(); return settings; } - - /* (non-Javadoc) - * @see org.eclipse.jface.dialogs.PopupDialog#open() - */ @Override public int open() { IWorkbench workbench = PlatformUI.getWorkbench(); @@ -213,9 +201,6 @@ public abstract class DebugPopup extends PopupDialog { return super.open(); } - /* (non-Javadoc) - * @see org.eclipse.jface.dialogs.PopupDialog#close() - */ @Override public boolean close() { if (fActivation != null) { diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DeferredDebugElementWorkbenchAdapter.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DeferredDebugElementWorkbenchAdapter.java index f718eb802..fdf8b4422 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DeferredDebugElementWorkbenchAdapter.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/DeferredDebugElementWorkbenchAdapter.java @@ -41,25 +41,16 @@ public abstract class DeferredDebugElementWorkbenchAdapter extends DebugElementW */ protected static final Object[] EMPTY = new Object[0]; - /* (non-Javadoc) - * @see org.eclipse.ui.progress.IDeferredWorkbenchAdapter#isContainer() - */ @Override public boolean isContainer() { return true; } - /* (non-Javadoc) - * @see org.eclipse.ui.progress.IDeferredWorkbenchAdapter#getRule(java.lang.Object) - */ @Override public ISchedulingRule getRule(Object object) { return null; } - /* (non-Javadoc) - * @see org.eclipse.ui.progress.IDeferredWorkbenchAdapter#fetchDeferredChildren(java.lang.Object, org.eclipse.ui.progress.IElementCollector, org.eclipse.core.runtime.IProgressMonitor) - */ @Override public void fetchDeferredChildren(Object object, IElementCollector collector, IProgressMonitor monitor) { if (monitor.isCanceled()) { diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/RefreshTab.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/RefreshTab.java index ddec50890..a6a58d5e6 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/RefreshTab.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/RefreshTab.java @@ -488,17 +488,11 @@ public class RefreshTab extends AbstractLaunchConfigurationTab { return null; } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#activated(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy) - */ @Override public void activated(ILaunchConfigurationWorkingCopy workingCopy) { // do nothing on activation } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#deactivated(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy) - */ @Override public void deactivated(ILaunchConfigurationWorkingCopy workingCopy) { // do nothing on deactivation diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/StringVariableSelectionDialog.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/StringVariableSelectionDialog.java index e5186cfb6..2edbbcae3 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/StringVariableSelectionDialog.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/StringVariableSelectionDialog.java @@ -199,9 +199,6 @@ public class StringVariableSelectionDialog extends ElementListSelectionDialog { super.setListElements(filtered.toArray()); } - /* (non-Javadoc) - * @see org.eclipse.jface.dialogs.Dialog#createContents(org.eclipse.swt.widgets.Composite) - */ @Override protected Control createContents(Composite parent) { Control ctrl = super.createContents(parent); @@ -209,9 +206,6 @@ public class StringVariableSelectionDialog extends ElementListSelectionDialog { return ctrl; } - /* (non-Javadoc) - * @see org.eclipse.jface.dialogs.Dialog#createDialogArea(org.eclipse.swt.widgets.Composite) - */ @Override protected Control createDialogArea(Composite parent) { Control control = super.createDialogArea(parent); @@ -376,9 +370,6 @@ public class StringVariableSelectionDialog extends ElementListSelectionDialog { fDescriptionText.setText(text); } - /* (non-Javadoc) - * @see org.eclipse.jface.dialogs.Dialog#okPressed() - */ @Override protected void okPressed() { fArgumentValue = fArgumentText.getText().trim(); @@ -394,9 +385,6 @@ public class StringVariableSelectionDialog extends ElementListSelectionDialog { return IDebugUIConstants.PLUGIN_ID + ".STRING_VARIABLE_SELECTION_DIALOG_SECTION"; //$NON-NLS-1$ } - /* (non-Javadoc) - * @see org.eclipse.jface.dialogs.Dialog#getDialogBoundsSettings() - */ @Override protected IDialogSettings getDialogBoundsSettings() { IDialogSettings settings = DebugUIPlugin.getDefault().getDialogSettings(); diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/WorkingDirectoryBlock.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/WorkingDirectoryBlock.java index 9f300d5a9..0ec29a56b 100644 --- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/WorkingDirectoryBlock.java +++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/WorkingDirectoryBlock.java @@ -150,9 +150,6 @@ public abstract class WorkingDirectoryBlock extends AbstractLaunchConfigurationT this.helpContextId = helpContextId; } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#createControl(org.eclipse.swt.widgets.Composite) - */ @Override public final void createControl(Composite parent) { Font font = parent.getFont(); @@ -327,9 +324,6 @@ public abstract class WorkingDirectoryBlock extends AbstractLaunchConfigurationT */ protected abstract IProject getProject(ILaunchConfiguration configuration) throws CoreException; - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#isValid(org.eclipse.debug.core.ILaunchConfiguration) - */ @Override public boolean isValid(ILaunchConfiguration config) { setErrorMessage(null); @@ -374,9 +368,6 @@ public abstract class WorkingDirectoryBlock extends AbstractLaunchConfigurationT config.setAttribute(workingDirectoryAttribteName, (String)null); } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#initializeFrom(org.eclipse.debug.core.ILaunchConfiguration) - */ @Override public void initializeFrom(ILaunchConfiguration configuration) { setLaunchConfiguration(configuration); @@ -392,9 +383,6 @@ public abstract class WorkingDirectoryBlock extends AbstractLaunchConfigurationT } } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#performApply(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy) - */ @Override public void performApply(ILaunchConfigurationWorkingCopy configuration) { if(fUseDefaultDirButton.getSelection()) { @@ -405,9 +393,6 @@ public abstract class WorkingDirectoryBlock extends AbstractLaunchConfigurationT } } - /* (non-Javadoc) - * @see org.eclipse.debug.ui.ILaunchConfigurationTab#getName() - */ @Override public String getName() { return DebugUIMessages.WorkingDirectoryBlock_Working_Directory_8; -- cgit v1.2.3