Planet
navi homePPSaboutscreenshotsdownloaddevelopmentforum

Ignore:
Timestamp:
Jan 17, 2010, 11:49:48 AM (14 years ago)
Author:
scheusso
Message:

changed diff behaviour in order to reduce datasize before and after compress
this reduces time needed for gamestate diff and compress about 50%

File:
1 edited

Legend:

Unmodified
Added
Removed

  • code/branches/network2/src/libraries/network/packet/Gamestate.cc

    r6417 r6449  
    4545#define PACKET_FLAG_GAMESTATE  PacketFlag::Reliable
    4646
    47 
    48 Gamestate::Gamestate()
     47inline bool memzero( uint8_t* data, uint32_t datalength)
     48{
     49  uint64_t* d = (uint64_t*)data;
     50
     51  for( unsigned int i=0; i<datalength/8; i++ )
     52  {
     53    if( *(d+i) != 0 )
     54      return false;
     55  }
     56  // now process the rest (when datalength isn't a multiple of 4)
     57  for( unsigned int j = 8*(datalength/8); j<datalength; j++ )
     58  {
     59    if( *(data+j) != 0 )
     60      return false;
     61  }
     62  return true;
     63}
     64
     65
     66Gamestate::Gamestate():
     67  header_(0)
    4968{
    5069  flags_ = flags_ | PACKET_FLAG_GAMESTATE;
    51   header_ = 0;
    52 }
     70}
     71
    5372
    5473Gamestate::Gamestate(uint8_t *data, unsigned int clientID):
    55     Packet(data, clientID)
     74  Packet(data, clientID)
    5675{
    5776  flags_ = flags_ | PACKET_FLAG_GAMESTATE;
    … …  
    5978}
    6079
     80
    6181Gamestate::Gamestate(uint8_t *data)
    6282{
    6383  flags_ = flags_ | PACKET_FLAG_GAMESTATE;
    64   data_=data;
     84  data_ = data;
    6585  header_ = new GamestateHeader(data_);
    6686}
    6787
     88
    6889Gamestate::Gamestate(const Gamestate& g) :
    69     Packet( *(Packet*)&g )
     90    Packet( *(Packet*)&g ), nrOfVariables_(0)
    7091{
    7192  flags_ = flags_ | PACKET_FLAG_GAMESTATE;
    7293  header_ = new GamestateHeader(data_);
     94  sizes_ = g.sizes_;
    7395}
    7496
    … …  
    79101    delete header_;
    80102}
     103
    81104
    82105bool Gamestate::collectData(int id, uint8_t mode)
    … …  
    91114    return false;
    92115  data_ = new uint8_t[size + GamestateHeader::getSize()];
    93   if(!data_){
     116  if(!data_)
     117  {
    94118    COUT(2) << "GameStateManager: could not allocate memory" << std::endl;
    95119    return false;
    … …  
    101125
    102126  //start collect data synchronisable by synchronisable
    103   uint8_t *mem=data_;
     127  uint8_t *mem = data_; // in this stream store all data of the variables and the headers of the synchronisable
    104128  mem += GamestateHeader::getSize();
    105129  ObjectList<Synchronisable>::iterator it;
    106   for(it = ObjectList<Synchronisable>::begin(); it; ++it){
     130  for(it = ObjectList<Synchronisable>::begin(); it; ++it)
     131  {
    107132
    108133//     tempsize=it->getSize(id, mode);
    109134
    110     tempsize = it->getData(mem, id, mode);
     135    tempsize = it->getData(mem, this->sizes_, id, mode);
    111136    if ( tempsize != 0 )
    112137      dataVector_.push_back( obj(it->getObjectID(), it->getCreatorID(), tempsize, mem-data_) );
    113138
    114139#ifndef NDEBUG
    115     if(currentsize+tempsize > size){
     140    if(currentsize+tempsize > size)
     141    {
    116142      assert(0); // if we don't use multithreading this part shouldn't be neccessary
    117143      // start allocate additional memory
    … …  
    148174}
    149175
     176
    150177bool Gamestate::spreadData(uint8_t mode)
    151178{
    … …  
    153180  assert(data_);
    154181  assert(!header_->isCompressed());
    155   assert(!header_->isDiffed());
    156182  uint8_t *mem=data_+GamestateHeader::getSize();
     183  bool diffed = header_->isDiffed();
    157184  Synchronisable *s;
    158185
    159186  // update the data of the objects we received
    160   while(mem < data_+GamestateHeader::getSize()+header_->getDataSize()){
     187  while(mem < data_+GamestateHeader::getSize()+header_->getDataSize())
     188  {
    161189    SynchronisableHeader objectheader(mem);
    162190
    … …  
    166194      if (!GameMode::isMaster())
    167195      {
    168         Synchronisable::fabricate(mem, mode);
     196        Synchronisable::fabricate(mem, diffed, mode);
    169197      }
    170198      else
    171199      {
    172         mem += objectheader.getDataSize();
     200        mem += objectheader.getDataSize()+SynchronisableHeader::getSize();
    173201      }
    174202    }
    … …  
    181209   // In debug mode, check first, whether there are no duplicate objectIDs
    182210#ifndef NDEBUG
    183   if(this->getID()%1000==0){
     211  if(this->getID()%1000==1)
     212  {
    184213    std::list<uint32_t> v1;
    185214    ObjectList<Synchronisable>::iterator it;
    186     for (it = ObjectList<Synchronisable>::begin(); it != ObjectList<Synchronisable>::end(); ++it) {
    187       if (it->getObjectID() == OBJECTID_UNKNOWN) {
    188         if (it->objectMode_ != 0x0) {
     215    for (it = ObjectList<Synchronisable>::begin(); it != ObjectList<Synchronisable>::end(); ++it)
     216    {
     217      if (it->getObjectID() == OBJECTID_UNKNOWN)
     218      {
     219        if (it->objectMode_ != 0x0)
     220        {
    189221          COUT(0) << "Found object with OBJECTID_UNKNOWN on the client with objectMode != 0x0!" << std::endl;
    190222          COUT(0) << "Possible reason for this error: Client created a synchronized object without the Server's approval." << std::endl;
    … …  
    193225        }
    194226      }
    195       else {
     227      else
     228      {
    196229        std::list<uint32_t>::iterator it2;
    197         for (it2 = v1.begin(); it2 != v1.end(); ++it2) {
    198           if (it->getObjectID() == *it2) {
     230        for (it2 = v1.begin(); it2 != v1.end(); ++it2)
     231        {
     232          if (it->getObjectID() == *it2)
     233          {
    199234            COUT(0) << "Found duplicate objectIDs on the client!" << std::endl
    200235                    << "Are you sure you don't create a Sychnronisable objcect with 'new' \
    … …  
    211246}
    212247
     248
    213249uint32_t Gamestate::getSize() const
    214250{
    … …  
    222258}
    223259
    224 bool Gamestate::operator==(packet::Gamestate gs){
     260
     261bool Gamestate::operator==(packet::Gamestate gs)
     262{
    225263  uint8_t *d1 = data_+GamestateHeader::getSize();
    226264  uint8_t *d2 = gs.data_+GamestateHeader::getSize();
    … …  
    233271}
    234272
     273
    235274bool Gamestate::process()
    236275{
    237276  return GamestateHandler::addGamestate(this, getClientID());
    238277}
    239 
    240278
    241279
    … …  
    253291  int retval;
    254292  retval = compress( dest, &buffer, source, (uLong)(header_->getDataSize()) );
    255   switch ( retval ) {
     293  switch ( retval )
     294  {
    256295    case Z_OK: COUT(5) << "G.St.Man: compress: successfully compressed" << std::endl; break;
    257296    case Z_MEM_ERROR: COUT(1) << "G.St.Man: compress: not enough memory available in gamestate.compress" << std::endl; return false;
    … …  
    270309  header_->setCompSize( buffer );
    271310  header_->setCompressed( true );
    272   COUT(5) << "gamestate compress datasize: " << header_->getDataSize() << " compsize: " << header_->getCompSize() << std::endl;
     311  COUT(0) << "gamestate compress datasize: " << header_->getDataSize() << " compsize: " << header_->getCompSize() << std::endl;
    273312  return true;
    274313}
     314
     315
    275316bool Gamestate::decompressData()
    276317{
    … …  
    289330  uLongf length=bufsize;
    290331  retval = uncompress( dest, &length, source, (uLong)compsize );
    291   switch ( retval ) {
     332  switch ( retval )
     333  {
    292334    case Z_OK: COUT(5) << "successfully decompressed" << std::endl; break;
    293335    case Z_MEM_ERROR: COUT(1) << "not enough memory available" << std::endl; return false;
    … …  
    301343  delete temp;
    302344
    303   if (this->bDataENetAllocated_){
     345  if (this->bDataENetAllocated_)
     346  {
    304347    // Memory was allocated by ENet. --> We let it be since enet_packet_destroy will
    305348    // deallocated it anyway. So data and packet stay together.
    306349    this->bDataENetAllocated_ = false;
    307350  }
    308   else{
     351  else
     352  {
    309353    // We allocated the memory in the first place (unlikely). So we destroy the old data
    310354    // and overwrite it with the new decompressed data.
    … …  
    320364}
    321365
    322 /*Gamestate *Gamestate::diff(Gamestate *base)
    323 {
    324   assert(data_);
    325   assert(!header_->isCompressed());
    326   assert(!header_->isDiffed());
    327   GamestateHeader diffHeader(base->data_);
    328   uint8_t *basep = GAMESTATE_START(base->data_), *gs = GAMESTATE_START(this->data_);
    329   uint32_t of=0; // pointers offset
    330   uint32_t dest_length=0;
    331   dest_length=header_->getDataSize();
    332   if(dest_length==0)
    333     return NULL;
    334   uint8_t *ndata = new uint8_t[dest_length*sizeof(uint8_t)+GamestateHeader::getSize()];
    335   uint8_t *dest = ndata + GamestateHeader::getSize();
    336   while(of < diffHeader.getDataSize() && of < header_->getDataSize()){
    337     *(dest+of)=*(basep+of)^*(gs+of); // do the xor
    338     ++of;
    339   }
    340   if(diffHeader.getDataSize()!=header_->getDataSize()){
    341     uint8_t n=0;
    342     if(diffHeader.getDataSize() < header_->getDataSize()){
    343       while(of<dest_length){
    344         *(dest+of)=n^*(gs+of);
    345         of++;
    346       }
    347     }
    348   }
    349 
    350   Gamestate *g = new Gamestate(ndata, getClientID());
    351   *(g->header_) = *header_;
    352   g->header_->setDiffed( true );
    353   g->header_->setBaseID( base->getID() );
    354   g->flags_=flags_;
    355   g->packetDirection_ = packetDirection_;
    356   return g;
    357 }*/
    358366
    359367Gamestate *Gamestate::diff(Gamestate *base)
    … …  
    363371  assert(!header_->isDiffed());
    364372
    365   uint8_t *basep = GAMESTATE_START(base->data_);
    366   uint8_t *gs = GAMESTATE_START(this->data_);
    367   uint32_t dest_length = header_->getDataSize();
    368 
    369   if(dest_length==0)
    370     return NULL;
    371 
    372   uint8_t *ndata = new uint8_t[dest_length*sizeof(uint8_t)+GamestateHeader::getSize()];
    373   uint8_t *dest = GAMESTATE_START(ndata);
    374 
    375   rawDiff( dest, gs, basep, header_->getDataSize(), base->header_->getDataSize() );
    376 #ifndef NDEBUG
    377   uint8_t *dest2 = new uint8_t[dest_length];
    378   rawDiff( dest2, dest, basep, header_->getDataSize(), base->header_->getDataSize() );
    379   assert( memcmp( dest2, gs, dest_length) == 0 );
    380   delete dest2;
    381 #endif
    382 
    383   Gamestate *g = new Gamestate(ndata, getClientID());
     373
     374  // *** first do a raw diff of the two gamestates
     375
     376  uint8_t *baseData = GAMESTATE_START(base->data_);
     377  uint8_t *origData = GAMESTATE_START(this->data_);
     378  uint32_t origLength = header_->getDataSize();
     379  uint32_t baseLength = base->header_->getDataSize();
     380
     381  assert( origLength && baseLength );
     382
     383  COUT(0) << "newSize: " << origLength + GamestateHeader::getSize() + sizeof(uint32_t)*this->nrOfVariables_ << endl;
     384  uint8_t *nData = new uint8_t[origLength + GamestateHeader::getSize() + sizeof(uint32_t)*this->nrOfVariables_]; // this is the maximum size needed in the worst case
     385  uint8_t *dest = GAMESTATE_START(nData);
     386
     387  uint32_t baseOffset = 0; //offset in the diffed stream
     388  uint32_t origOffset = 0; //offset in the new stream with removed 0's
     389  std::vector<uint32_t>::iterator sizes = this->sizes_.begin();
     390
     391  while( origOffset < origLength )
     392  {
     393    //iterate through all objects
     394
     395    SynchronisableHeader h(origData+origOffset);
     396
     397    // Find (if possible) the current object in the datastream of the old gamestate
     398    // Start at the current offset position
     399    if(baseOffset >= baseLength)
     400      baseOffset = 0;
     401    uint8_t* temp = baseData + baseOffset;
     402    uint32_t objectID = h.getObjectID();
     403    assert(temp < baseData+baseLength);
     404    assert(dest < nData + origLength + GamestateHeader::getSize() + sizeof(uint32_t)*this->nrOfVariables_);
     405    assert(sizes != this->sizes_.end());
     406    while ( temp < baseData+baseLength )
     407    {
     408      SynchronisableHeader htemp(temp);
     409      if ( htemp.getObjectID() == objectID )
     410      {
     411        assert( h.getClassID() == htemp.getClassID() );
     412        goto DODIFF;
     413      }
     414      temp += htemp.getDataSize()+SynchronisableHeader::getSize();
     415    }
     416    // If not found start looking at the beginning
     417    temp = baseData;
     418    while ( temp < baseData+baseOffset )
     419    {
     420      SynchronisableHeader htemp(temp);
     421      if ( htemp.getObjectID() == objectID )
     422      {
     423        assert( h.getClassID() == htemp.getClassID() );
     424        goto DODIFF;
     425      }
     426      temp += htemp.getDataSize()+SynchronisableHeader::getSize();
     427    }
     428    // Object is new, thus never transmitted -> just copy over
     429    goto DOCOPY;
     430
     431
     432DODIFF:
     433    {
     434//       if(baseOffset==0)
     435//       {
     436//         assert(origOffset==0);
     437//       }
     438      uint32_t objectOffset = SynchronisableHeader::getSize(); // offset inside the object in the origData and baseData
     439      // Check whether the whole object stayed the same
     440      if( memcmp( origData+origOffset+objectOffset, temp+objectOffset, h.getDataSize()) == 0 )
     441      {
     442        origOffset += objectOffset+ h.getDataSize(); // skip the whole object
     443        baseOffset = temp + h.getDataSize()+SynchronisableHeader::getSize() - baseData;
     444        sizes += Synchronisable::getSynchronisable(h.getObjectID())->getNrOfVariables();
     445      }
     446      else
     447      {
     448//         COUT(4) << "diff " << h.getObjectID() << ":";
     449        // Now start to diff the Object
     450        SynchronisableHeader h2(dest);
     451        h2 = h; // copy over the objectheader
     452        uint32_t variableID = 0;
     453        uint32_t newObjectOffset = SynchronisableHeader::getSize();
     454        // iterate through all variables
     455        while( objectOffset < h.getDataSize()+SynchronisableHeader::getSize() )
     456        {
     457          // check whether variable changed and write id and copy over variable to the new stream
     458          // otherwise skip variable
     459          assert(sizes != this->sizes_.end());
     460          uint32_t varSize = *sizes;
     461          assert( varSize == Synchronisable::getSynchronisable(h.getObjectID())->getVarSize(variableID) );
     462          if ( varSize != 0 )
     463          {
     464            if ( memcmp(origData+origOffset+objectOffset, temp+objectOffset, varSize) != 0 )
     465            {
     466//               COUT(4) << " c" << varSize;
     467              *(uint32_t*)(dest+newObjectOffset) = variableID; // copy over the variableID
     468              newObjectOffset += sizeof(uint32_t);
     469              memcpy( dest+newObjectOffset, origData+origOffset+objectOffset, varSize );
     470              newObjectOffset += varSize;
     471              objectOffset += varSize;
     472            }
     473            else
     474            {
     475//               COUT(4) << " s" << varSize;
     476              objectOffset += varSize;
     477            }
     478          }
     479
     480          ++variableID;
     481          ++sizes;
     482        }
     483        if( Synchronisable::getSynchronisable(h.getObjectID())->getNrOfVariables() != variableID )
     484          sizes += Synchronisable::getSynchronisable(h.getObjectID())->getNrOfVariables() - variableID;
     485//         COUT(4) << endl;
     486        h2.setDiffed(true);
     487        h2.setDataSize(newObjectOffset-SynchronisableHeader::getSize());
     488        assert(objectOffset == h.getDataSize()+SynchronisableHeader::getSize());
     489        origOffset += objectOffset;
     490        baseOffset += temp + h.getDataSize()+SynchronisableHeader::getSize() - baseData;
     491        dest += newObjectOffset;
     492      }
     493
     494      continue;
     495    }
     496
     497DOCOPY:
     498    {
     499      // Just copy over the whole Object
     500      memcpy( dest, origData+origOffset, h.getDataSize()+SynchronisableHeader::getSize() );
     501      dest += h.getDataSize()+SynchronisableHeader::getSize();
     502      origOffset += h.getDataSize()+SynchronisableHeader::getSize();
     503      assert( Synchronisable::getSynchronisable(h.getObjectID()) );
     504//       COUT(4) << "copy " << h.getObjectID() << endl;
     505//       COUT(4) << "copy " << h.getObjectID() << ":";
     506      //sizes += Synchronisable::getSynchronisable(h.getObjectID())->getNrOfVariables();
     507      for( unsigned int i = 0; i < Synchronisable::getSynchronisable(h.getObjectID())->getNrOfVariables(); ++i )
     508      {
     509//         COUT(4) << " " << *sizes;
     510        ++sizes;
     511      }
     512//       COUT(4) << endl;
     513      assert(sizes != this->sizes_.end() || origOffset>=origLength);
     514      continue;
     515    }
     516  }
     517
     518
     519  Gamestate *g = new Gamestate(nData, getClientID());
    384520  assert(g->header_);
    385521  *(g->header_) = *header_;
    386522  g->header_->setDiffed( true );
    387523  g->header_->setBaseID( base->getID() );
     524  g->header_->setDataSize(dest - nData - GamestateHeader::getSize());
    388525  g->flags_=flags_;
    389526  g->packetDirection_ = packetDirection_;
    … …  
    393530}
    394531
    395 Gamestate *Gamestate::undiff(Gamestate *base)
    396 {
    397   assert(this && base); assert(data_ && base->data_);
    398   assert(!header_->isCompressed() && !base->header_->isCompressed());
    399   assert(header_->isDiffed());
    400 
    401   uint8_t *basep = GAMESTATE_START(base->data_);
    402   uint8_t *gs = GAMESTATE_START(this->data_);
    403   uint32_t dest_length = header_->getDataSize();
    404 
    405   if(dest_length==0)
    406     return NULL;
    407 
    408   uint8_t *ndata = new uint8_t[dest_length*sizeof(uint8_t)+GamestateHeader::getSize()];
    409   uint8_t *dest = ndata + GamestateHeader::getSize();
    410 
    411   rawDiff( dest, gs, basep, header_->getDataSize(), base->header_->getDataSize() );
    412 
    413   Gamestate *g = new Gamestate(ndata, getClientID());
    414   assert(g->header_);
    415   *(g->header_) = *header_;
    416   g->header_->setDiffed( false );
    417   g->flags_=flags_;
    418   g->packetDirection_ = packetDirection_;
    419   assert(!g->isDiffed());
    420   assert(!g->isCompressed());
    421   return g;
    422 }
    423 
    424 
    425 // Gamestate *Gamestate::diff(Gamestate *base)
    426 // {
    427 //   assert(data_);
    428 //   assert(!header_->isCompressed());
    429 //   assert(!header_->isDiffed());
    430 //   GamestateHeader diffHeader(base->data_);
    431 //   uint8_t *basep = GAMESTATE_START(base->data_), *gs = GAMESTATE_START(this->data_);
    432 //   uint32_t of=0; // pointers offset
    433 //   uint32_t dest_length=0;
    434 //   dest_length=header_->getDataSize();
    435 //   if(dest_length==0)
    436 //     return NULL;
    437 //   uint8_t *ndata = new uint8_t[dest_length*sizeof(uint8_t)+GamestateHeader::getSize()];
    438 //   uint8_t *dest = ndata + GamestateHeader::getSize();
    439 //
    440 //
    441 //   // LOOP-UNROLLED DIFFING
    442 //   uint32_t *dest32 = (uint32_t*)dest, *base32 = (uint32_t*)basep, *gs32 = (uint32_t*)gs;
    443 //   // diff in 4-byte steps
    444 //   while( of < (uint32_t)(header_->getDataSize())/4 ){
    445 //     if( of < (uint32_t)(diffHeader.getDataSize())/4 )
    446 //     {
    447 //       *(dest32+of)=*(base32+of) ^ *(gs32+of); // do the xor
    448 //       ++of;
    449 //     }
    450 //     else
    451 //     {
    452 //       *(dest32+of)=*(gs32+of); // same as 0 ^ *(gs32+of)
    453 //       ++of;
    454 //     }
    455 //   }
    456 //   for( unsigned int of2 = 0; of2 < header_->getDataSize()%4; ++of2 )
    457 //   {
    458 //     if( of*4+of2 < diffHeader.getDataSize() )
    459 //     {
    460 //       *(dest+4*of+of2)=*(basep+4*of+of2) ^ *(gs+4*of+of2); // do the xor
    461 //     }
    462 //     else
    463 //     {
    464 //       *(dest+4*of+of2)=*(gs+4*of+of2); // same as 0 ^ *(gs32+of)
    465 //     }
    466 //   }
    467 //
    468 //   Gamestate *g = new Gamestate(ndata, getClientID());
    469 //   *(g->header_) = *header_;
    470 //   g->header_->setDiffed( true );
    471 //   g->header_->setBaseID( base->getID() );
    472 //   g->flags_=flags_;
    473 //   g->packetDirection_ = packetDirection_;
    474 //   return g;
    475 // }
    476 
    477 
    478 void Gamestate::rawDiff( uint8_t* newdata, uint8_t* data, uint8_t* basedata, uint32_t datalength, uint32_t baselength)
    479 {
    480   uint64_t* gd = (uint64_t*)data;
    481   uint64_t* bd = (uint64_t*)basedata;
    482   uint64_t* nd = (uint64_t*)newdata;
    483 
    484   unsigned int i;
    485   for( i=0; i<datalength/8; i++ )
    486   {
    487     if( i<baselength/8 )
    488       *(nd+i) = *(gd+i) ^ *(bd+i);  // xor the data
    489     else
    490       *(nd+i) = *(gd+i); // just copy over the data
    491   }
    492   unsigned int j;
    493   // now process the rest (when datalength isn't a multiple of 4)
    494   for( j = 8*(datalength/8); j<datalength; j++ )
    495   {
    496     if( j<baselength )
    497       *(newdata+j) = *(data+j) ^ *(basedata+j); // xor
    498     else
    499       *(newdata+j) = *(data+j); // just copy
    500   }
    501   assert(j==datalength);
    502 }
    503532
    504533Gamestate* Gamestate::doSelection(unsigned int clientID, unsigned int targetSize){
    … …  
    529558//   COUT(0) << "myvector contains:";
    530559//   for ( itt=dataVector_.begin() ; itt!=dataVector_.end(); itt++ )
    531 //     COUT(0) << ' ' << (*itt).objID;
     560//     COUT(0) << " " << (*itt).objID;
    532561//   COUT(0) << endl;
    533562  for(it=dataVector_.begin(); it!=dataVector_.end();){
    534563    SynchronisableHeader oldobjectheader(origdata);
    535564    SynchronisableHeader newobjectheader(newdata);
    536     if ( it->objSize == 0 )
     565    if ( (*it).objSize == 0 )
    537566    {
    538567      ++it;
    539568      continue;
    540569    }
    541     objectsize = oldobjectheader.getDataSize();
     570    objectsize = oldobjectheader.getDataSize()+SynchronisableHeader::getSize();
    542571    objectOffset=SynchronisableHeader::getSize(); //skip the size and the availableData variables in the objectheader
    543     if ( it->objID == oldobjectheader.getObjectID() ){
     572    if ( (*it).objID == oldobjectheader.getObjectID() ){
    544573      memcpy(newdata, origdata, objectsize);
    545       assert(newobjectheader.isDataAvailable()==true);
    546574      ++it;
    547575    }else{
    548576      newobjectheader = oldobjectheader;
    549       newobjectheader.setDataAvailable(false);
    550577      memset(newdata+objectOffset, 0, objectsize-objectOffset);
    551578    }
    … …  
    559586  {
    560587    SynchronisableHeader oldobjectheader(origdata);
    561     objectsize = oldobjectheader.getDataSize();
     588    objectsize = oldobjectheader.getDataSize()+SynchronisableHeader::getSize();
    562589    origdata += objectsize;
    563590    origsize += objectsize;
    … …  
    571598
    572599
    573 /*Gamestate *Gamestate::undiff(Gamestate *base)
    574 {
    575   assert(this && base);assert(data_);
    576   assert(header_->isDiffed());
    577   assert(!header_->isCompressed() && !base->header_->isCompressed());
    578   uint8_t *basep = GAMESTATE_START(base->data_);
    579   uint8_t *gs = GAMESTATE_START(this->data_);
    580   uint32_t of=0; // pointers offset
    581   uint32_t dest_length=0;
    582   dest_length=header_->getDataSize();
    583   if(dest_length==0)
    584     return NULL;
    585   uint8_t *ndata = new uint8_t[dest_length*sizeof(uint8_t)+GamestateHeader::getSize()];
    586   uint8_t *dest = ndata + GamestateHeader::getSize();
    587   while(of < base->header_->getDataSize() && of < header_->getDataSize()){
    588     *(dest+of)=*(basep+of)^*(gs+of); // do the xor
    589     ++of;
    590   }
    591   if(base->header_->getDataSize()!=header_->getDataSize()){
    592     uint8_t n=0;
    593     if(base->header_->getDataSize() < header_->getDataSize()){
    594       while(of < dest_length){
    595         *(dest+of)=n^*(gs+of);
    596         of++;
    597       }
    598     }
    599   }
    600   Gamestate *g = new Gamestate(ndata, getClientID());
    601   assert(g->header_);
    602   *(g->header_) = *header_;
    603   g->header_->setDiffed( false );
    604   g->flags_=flags_;
    605   g->packetDirection_ = packetDirection_;
    606   assert(!g->isDiffed());
    607   assert(!g->isCompressed());
    608   return g;
    609 }*/
    610 
    611600uint32_t Gamestate::calcGamestateSize(int32_t id, uint8_t mode)
    612601{
    613   uint32_t size=0;
     602  uint32_t size = 0;
     603  uint32_t nrOfVariables = 0;
    614604    // get the start of the Synchronisable list
    615605  ObjectList<Synchronisable>::iterator it;
    616606    // get total size of gamestate
    617   for(it = ObjectList<Synchronisable>::begin(); it; ++it)
     607  for(it = ObjectList<Synchronisable>::begin(); it; ++it){
    618608    size+=it->getSize(id, mode); // size of the actual data of the synchronisable
     609    nrOfVariables += it->getNrOfVariables();
     610  }
     611//   COUT(0) << "allocating " << nrOfVariables << " ints" << endl;
     612  this->sizes_.reserve(nrOfVariables);
    619613  return size;
    620614}
     615
    621616
    622617} //namespace packet
Note: See TracChangeset for help on using the changeset viewer.