Bug 425962 - Remove (non-Javadoc) @see statements if @Override is used

Also fixes some minor cut and paste errors in the comments, mentioning
MMenuItem

Change-Id: Ia86993953e720199364e3df959f0397594baa22a
Signed-off-by: Lars Vogel <Lars.Vogel@gmail.com>

1 files changed, 4 insertions, 18 deletions

diff --git a/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/ToolBarManagerRenderer.java b/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/ToolBarManagerRenderer.java
index 9ace404e51d..cbfc1f3da56 100644
--- a/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/ToolBarManagerRenderer.java
+++ b/bundles/org.eclipse.e4.ui.workbench.renderers.swt/src/org/eclipse/e4/ui/workbench/renderers/swt/ToolBarManagerRenderer.java
@@ -100,7 +100,7 @@ public class ToolBarManagerRenderer extends SWTPartRenderer {
 	IEventBroker eventBroker;
 	private EventHandler itemUpdater = new EventHandler() {
 		public void handleEvent(Event event) {
-			// Ensure that this event is for a MMenuItem
+			// Ensure that this event is for a MToolBarElement
 			if (!(event.getProperty(UIEvents.EventTags.ELEMENT) instanceof MToolBarElement))
 				return;
 
@@ -188,7 +188,7 @@ public class ToolBarManagerRenderer extends SWTPartRenderer {
 
 	private EventHandler selectionUpdater = new EventHandler() {
 		public void handleEvent(Event event) {
-			// Ensure that this event is for a MToolItem
+			// Ensure that this event is for a MToolBarElement
 			if (!(event.getProperty(UIEvents.EventTags.ELEMENT) instanceof MToolBarElement))
 				return;
 
@@ -281,13 +281,6 @@ public class ToolBarManagerRenderer extends SWTPartRenderer {
 		eventBroker.unsubscribe(childAdditionUpdater);
 	}
 
-	/*
-	 * (non-Javadoc)
-	 * 
-	 * @see
-	 * org.eclipse.e4.ui.internal.workbench.swt.AbstractPartRenderer#createWidget
-	 * (org.eclipse.e4.ui.model.application.ui.MUIElement, java.lang.Object)
-	 */
 	@Override
 	public Object createWidget(final MUIElement element, Object parent) {
 		if (!(element instanceof MToolBar) || !(parent instanceof Composite))
@@ -322,7 +315,8 @@ public class ToolBarManagerRenderer extends SWTPartRenderer {
 	}
 
 	/**
-	 * @param element
+	 * @param toolbarModel
+	 * @param elementId
 	 */
 	public void processContribution(MToolBar toolbarModel, String elementId) {
 		final ArrayList<MToolBarContribution> toContribute = new ArrayList<MToolBarContribution>();
@@ -406,7 +400,6 @@ public class ToolBarManagerRenderer extends SWTPartRenderer {
 	private ToolBar createToolbar(final MUIElement element, Composite parent) {
 		int orientation = getOrientation(element);
 		int style = orientation | SWT.WRAP | SWT.FLAT | SWT.RIGHT;
-
 		ToolBarManager manager = getManager((MToolBar) element);
 		if (manager == null) {
 			manager = new ToolBarManager(style);
@@ -489,13 +482,6 @@ public class ToolBarManagerRenderer extends SWTPartRenderer {
 		return SWT.HORIZONTAL;
 	}
 
-	/*
-	 * (non-Javadoc)
-	 * 
-	 * @see
-	 * org.eclipse.e4.ui.workbench.renderers.swt.SWTPartRenderer#processContents
-	 * (org.eclipse.e4.ui.model.application.ui.MElementContainer)
-	 */
 	@Override
 	public void processContents(MElementContainer<MUIElement> container) {
 		// I can either simply stop processing, or we can walk the model