Changeset 7672

Timestamp:

Nov 24, 2010, 11:12:30 PM (14 years ago)

Author:

smerkli

Message:

just added some notes so I understand my own stuff next week.

Location:

code/branches/masterserver/src/libraries/network

Files:

• MasterServerComm.cc (modified) (1 diff)
• WANDiscovery.cc (modified) (3 diffs)
• WANDiscovery.h (modified) (1 diff)

code/branches/masterserver/src/libraries/network/MasterServerComm.cc

@@ -109 +109 @@
     COUT(2) << "MARK polling...\n";
 
-    if( enet_host_service( this->client, &this->event, 1000 ) >= 0 )
+    /* enet_host_service returns 0 if no event occured */
+    /* just newly set below test to >0 from >= 0, to be tested */
+    if( enet_host_service( this->client, &this->event, 1000 ) > 0 )
     { 
       /* address buffer */

code/branches/masterserver/src/libraries/network/WANDiscovery.cc

@@ -43 +43 @@
   WANDiscovery::WANDiscovery()
   {
-    COUT(0) << "WANDiscovery created.\n";
-    /* create master server communications object */
-    //this->msc = MasterServerComm();
+    COUT(4) << "Creating WANDiscovery.\n";
 
     /* initialize it and see if it worked */
     if( msc.initialize() )
-      COUT(0) << "Error: could not initialize master server communications!\n";
+      COUT(2) << "Error: could not initialize master server communications!\n";
 
     /* connect and see if it worked */
     if( msc.connect( MS_ADDRESS, 1234 ) )
-      COUT(0) << "Error: could not connect to master server!\n";
+      COUT(2) << "Error: could not connect to master server!\n";
 
-    COUT(0) << "Initialization of WANDiscovery complete.\n";
+    COUT(4) << "Initialization of WANDiscovery complete.\n";
   }
 
@@ -92 +90 @@
     else if( !strncmp( (char*)ev->packet->data, MSPROTO_SERVERLIST_END,
       MSPROTO_SERVERLIST_END_LEN ) )
-    { return 1; }
+    { 
+      /* this is the only case where 1 should be returned,
+       * as 1 is used to signal that we're done receiving
+       * the list
+       */
+      return 1; 
+    }
 
     /* done handling, return all ok code 0 */
@@ -106 +110 @@
     this->msc.sendRequest( MSPROTO_CLIENT " " MSPROTO_REQ_LIST );
 
-    /* deal with replies */
+    /* poll for replies */
+    /* TODO add some timeout here so we don't wait indefinitely */
     while( !((this->msc).pollForReply( rhandler )) )
       /* nothing */;

code/branches/masterserver/src/libraries/network/WANDiscovery.h

@@ -78 +78 @@
       static WANDiscovery& getInstance() { return Singleton<WANDiscovery>::getInstance(); } // tolua_export
     
+      /* todo: might make this private and use getter/setter methods
+       * at some later time. 
+       */
       /** game server list */
       std::vector<packet::ServerInformation> servers_;