From a4aebd75dfd9376b9924442ef92708a53c5b8007 Mon Sep 17 00:00:00 2001
From: Vincent Aravantinos <aravantinos@fortiss.org>
Date: Thu, 24 Apr 2014 11:36:35 +0000
Subject: [PATCH] Fixed comment refs 1943

---
 .../internal/storage/eclipse/ModelContext.java      | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/storage/eclipse/ModelContext.java b/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/storage/eclipse/ModelContext.java
index 8c02c6250..6aea0e5ad 100644
--- a/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/storage/eclipse/ModelContext.java
+++ b/org.fortiss.tooling.kernel/trunk/src/org/fortiss/tooling/kernel/internal/storage/eclipse/ModelContext.java
@@ -58,7 +58,7 @@ import org.fortiss.tooling.kernel.utils.UniqueIDUtils;
  * @author hummel
  * @author $Author$
  * @version $Rev$
- * @ConQAT.Rating YELLOW Hash: A7B00E66E3893E14F4097D435147ACFC
+ * @ConQAT.Rating YELLOW Hash: 1E09DC71D8E0EBDD16B88FD1C86085D6
  */
 class ModelContext implements ITopLevelElement, CommandStackListener {
 
@@ -87,10 +87,13 @@ class ModelContext implements ITopLevelElement, CommandStackListener {
 	private int maxId = -1;
 
 	/**
-	 * TODO (dan): I do not understand this comment
-	 * The map containing the unknown features. Can be non-empty, in which case the migrators have
-	 * to deal with it.
-	 * Only if they don't the model context is buggy.
+	 * There might be mismatches between the meta-model of a file and the meta-model of AF3
+	 * (typically when using a file saved with an older version of AF3). In this case, the parser
+	 * will parse the file (or fail if the file is really too different from what is expected), but
+	 * it will also return the XML nodes ("features") that it failed to parse. These are stored in a
+	 * map "EObject to AnyType": EObject gives the context in which the unknown feature has been met
+	 * and AnyType provides the actual unknown data. If non-empty, the migrators have to deal with
+	 * it. If they don't the model context is considered buggy.
 	 */
 	private Map<EObject, AnyType> unknownFeatures;
 
-- 
GitLab