RED

refs 1180

org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/utils/ModelUtils.java

@@ -61,14 +61,19 @@ public class ModelUtils {
 	 *             if the context is not contained in a persisted model.
 	 */
 	public static void prepareUniqueID(EObject element, EObject context) {
+		// FIXME: why not use org.fortiss.tooling.kernel.utils.UniqueIDUtils.prepareIDs(EObject,
+		// EObject, int)
 		if(element instanceof IIdLabeled) {
 			ITopLevelElement top = IPersistencyService.INSTANCE.getTopLevelElementFor(context);
 
 			if(top == null) {
 				EObject root = context;
-				while(root.eContainer() != null)
+				while(root.eContainer() != null) {
 					root = root.eContainer();
+				}
 
+				// FIXME: why not start with 0
+				// see also org.fortiss.tooling.kernel.internal.storage.eclipse.ModelContext:129
 				int id = Integer.MIN_VALUE;
 				for(TreeIterator<EObject> iter = root.eAllContents(); iter.hasNext();) {
 					EObject o = iter.next();