Changeset 10363 for code/branches/core7/src/libraries/core/class/Identifier.cc

Timestamp:

Apr 13, 2015, 11:15:04 PM (9 years ago)

Author:

landauf

Message:

bugfix: class hierarchy wrongly considered the classes of nested members as base classes of the surrounding class. this is now fixed by using RTTI (i.e. dynamic_cast). also added some tests.

File:

• code/branches/core7/src/libraries/core/class/Identifier.cc (modified) (2 diffs)

code/branches/core7/src/libraries/core/class/Identifier.cc

@@ -144 +144 @@
     /**
      * @brief Initializes the parents of this Identifier while creating the class hierarchy.
-     * @param identifiers All identifiers that were used to create an instance of this class (including this identifier itself)
+     * @param instance The instance that was used to determine the class hierarchy of this identifier.
+     * @param identifiers All identifiers that were used to create the instance of this class (including this identifier itself)
      */
-    void Identifier::initializeParents(const std::set<const Identifier*>& identifiers)
+    void Identifier::initializeParents(Identifiable* instance, const std::set<const Identifier*>& identifiers)
     {
         if (!IdentifierManager::getInstance().isCreatingHierarchy())
@@ -154 +155 @@
         }
 
+        // Add all identifiers which are real parents (dynamic_cast is possible) and that are not equal to THIS identifier.
+        // Checking for dynamic_cast is necessary because some classes may have other classes as nested members. This means that the Identifiers of the nested
+        // classes are also added to the set of potential parents. The only way to distinguish them is to use RTTI (i.e. dynamic_cast).
         for (std::set<const Identifier*>::const_iterator it = identifiers.begin(); it != identifiers.end(); ++it)
-            if (*it != this)
+            if (*it != this && (*it)->canDynamicCastObjectToIdentifierClass(instance))
                 this->parents_.insert(*it);
     }