From 04494f100baac2df9785d05d3487d6193f8545fe Mon Sep 17 00:00:00 2001 From: Dana Jansens Date: Wed, 19 Feb 2003 01:29:11 +0000 Subject: [PATCH] dont assume only 5 mouse buttons. dont need to track them all either, kill the queue, and watch one at a timesmarter --- src/actions.cc | 78 +++++++++++++------------------------------------- src/actions.hh | 16 ++--------- 2 files changed, 22 insertions(+), 72 deletions(-) diff --git a/src/actions.cc b/src/actions.cc index c340d64b..9659b2f0 100644 --- a/src/actions.cc +++ b/src/actions.cc @@ -16,63 +16,20 @@ namespace ob { -const int Actions::BUTTONS; - Actions::Actions() - : _button(0), - _dragging(false) + : _dragging(false) { - for (int i=0; i 0; --i) { - _posqueue[i] = _posqueue[i-1]; - } - _posqueue[0] = a; - a->win = e.window; - a->button = e.button; - a->pos = otk::Point(e.x_root, e.y_root); - - Client *c = openbox->findClient(e.window); - if (c) a->clientarea = c->area(); -} - -void Actions::removePress(const XButtonEvent &e) -{ - int i; - ButtonPressAction *a = 0; - for (i=0; ibutton == e.button) { - a = _posqueue[i]; - break; - } - if (a) { // found one, remove it and shift the rest up one - for (; i < BUTTONS-1; ++i) - _posqueue[i] = _posqueue[i+1]; - _posqueue[BUTTONS-1] = a; - } - _posqueue[BUTTONS-1]->button = 0; -} - void Actions::buttonPressHandler(const XButtonEvent &e) { otk::EventHandler::buttonPressHandler(e); - insertPress(e); - - printf("press queue %u pressed %u\n", _button, e.button); MouseContext::MC context; EventHandler *h = openbox->findHandler(e.window); @@ -86,20 +43,20 @@ void Actions::buttonPressHandler(const XButtonEvent &e) else return; // not a valid mouse context - if (_button) { + if (_press.button) { unsigned int mask; - switch(_button) { + switch(_press.button) { case Button1: mask = Button1Mask; break; case Button2: mask = Button2Mask; break; case Button3: mask = Button3Mask; break; case Button4: mask = Button4Mask; break; case Button5: mask = Button5Mask; break; - default: assert(false); return; // unhandled button + default: mask = 0; // on other buttons we have to assume its not pressed... } // was the button released but we didnt get the event? (pointergrabs cause // this) if (!(e.state & mask)) - _button = 0; + _press.button = 0; } // run the PRESS python hook @@ -116,9 +73,15 @@ void Actions::buttonPressHandler(const XButtonEvent &e) MouseAction::Press); openbox->bindings()->fireButton(&data); - if (_button) return; // won't count toward CLICK events + if (_press.button) return; // won't count toward CLICK events + + _press.win = e.window; + _press.button = e.button; + _press.pos = otk::Point(e.x_root, e.y_root); + if (c) + _press.clientarea = c->area(); - _button = e.button; + printf("press queue %u pressed %u\n", _press.button, e.button); if (context == MouseContext::Window) { /* @@ -136,7 +99,6 @@ void Actions::buttonPressHandler(const XButtonEvent &e) void Actions::buttonReleaseHandler(const XButtonEvent &e) { otk::EventHandler::buttonReleaseHandler(e); - //removePress(e); MouseContext::MC context; EventHandler *h = openbox->findHandler(e.window); @@ -165,9 +127,9 @@ void Actions::buttonReleaseHandler(const XButtonEvent &e) openbox->bindings()->fireButton(&data); // not for the button we're watching? - if (_button != e.button) return; + if (_press.button != e.button) return; - _button = 0; + _press.button = 0; _dragging = false; // find the area of the window @@ -308,7 +270,7 @@ void Actions::motionHandler(const XMotionEvent &e) if (!e.same_screen) return; // this just gets stupid - if (e.window != _posqueue[0]->win) return; + if (e.window != _press.win) return; MouseContext::MC context; EventHandler *h = openbox->findHandler(e.window); @@ -339,8 +301,8 @@ void Actions::motionHandler(const XMotionEvent &e) screen = otk::display->findScreen(e.root)->screen(); if (!_dragging) { - int dx = x_root - _posqueue[0]->pos.x(); - int dy = y_root - _posqueue[0]->pos.y(); + int dx = x_root - _press.pos.x(); + int dy = y_root - _press.pos.y(); long threshold = openbox->screen(screen)->config().drag_threshold; if (!(std::abs(dx) >= threshold || std::abs(dy) >= threshold)) return; // not at the threshold yet @@ -353,10 +315,10 @@ void Actions::motionHandler(const XMotionEvent &e) // kill off the Button1Mask etc, only want the modifiers unsigned int state = e.state & (ControlMask | ShiftMask | Mod1Mask | Mod2Mask | Mod3Mask | Mod4Mask | Mod5Mask); - unsigned int button = _posqueue[0]->button; + unsigned int button = _press.button; MouseData data(screen, c, e.time, state, button, context, MouseAction::Motion, x_root, y_root, - _posqueue[0]->pos, _posqueue[0]->clientarea); + _press.pos, _press.clientarea); openbox->bindings()->fireButton(&data); } diff --git a/src/actions.hh b/src/actions.hh index c2de2b68..4165dd2e 100644 --- a/src/actions.hh +++ b/src/actions.hh @@ -43,26 +43,14 @@ public: }; #endif // SWIG private: - // milliseconds XXX: config option - static const int BUTTONS = 5; - - //! The mouse button currently being watched from a press for a CLICK - unsigned int _button; //! The last button release processed for CLICKs ButtonReleaseAction _release; - //! The point where the mouse was when each mouse button was pressed - /*! - Used for motion events as the starting position. - */ - ButtonPressAction *_posqueue[BUTTONS]; + //! The last button press processed for CLICKs + ButtonPressAction _press; //! This is set to true once a drag has started and false when done to make //! sure the threshold isnt checked anymore once a drag is underway bool _dragging; - - void insertPress(const XButtonEvent &e); - void removePress(const XButtonEvent &e); - public: //! Constructs an Actions object Actions(); -- 2.39.2