Changeset 2062 for code/branches/objecthierarchy/src/network

Timestamp:

Oct 29, 2008, 5:19:02 PM (16 years ago)

Author:

scheusso

Message:

client→server synchronisation of controllable entities works now

Location:

code/branches/objecthierarchy/src/network

Files:

• Synchronisable.cc (modified) (7 diffs)
• packet/Gamestate.cc (modified) (1 diff)

code/branches/objecthierarchy/src/network/Synchronisable.cc

@@ -147 +147 @@
     synchronisableHeader *header = (synchronisableHeader *)mem;
 
+    if(!header->dataAvailable)
+    {
+      mem += header->size;
+      return 0;
+    }
+    
     COUT(4) << "fabricating object with id: " << header->objectID << std::endl;
 
@@ -314 +320 @@
       }
       // if the variable gets synchronised bidirectional, then add the reference to the bytestream
-      if( ( (*i)->mode & direction::bidirectional ) )
+      if( ( (*i)->mode & direction::bidirectional ) == direction::bidirectional )
       {
         *(uint8_t*)mem = (*i)->varReference;
         mem += sizeof( (*i)->varReference );
+        tempsize += sizeof( (*i)->varReference );
       }
       switch((*i)->type){
@@ -323 +330 @@
           memcpy( (void *)(mem), (void*)((*i)->var), (*i)->size);
           mem+=(*i)->size;
-          tempsize+=(*i)->size + sizeof( (*i)->varReference );
+          tempsize+=(*i)->size;
           break;
         case STRING:
@@ -332 +339 @@
           COUT(5) << "synchronisable: char: " << (const char *)(mem) << " data: " << data << " string: " << *(std::string *)((*i)->var) << std::endl;
           mem+=(*i)->size;
-          tempsize+=(*i)->size + sizeof( (*i)->varReference ) + sizeof(size_t);
+          tempsize+=(*i)->size + sizeof(size_t);
           break;
       }
@@ -381 +388 @@
       switch((*i)->type){
         case DATA:
-          if( ( (*i)->mode & direction::bidirectional ) )
+          if( ( (*i)->mode & direction::bidirectional ) == direction::bidirectional )
           {
             if( ( mode == 0x1 && (*i)->mode == direction::serverMaster ) || \
@@ -406 +413 @@
           break;
         case STRING:
-          if( ( (*i)->mode & direction::bidirectional ) )
+          if( ( (*i)->mode & direction::bidirectional ) == direction::bidirectional )
           {
             if( ( mode == 0x1 && (*i)->mode == direction::serverMaster ) || \
@@ -469 +476 @@
         break;
       }
-      if( ( (*i)->mode & direction::bidirectional ) != 0)
+      if( ( (*i)->mode & direction::bidirectional ) == direction::bidirectional )
       {
         tsize+=sizeof( (*i)->varReference );

code/branches/objecthierarchy/src/network/packet/Gamestate.cc

@@ -357 +357 @@
     assert(it->second->objectID==oldobjectheader->objectID);
     *newobjectheader = *oldobjectheader;
-    objectOffset=sizeof(uint8_t)+sizeof(bool); //skip the size and the availableDate variables in the objectheader
+    objectOffset=sizeof(uint8_t)+sizeof(bool); //skip the size and the availableData variables in the objectheader
     if(it->second->doSelection(HEADER->id)){
       newobjectheader->dataAvailable=true; //TODO: probably not neccessary