Se pone mas prolijo el tema de la respuesta.

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

diff --git a/Server/src/runnable.cpp b/Server/src/runnable.cpp
index 7fd410535c9c6893145182e8a3b3f358636eb4fc..6d000ce9c6d68f6adc47b271680ce537619222ee 100644 (file)
--- a/Server/src/runnable.cpp
+++ b/Server/src/runnable.cpp
@@ -26,50 +26,55 @@
 //
 
 #include "plaqui/server/runnable.h"
-#include <sigc++/class_slot.h>
+#include <sigc++/slot.h>
 #include <glibmm/thread.h>
 #ifdef DEBUG
 #      include <iostream>
 #endif // DEBUG
 
-PlaQui::Server::Runnable::~Runnable(void) {
+using namespace std;
+
+namespace PlaQui {
+
+namespace Server {
+
+Runnable::~Runnable(void) {
 #ifdef DEBUG
-       std::cerr << __FILE__ << ": destructor(this = " << this << ")"
-               << std::endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": destructor(this = " << this << ")." << endl;
 #endif // DEBUG
 }
 
-PlaQui::Server::Runnable::Runnable(void): thread(0), stop(false) {
+Runnable::Runnable(void): _thread(NULL), _stop(false) {
 #ifdef DEBUG
-       std::cerr << __FILE__ << ": constructor." << std::endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": constructor(this = " << this << ")." << endl;
 #endif // DEBUG
 }
 
-void PlaQui::Server::Runnable::static_run(Runnable* runner) {
+void Runnable::static_run(Runnable* runner) {
 #ifdef DEBUG
-       std::cerr << __FILE__ << ": static_run(runner = " << runner << ")"
-               << std::endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": static_run(runner = " << runner << ")" << endl;
 #endif // DEBUG
+       // Corre tarea.
        runner->real_run();
-       runner->signal_finished().emit();
+       // Manda señal de tarea finalizada
+       runner->_finished();
        delete runner;
 }
 
-void PlaQui::Server::Runnable::run(bool detach) {
+void Runnable::run(bool detach) {
 #ifdef DEBUG
-       std::cerr << __FILE__ << ": run(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 en una funcion estática para poder destruirlo al
                // finalizar, pasandole el puntero al objeto.
-               thread = Glib::Thread::create(
-                               SigC::bind<Runnable*>(SigC::slot(&Runnable::static_run), this),
-                               true);
+               _thread = Glib::Thread::create(
+                               SigC::bind(SigC::slot(&Runnable::static_run), this), false);
        // Si no corremos la tarea normalmente.
        } else {
                real_run();
@@ -77,19 +82,36 @@ void PlaQui::Server::Runnable::run(bool detach) {
                
 }
 
-void PlaQui::Server::Runnable::finish(bool attach) {
+void Runnable::finish(void) {
 #ifdef DEBUG
-       std::cerr << __FILE__ << ": finish(attach = " << attach << ")" << std::endl;
+       cerr << __FILE__ << "(" << __LINE__ << ")"
+               << ": finish();" << endl;
 #endif // DEBUG
-       // TODO - necesita un mutex?
-       stop = true;
-       if (attach) {
-               thread->join();
-       }
+       stop(true);
+}
+
+bool Runnable::stop(void) {
+       Glib::Mutex::Lock lock(stop_mutex);
+       bool tmp = _stop;
+       return tmp;
 }
 
-PlaQui::Server::Runnable::SignalFinished&
-PlaQui::Server::Runnable::signal_finished(void) {
-       return finished;
+bool Runnable::stop(bool stop) {
+       Glib::Mutex::Lock lock(stop_mutex);
+       bool tmp = _stop;
+       _stop = stop;
+       return tmp;
 }
 
+Runnable::SignalFinished& Runnable::signal_finished(void) {
+       return _finished;
+}
+
+Runnable::SignalError& Runnable::signal_error(void) {
+       return _error;
+}
+
+} // namespace Server
+
+} // namespace PlaQui
+