X-Git-Url: https://git.llucax.com/z.facultad/75.42/plaqui.git/blobdiff_plain/ab9300aaf1217f0c3b45dd5f1f1fdc1c5e5bd1be..751302dd67ffdeb0dfe72c4d2a7a2d58303cf5f3:/Server/src/tcpserver.cpp

diff --git a/Server/src/tcpserver.cpp b/Server/src/tcpserver.cpp
index 4131224..a9ce030 100644
--- a/Server/src/tcpserver.cpp
+++ b/Server/src/tcpserver.cpp
@@ -31,6 +31,8 @@
 #	include <iostream>
 #endif // DEBUG
 
+const unsigned PlaQui::Server::TCPServer::MAX_PENDING_CONNECTIONS = 5;
+
 PlaQui::Server::TCPServer::~TCPServer(void) {
 #ifdef DEBUG
 	std::cerr << __FILE__ <<  ": destructor." << std::endl;
@@ -47,7 +49,7 @@ PlaQui::Server::TCPServer::TCPServer(int port):
 	std::cerr << __FILE__ <<  ": escuchando en " << socket.localhost()
 		<< ":" << socket.localport() << "." << std::endl;
 #endif // DEBUG
-	socket.listen();
+	socket.listen(MAX_PENDING_CONNECTIONS);
 #ifdef DEBUG
 	std::cerr << __FILE__ <<  ": [despues de listen()] escuchando en "
 		<< socket.localhost() << ":" << socket.localport() << "." << std::endl;
@@ -60,7 +62,7 @@ void PlaQui::Server::TCPServer::on_connection_finished(
 	std::cerr << __FILE__ <<  ": on_connection_finished(connection = "
 		<< connection << ")" << std::endl;
 #endif // DEBUG
-	// TODO: poner lock.
+	mutex.lock();
 	connections.remove(connection);
 #ifdef DEBUG
 	std::cerr << __FILE__ <<  ": lista de conexiones" << std::endl;
@@ -69,33 +71,32 @@ void PlaQui::Server::TCPServer::on_connection_finished(
 		std::cerr << "\t " << *i << std::endl;
 	}
 #endif // DEBUG
-	// TODO: sacar lock.
+	mutex.unlock();
 }
 
+/// \todo TODO: ver tema de timeout o como salir de un accept().
 void PlaQui::Server::TCPServer::real_run(void) {
 #ifdef DEBUG
 	std::cerr << __FILE__ <<  ": real_run()" << std::endl;
 #endif // DEBUG
 	Connection* connection;
 	while (!stop) {
-		// TODO: ver tema de timeout o como salir de un accept().
 		// Forma grasa de salir del accept: crear conexion que salga al toque.
 		connection = new_connection(socket.accept());
 #ifdef DEBUG
-	std::cerr << __FILE__ <<  ": real_run(): connection = " << connection
-		<< std::endl;
+		std::cerr << __FILE__ <<  ": real_run(): connection = " << connection
+			<< std::endl;
 #endif // DEBUG
-		// TODO: poner lock.
+		mutex.lock();
 		connections.push_back(connection);
 #ifdef DEBUG
-	std::cerr << __FILE__ <<  ": real_run(): lista de conexiones" << std::endl;
-	for (ConnectionList::const_iterator i = connections.begin();
-			i != connections.end(); i++) {
-		std::cerr << "\t " << *i << std::endl;
-	}
+		std::cerr << __FILE__ <<  ": real_run(): lista de conexiones" << std::endl;
+		for (ConnectionList::const_iterator i = connections.begin();
+				i != connections.end(); i++) {
+			std::cerr << "\t " << *i << std::endl;
+		}
 #endif // DEBUG
-		// TODO: sacar lock.
-		// TODO: esto va en Server::new_connection()
+		mutex.unlock();
 		// Conecto la señal para cuando termina una conexión, borrarla.
 		connection->signal_finished().connect(
 				SigC::bind<Connection*>(