Cleanup in ModelEditor

Remove warnings etc.

Change-Id: Ib8a627b0659006a82a80581dfd5e8ae83836fb98
Signed-off-by: Jens Lidestrom <jens@lidestrom.se>
diff --git a/bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/ModelEditor.java b/bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/ModelEditor.java
index 2a7a2d7..65dd624 100644
--- a/bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/ModelEditor.java
+++ b/bundles/org.eclipse.e4.tools.emf.ui/src/org/eclipse/e4/tools/emf/ui/internal/common/ModelEditor.java
@@ -1970,7 +1970,7 @@
 		 * @param destFeature the target feature in the model where data must be dropped
 		 * @param parent      the destination parent
 		 * @param index       the index in the parent list
-		 * @see bug #429684
+		 * @see bug 429684
 		 * @return the compound command
 		 */
 		private Command createRemoveAddCommand(Object data, EStructuralFeature destFeature, EObject parent, int index) {
@@ -2119,7 +2119,8 @@
 
 	// This will ensure the provider has created the tree node (so we can reveal
 	// it).
-	private Object getFirstMatchingItem(EObject target, ObservableListTreeContentProvider<?> provider, Object[] items) {
+	private static Object getFirstMatchingItem(EObject target, ObservableListTreeContentProvider<?> provider,
+			Object[] items) {
 		for (int i = 0; i < items.length; i++) {
 			if (items[i] == target) {
 				return items[i];