Bug 508144 - Add @Override to org.eclipse.pde.ua.ui (XI.)
added @override (removed non-javadoc if needed)
cleaned empty or effectively empty comments
Change-Id: I43b41a41c5f25f83123eba9cd95f0412a4ca482b
Signed-off-by: Martin Karpisek <martin.karpisek@gmail.com>
diff --git a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/PDEUserAssistanceLabelProvider.java b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/PDEUserAssistanceLabelProvider.java
index 575df57..eb742b0 100644
--- a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/PDEUserAssistanceLabelProvider.java
+++ b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/PDEUserAssistanceLabelProvider.java
@@ -26,6 +26,7 @@
public PDEUserAssistanceLabelProvider() {
}
+ @Override
public String getText(Object obj) {
if (obj instanceof ISimpleCSObject) {
return getObjectText((ISimpleCSObject) obj);
@@ -67,9 +68,6 @@
return PDETextHelper.truncateAndTrailOffText(PDETextHelper.translateReadText(obj.getName()), limit);
}
- /**
- * @param obj
- */
public String getObjectText(TocObject obj) {
return PDETextHelper.translateReadText(obj.getName());
}
@@ -78,6 +76,7 @@
return PDETextHelper.translateReadText(obj.getName());
}
+ @Override
public Image getImage(Object obj) {
if (obj instanceof ISimpleCSObject) {
return getObjectImage((ISimpleCSObject) obj);
@@ -114,9 +113,6 @@
return get(PDEUserAssistanceUIPluginImages.DESC_SIMPLECS_OBJ, F_ERROR);
}
- /**
- * @param object
- */
private Image getObjectImage(ICompCSObject object) {
if (object.getType() == ICompCSConstants.TYPE_TASK) {
@@ -129,9 +125,6 @@
return get(PDEUserAssistanceUIPluginImages.DESC_SIMPLECS_OBJ, F_ERROR);
}
- /**
- * @param object
- */
private Image getObjectImage(TocObject object) {
switch (object.getType()) {
case ITocConstants.TYPE_TOC : {
@@ -155,9 +148,6 @@
}
}
- /**
- * @param object
- */
private Image getObjectImage(CtxHelpObject object) {
switch (object.getType()) {
case ICtxHelpConstants.TYPE_ROOT : {
diff --git a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/PDEUserAssistanceUIPlugin.java b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/PDEUserAssistanceUIPlugin.java
index 6c61c2b..95b7da7 100644
--- a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/PDEUserAssistanceUIPlugin.java
+++ b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/PDEUserAssistanceUIPlugin.java
@@ -43,19 +43,13 @@
public PDEUserAssistanceUIPlugin() {
}
- /*
- * (non-Javadoc)
- * @see org.eclipse.ui.plugin.AbstractUIPlugin#start(org.osgi.framework.BundleContext)
- */
+ @Override
public void start(BundleContext context) throws Exception {
super.start(context);
plugin = this;
}
- /*
- * (non-Javadoc)
- * @see org.eclipse.ui.plugin.AbstractUIPlugin#stop(org.osgi.framework.BundleContext)
- */
+ @Override
public void stop(BundleContext context) throws Exception {
plugin = null;
super.stop(context);
@@ -103,6 +97,7 @@
: Display.getDefault();
final IStatus fstatus = status;
display.asyncExec(new Runnable() {
+ @Override
public void run() {
ErrorDialog.openError(null, title, null, fstatus);
}
diff --git a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpOperation.java b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpOperation.java
index a887c94..a41b722 100644
--- a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpOperation.java
+++ b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpOperation.java
@@ -45,9 +45,7 @@
fFile = file;
}
- /* (non-Javadoc)
- * @see org.eclipse.ui.actions.WorkspaceModifyOperation#execute(org.eclipse.core.runtime.IProgressMonitor)
- */
+ @Override
protected void execute(IProgressMonitor monitor) throws CoreException, InvocationTargetException, InterruptedException {
CtxHelpModel model = new CtxHelpModel(CoreUtility.getTextDocument(fFile.getContents()), false);
model.setUnderlyingResource(fFile);
@@ -76,6 +74,7 @@
*/
protected void openFile() {
Display.getCurrent().asyncExec(new Runnable() {
+ @Override
public void run() {
IWorkbenchWindow ww = PDEUserAssistanceUIPlugin.getActiveWorkbenchWindow();
if (ww == null) {
diff --git a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpWizard.java b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpWizard.java
index d2bdbf6..0d6da8c 100644
--- a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpWizard.java
+++ b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpWizard.java
@@ -27,33 +27,25 @@
protected NewCtxHelpWizardPage fMainPage;
- /* (non-Javadoc)
- * @see org.eclipse.ui.wizards.newresource.BasicNewFileResourceWizard#addPages()
- */
+ @Override
public void addPages() {
fMainPage = new NewCtxHelpWizardPage("new context help", getSelection()); //$NON-NLS-1$
addPage(fMainPage);
}
- /* (non-Javadoc)
- * @see org.eclipse.ui.wizards.newresource.BasicNewFileResourceWizard#init(org.eclipse.ui.IWorkbench, org.eclipse.jface.viewers.IStructuredSelection)
- */
+ @Override
public void init(IWorkbench workbench, IStructuredSelection currentSelection) {
super.init(workbench, currentSelection);
setWindowTitle(CtxWizardMessages.NewCtxHelpWizard_title);
setNeedsProgressMonitor(true);
}
- /* (non-Javadoc)
- * @see org.eclipse.ui.wizards.newresource.BasicNewFileResourceWizard#initializeDefaultPageImageDescriptor()
- */
+ @Override
protected void initializeDefaultPageImageDescriptor() {
// setDefaultPageImageDescriptor(PDEUserAssistanceUIPluginImages.DESC_TARGET_WIZ);
}
- /* (non-Javadoc)
- * @see org.eclipse.ui.wizards.newresource.BasicNewFileResourceWizard#performFinish()
- */
+ @Override
public boolean performFinish() {
try {
getContainer().run(false, true, getOperation());
diff --git a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpWizardPage.java b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpWizardPage.java
index 5b1f0c6..9916db1 100644
--- a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpWizardPage.java
+++ b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/NewCtxHelpWizardPage.java
@@ -35,10 +35,12 @@
setFileExtension(EXTENSION);
}
+ @Override
protected void createAdvancedControls(Composite parent) {
// We don't want any advanced controls showing up
}
+ @Override
public void createControl(Composite parent) {
super.createControl(parent);
diff --git a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/RegisterCtxHelpOperation.java b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/RegisterCtxHelpOperation.java
index 6ff37e3..267f9b4 100644
--- a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/RegisterCtxHelpOperation.java
+++ b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/RegisterCtxHelpOperation.java
@@ -77,13 +77,7 @@
fShell = shell;
}
- /*
- * (non-Javadoc)
- *
- * @see
- * org.eclipse.ui.actions.WorkspaceModifyOperation#execute(org.eclipse.core
- * .runtime.IProgressMonitor)
- */
+ @Override
protected void execute(IProgressMonitor monitor) throws CoreException,
InvocationTargetException, InterruptedException {
try {
@@ -116,6 +110,7 @@
}
// Perform the modification of the plugin manifest file
ModelModification mod = new ModelModification(fProject) {
+ @Override
protected void modifyModel(IBaseModel model,
IProgressMonitor monitor) throws CoreException {
doModifyPluginModel(model, monitor);
@@ -228,6 +223,7 @@
}
// Perform the modification of the manifest file
ModelModification mod = new ModelModification(fProject) {
+ @Override
protected void modifyModel(IBaseModel model,
IProgressMonitor monitor) throws CoreException {
doModifyManifestModel(model);
diff --git a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/RegisterCtxHelpWizard.java b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/RegisterCtxHelpWizard.java
index c753d7c..9ca4587 100644
--- a/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/RegisterCtxHelpWizard.java
+++ b/ua/org.eclipse.pde.ua.ui/src/org/eclipse/pde/internal/ua/ui/wizards/ctxhelp/RegisterCtxHelpWizard.java
@@ -43,17 +43,13 @@
setNeedsProgressMonitor(true);
}
- /* (non-Javadoc)
- * @see org.eclipse.jface.wizard.Wizard#addPages()
- */
+ @Override
public void addPages() {
fMainPage = new RegisterCtxHelpWizardPage(CtxWizardMessages.RegisterCtxHelpWizard_title);
addPage(fMainPage);
}
- /* (non-Javadoc)
- * @see org.eclipse.jface.wizard.Wizard#performFinish()
- */
+ @Override
public boolean performFinish() {
try {
getContainer().run(false, true, new RegisterCtxHelpOperation(getShell(), fWizModel, fMainPage.getPluginText()));
@@ -80,6 +76,7 @@
setMessage(CtxWizardMessages.RegisterCtxHelpWizard_pageMessage);
}
+ @Override
public void createControl(Composite parent) {
Composite composite = new Composite(parent, SWT.NONE);
composite.setLayout(new GridLayout(2, false));
