Se arregla el bug que hacia que se cuelgue el cliente al desconectarse.

[z.facultad/75.42/plaqui.git] / Server / src / runnable.cpp

diff --git a/Server/src/runnable.cpp b/Server/src/runnable.cpp
index 0e8b4bde525dae9c80e9cdd4382fab2e5bd47b41..c134b834fbe0709a0314b241a72d93b9cce74661 100644 (file)
--- a/Server/src/runnable.cpp
+++ b/Server/src/runnable.cpp
@@ -26,30 +26,56 @@
 //
 
 #include "plaqui/server/runnable.h"
-#include <sigc++/class_slot.h>
+#include <sigc++/slot.h>
 #include <glibmm/thread.h>
 #ifdef DEBUG
 #      include <iostream>
 #endif // DEBUG
 
-using namespace Plaqui;
+using namespace std;
 
-Runnable::Runnable(void): thread(0) {
+namespace PlaQui {
+
+namespace Server {
+
+Runnable::~Runnable(void) {
+#ifdef DEBUG
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": destructor(this = " << this << ")"
+               << endl;
+#endif // DEBUG
+}
+
+Runnable::Runnable(void): thread(0), stop(false) {
+#ifdef DEBUG
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": constructor." << endl;
+#endif // DEBUG
+}
+
+void Runnable::static_run(Runnable* runner) {
+#ifdef DEBUG
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": static_run(runner = " << runner << ")"
+               << endl;
+#endif // DEBUG
+       runner->real_run();
+       runner->finished();
+       delete runner;
 }
 
 void Runnable::run(bool detach) {
 #ifdef DEBUG
-       std::cerr << __FILE__ << ": detach = " << detach << std::endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": run(detach = " << detach << ")" << endl;
 #endif // DEBUG
        // Si vamos a correr la tarea en un thread.
        if (detach) {
-               // Nos aseguramos de tener threads.
-               if (!Glib::thread_supported()) {
-                       Glib::thread_init();
-               }
-               // Corremos el thread.
+               // Corremos el thread en una funcion estática para poder destruirlo al
+               // finalizar, pasandole el puntero al objeto.
                thread = Glib::Thread::create(
-                               SigC::slot_class(*this, &Runnable::real_run), true);
+                               SigC::bind<Runnable*>(SigC::slot(&Runnable::static_run), this),
+                               true);
        // Si no corremos la tarea normalmente.
        } else {
                real_run();
@@ -57,3 +83,26 @@ void Runnable::run(bool detach) {
                
 }
 
+void Runnable::finish(bool attach) {
+#ifdef DEBUG
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": finish(attach = " << attach << ")" << endl;
+#endif // DEBUG
+       stop = true;
+       if (attach) {
+               thread->join();
+       }
+}
+
+Runnable::SignalFinished& Runnable::signal_finished(void) {
+       return finished;
+}
+
+Runnable::SignalError& Runnable::signal_error(void) {
+       return error;
+}
+
+} // namespace Server
+
+} // namespace PlaQui
+