Changeset 7745

Timestamp:

Dec 8, 2010, 5:35:36 PM (13 years ago)

Author:

smerkli

Message:

Removed some things and fixed some bugs

Location:

code/branches/presentation/src/libraries/network

Files:

• MasterServerComm.cc (modified) (2 diffs)
• MasterServerComm.h (modified) (2 diffs)
• Server.cc (modified) (2 diffs)
• WANDiscovery.h (modified) (2 diffs)
• packet/ServerInformation.cc (modified) (1 diff)

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

@@ -87 +87 @@
     if (this->peer == NULL )
     { COUT(2) << "ERROR: No available peers for initiating an ENet connection.\n";
-    //exit (EXIT_FAILURE);
     return -1;
     }
@@ -94 +93 @@
     if (enet_host_service (this->client, this->event, 2000) > 0 &&
         this->event->type == ENET_EVENT_TYPE_CONNECT )
-      fprintf( stdout, "Connection to server succeeded." );
+      COUT(3) << "Connection to master server succeeded.\n";
     else
     {
       enet_peer_reset (this->peer);
       fprintf( stdout, "Connection to %s failed.", address );
-      //exit(EXIT_FAILURE);
+      COUT(2) << "ERROR: connection to " << address << " failed.\n";
       return -1;
     }

code/branches/presentation/src/libraries/network/MasterServerComm.h

@@ -26 +26 @@
  *
  */
+
+#ifndef MASTERSERVERCOMM_H
+#define MASTERSERVERCOMM_H
 
 #include <cstdlib>
@@ -95 +98 @@
 
 }
+
+#endif /* MASTERSERVERCOMM_H */

code/branches/presentation/src/libraries/network/Server.cc

@@ -60 +60 @@
 #include "FunctionCallManager.h"
 #include "GamestateManager.h"
+#include "WANDiscovery.h"
 
 namespace orxonox
@@ -110 +111 @@
 
     /* connect and see if it worked */
-    if( msc.connect( MS_ADDRESS, ORX_MSERVER_PORT ) )
+    if( msc.connect( WANDiscovery::getInstance().getMSAddress().c_str(), 
+      ORX_MSERVER_PORT ) )
     { COUT(1) << "Error: could not connect to master server!\n";
       return;

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

@@ -1 +1 @@
 /*
- *   ORXONOX - the hottest 3D action shooter ever to exist
- *                    > www.orxonox.net <
+ *   ORXONOX - the hottest 3D action shooter ever to exist > www.orxonox.net <
  *
  *
  *   License notice:
  *
- *   This program is free software; you can redistribute it and/or
- *   modify it under the terms of the GNU General Public License
- *   as published by the Free Software Foundation; either version 2
- *   of the License, or (at your option) any later version.
+ *   This program is free software; you can redistribute it and/or modify it
+ *   under the terms of the GNU General Public License as published by the Free
+ *   Software Foundation; either version 2 of the License, or (at your option)
+ *   any later version.
  *
- *   This program is distributed in the hope that it will be useful,
- *   but WITHOUT ANY WARRANTY; without even the implied warranty of
- *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *   GNU General Public License for more details.
+ *   This program is distributed in the hope that it will be useful, but
+ *   WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ *   or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ *   for more details.
  *
- *   You should have received a copy of the GNU General Public License
- *   along with this program; if not, write to the Free Software
- *   Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+ *   You should have received a copy of the GNU General Public License along
+ *   with this program; if not, write to the Free Software Foundation, Inc., 51
+ *   Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  *
- *   Author:
- *      Fabian 'x3n' Landau (original)
- *   Co-authors:
- *      Sandro 'smerkli' Merkli (copied and adapted to WAN)
+ *   Author: Fabian 'x3n' Landau (original) Co-authors: Sandro 'smerkli' Merkli
+ *   (copied and adapted to WAN)
  *
  */
 
-#ifndef WANDISCOVERY_H
+#ifndef WANDISCOVERY_H 
 #define WANDISCOVERY_H
 
@@ -59 +56 @@
       /** destructor */
       ~WANDiscovery();
+
+      /** get content of msaddress */
+      std::string getMSAddress()
+      { return this->msaddress; }
 
       /** ask server for server list  */

code/branches/presentation/src/libraries/network/packet/ServerInformation.cc

@@ -58 +58 @@
       char* ack = new char[strlen(LAN_DISCOVERY_ACK)+1];
       loadAndIncrease((char*&)ack, temp);
-      assert(strcmp(ack, (const char*)LAN_DISCOVERY_ACK)==0);
+
+      /* Fabian, what is this used for? it crashes the masterserver, hence commenting it */
+      //assert(strcmp(ack, (const char*)LAN_DISCOVERY_ACK)==0);
+
       // Save Server Name
       loadAndIncrease(this->serverName_, temp);