diff --git a/bundles/org.eclipse.e4.ui.progress/src/org/eclipse/e4/ui/progress/internal/ProgressServiceImpl.java b/bundles/org.eclipse.e4.ui.progress/src/org/eclipse/e4/ui/progress/internal/ProgressServiceImpl.java
index 5feb3ff569b..ee3e456d967 100644
--- a/bundles/org.eclipse.e4.ui.progress/src/org/eclipse/e4/ui/progress/internal/ProgressServiceImpl.java
+++ b/bundles/org.eclipse.e4.ui.progress/src/org/eclipse/e4/ui/progress/internal/ProgressServiceImpl.java
@@ -302,7 +302,7 @@ private void setUserInterfaceActive(boolean active) {
 				}
 			}
 		} else {
-			// Deactive shells in reverse order
+			// Deactivate shells in reverse order
 			for (int i = shells.length - 1; i >= 0; i--) {
 				if (!shells[i].isDisposed()) {
 					shells[i].setEnabled(active);
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/ContentViewer.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/ContentViewer.java
index 50b229818d0..e4b77e1ceb6 100644
--- a/bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/ContentViewer.java
+++ b/bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/ContentViewer.java
@@ -256,7 +256,7 @@ public void setContentProvider(IContentProvider contentProvider) {
 	 * The <code>ContentViewer</code> implementation of this <code>Viewer</code>
 	 * method invokes <code>inputChanged</code> on the content provider and then the
 	 * <code>inputChanged</code> hook method. This method fails if this viewer does
-	 * not have a content provider. Subclassers are advised to override
+	 * not have a content provider. Subclasses are advised to override
 	 * <code>inputChanged</code> rather than this method, but may extend this method
 	 * if required.
 	 */
diff --git a/bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/StructuredViewer.java b/bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/StructuredViewer.java
index 153ca92b2bb..5b1ab4432d3 100644
--- a/bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/StructuredViewer.java
+++ b/bundles/org.eclipse.jface/src/org/eclipse/jface/viewers/StructuredViewer.java
@@ -1852,8 +1852,8 @@ public Widget[] testFindItems(Object element) {
 	/**
 	 * Removes all elements from the map.
 	 * <p>
-	 * This method is internal to the framework; subclassers should not call
-	 * this method.
+	 * This method is internal to the framework; subclasses should not call this
+	 * method.
 	 * </p>
 	 */
 	protected void unmapAllElements() {
@@ -1881,17 +1881,16 @@ protected void unmapElement(Object element) {
 	}
 
 	/**
-	 * Removes the given association from the internal element to widget map.
-	 * Does nothing if mapping is disabled, or if the given element does not map
-	 * to the given item.
+	 * Removes the given association from the internal element to widget map. Does
+	 * nothing if mapping is disabled, or if the given element does not map to the
+	 * given item.
 	 * <p>
-	 * This method is internal to the framework; subclassers should not call
-	 * this method.
+	 * This method is internal to the framework; subclasses should not call this
+	 * method.
 	 * </p>
 	 *
-	 * @param element
-	 *            the element
-	 * @param item the item to unmap
+	 * @param element the element
+	 * @param item    the item to unmap
 	 * @since 2.0
 	 */
 	protected void unmapElement(Object element, Widget item) {
@@ -2101,14 +2100,12 @@ protected void internalUpdate(Widget widget, Object element, String[] properties
 	/**
 	 * Copies attributes of the given element into the given widget.
 	 * <p>
-	 * This method is internal to the framework; subclassers should not call
-	 * this method. Calls <code>doUpdateItem(widget, element, true)</code>.
+	 * This method is internal to the framework; subclasses should not call this
+	 * method. Calls <code>doUpdateItem(widget, element, true)</code>.
 	 * </p>
 	 *
-	 * @param widget
-	 *            the widget
-	 * @param element
-	 *            the element
+	 * @param widget  the widget
+	 * @param element the element
 	 */
 	protected final void updateItem(Widget widget, Object element) {
 		SafeRunnable.run(new UpdateItemSafeRunnable(widget, element, true));
diff --git a/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/IWorkbenchPartReference.java b/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/IWorkbenchPartReference.java
index aeb5f5cef5a..b44f7fe059c 100644
--- a/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/IWorkbenchPartReference.java
+++ b/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/IWorkbenchPartReference.java
@@ -68,7 +68,7 @@ public interface IWorkbenchPartReference {
 	void addPropertyListener(IPropertyListener listener);
 
 	/**
-	 * @param listener the poperty listener to remove
+	 * @param listener the property listener to remove
 	 * @see IWorkbenchPart#removePropertyListener
 	 */
 	void removePropertyListener(IPropertyListener listener);
diff --git a/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/EditorMenuManager.java b/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/EditorMenuManager.java
index c2b9bcae447..f276973661e 100644
--- a/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/EditorMenuManager.java
+++ b/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/EditorMenuManager.java
@@ -65,7 +65,7 @@ public Integer getAccelerator(IContributionItem item) {
 			if (getEnabled(item) == null) {
 				return getParentMenuManager().getOverrides().getAccelerator(item);
 			}
-			// no acclerator if the item is disabled
+			// no accelerator if the item is disabled
 			return Integer.valueOf(0);
 		}
 
diff --git a/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/WorkbenchWindow.java b/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/WorkbenchWindow.java
index 3a87c44d459..48a5c0d11fe 100644
--- a/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/WorkbenchWindow.java
+++ b/bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/WorkbenchWindow.java
@@ -1797,7 +1797,7 @@ public void runWithException() throws Throwable {
 	 * @since 3.1
 	 */
 	private void fireWindowClosed() {
-		// let the application do further deconfiguration
+		// let the application do further reconfiguration
 		getWindowAdvisor().postWindowClose();
 		getWorkbenchImpl().fireWindowClosed(this);
 	}
@@ -2034,7 +2034,7 @@ private boolean hardClose(boolean remove) {
 				hideNonRestorableViews();
 			}
 
-			// clear some lables
+			// clear some labels
 			// Remove the handler submissions. Bug 64024.
 			final IWorkbench workbench = getWorkbench();
 			LegacyHandlerService windowHs = (LegacyHandlerService) model.getContext().get(IHandlerService.class);
@@ -2547,7 +2547,7 @@ public final void largeUpdateEnd() {
 
 	/**
 	 * Update the visible action sets. This method is typically called from a page
-	 * when the user changes the visible action sets within the prespective.
+	 * when the user changes the visible action sets within the perspective.
 	 */
 	public void updateActionSets() {
 		if (updateDisabled) {