Changeset 10564

Timestamp:

Aug 30, 2015, 1:36:54 PM (9 years ago)

Author:

landauf

Message:

bugfix: after receiving a ClassID packet, the client erases the list of network ids (IdentifierManager::clearNetworkIDs()). afterwards, each Identifier gets its new NetworkID. therefore it's not necessary to call removeIdentifier() - in fact, this led to errors because wrong identifiers were removed (if a another identifier received the same network id from the server already before)

Location:

code/branches/core7/src/libraries

Files:

• core/class/Identifier.cc (modified) (1 diff)
• network/synchronisable/Synchronisable.cc (modified) (1 diff)

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

@@ -126 +126 @@
     void Identifier::setNetworkID(uint32_t id)
     {
-        IdentifierManager::getInstance().removeIdentifier(this); // remove with old id
         this->networkID_ = id;
         IdentifierManager::getInstance().addIdentifier(this);    // add with new id

code/branches/core7/src/libraries/network/synchronisable/Synchronisable.cc

@@ -151 +151 @@
         for(int i = 0; i<160; i++)
             orxout(user_error, context::network) << "classid: " << i << " identifier: " << ClassByID(i) << endl;
-        orxout(user_error, context::network) << "Assertion failed: id" << endl;
+        orxout(user_error, context::network) << "Assertion failed: Could not find Identifier for ClassID " << header.getClassID() << endl;
         orxout(user_error, context::network) << "Possible reason for this error: Client received a synchronizable object whose class has no factory." << endl;
         abort();