X-Git-Url: https://git.brokenzipper.com/gitweb?a=blobdiff_plain;f=src%2Fopenbox.cc;h=ee2fffb52be6bcaabcb3834d5b3a9d53bf92a041;hb=5dc5cc32b0c00c3bfba8ea5727599f49b56ae817;hp=440c4f70fd6c56f5be43acfaa6110b8e51b2bb51;hpb=ace8c8896aa13a6fc4e489277cf9c96f49175322;p=chaz%2Fopenbox diff --git a/src/openbox.cc b/src/openbox.cc index 440c4f70..ee2fffb5 100644 --- a/src/openbox.cc +++ b/src/openbox.cc @@ -6,6 +6,7 @@ #include "../version.h" #include "openbox.hh" +#include "otk/property.hh" #include "otk/display.hh" extern "C" { @@ -40,26 +41,9 @@ extern "C" { namespace ob { - Openbox *Openbox::instance = (Openbox *) 0; -int Openbox::xerrorHandler(Display *d, XErrorEvent *e) -{ -#ifdef DEBUG - char errtxt[128]; - - XGetErrorText(d, e->error_code, errtxt, 128); - printf("X Error: %s\n", errtxt); -#else - (void)d; - (void)e; -#endif - - return false; -} - - void Openbox::signalHandler(int signal) { switch (signal) { @@ -70,12 +54,13 @@ void Openbox::signalHandler(int signal) case SIGINT: case SIGTERM: case SIGPIPE: - printf("Caught signal %d. Exiting.", signal); - // XXX: Make Openbox exit + printf("Caught signal %d. Exiting.\n", signal); + instance->shutdown(); + break; case SIGFPE: case SIGSEGV: - printf("Caught signal %d. Aborting and dumping core.", signal); + printf("Caught signal %d. Aborting and dumping core.\n", signal); abort(); } } @@ -85,12 +70,13 @@ Openbox::Openbox(int argc, char **argv) { struct sigaction action; - _state = State_Starting; + _state = State_Starting; // initializing everything Openbox::instance = this; _displayreq = (char*) 0; _argv0 = argv[0]; + _doshutdown = false; parseCommandLine(argc, argv); @@ -109,7 +95,8 @@ Openbox::Openbox(int argc, char **argv) sigaction(SIGINT, &action, (struct sigaction *) 0); sigaction(SIGHUP, &action, (struct sigaction *) 0); - + _property = new otk::OBProperty(); + _state = State_Normal; // done starting } @@ -117,6 +104,12 @@ Openbox::Openbox(int argc, char **argv) Openbox::~Openbox() { + _state = State_Exiting; // time to kill everything + + // unmanage all windows + while (!_clients.empty()) + _xeventhandler.unmanageWindow(_clients.begin()->second); + // close the X display otk::OBDisplay::destroy(); } @@ -208,54 +201,46 @@ void Openbox::showHelp() void Openbox::eventLoop() { - const int xfd = ConnectionNumber(otk::OBDisplay::display); - - while (_state == State_Normal) { + while (!_doshutdown) { if (XPending(otk::OBDisplay::display)) { XEvent e; XNextEvent(otk::OBDisplay::display, &e); - process_event(&e); + //process_event(&e); + _xeventhandler.handle(e); } else { - fd_set rfds; - timeval now, tm, *timeout = (timeval *) 0; - - FD_ZERO(&rfds); - FD_SET(xfd, &rfds); - -/* if (! timerList.empty()) { - const BTimer* const timer = timerList.top(); - - gettimeofday(&now, 0); - tm = timer->timeRemaining(now); + _timermanager.fire(); + } + } +} - timeout = &tm; - } - select(xfd + 1, &rfds, 0, 0, timeout); +void Openbox::addClient(Window window, OBClient *client) +{ + _clients[window] = client; +} - // check for timer timeout - gettimeofday(&now, 0); - // there is a small chance for deadlock here: - // *IF* the timer list keeps getting refreshed *AND* the time between - // timer->start() and timer->shouldFire() is within the timer's period - // then the timer will keep firing. This should be VERY near impossible. - while (! timerList.empty()) { - BTimer *timer = timerList.top(); - if (! timer->shouldFire(now)) - break; +void Openbox::removeClient(Window window) +{ + ClientMap::iterator it = _clients.find(window); + if (it != _clients.end()) + _clients.erase(it); +} - timerList.pop(); - timer->fireTimeout(); - timer->halt(); - if (timer->isRecurring()) - timer->start(); - }*/ - } - } +OBClient *Openbox::findClient(Window window) +{ + /* + NOTE: we dont use _clients[] to find the value because that will insert + a new null into the hash, which really sucks when we want to clean up the + hash at shutdown! + */ + ClientMap::iterator it = _clients.find(window); + if (it != _clients.end()) + return it->second; + else + return (OBClient*) 0; } - }