X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=src%2Fpython.cc;h=ed71463c8489f1b68ddd9aee5dc69aa1481a94c0;hb=d4853f108c1d22c344c1cb9a8a8b7cdc46648983;hp=ed171f9cc4880659e2f3642dd310b606964751d4;hpb=c645416035d64cc959d1dd0e937b31b07489e54b;p=chaz%2Fopenbox diff --git a/src/python.cc b/src/python.cc index ed171f9c..ed71463c 100644 --- a/src/python.cc +++ b/src/python.cc @@ -1,6 +1,7 @@ // -*- mode: C++; indent-tabs-mode: nil; c-basic-offset: 2; -*- #include "python.hh" +#include "openbox.hh" #include #include @@ -10,6 +11,8 @@ namespace ob { typedef std::vector FunctionList; static FunctionList callbacks[OBActions::NUM_ACTIONS]; +static FunctionList keyfuncs; +static FunctionList mousefuncs; bool python_register(int action, PyObject *callback) { @@ -125,4 +128,119 @@ void python_callback(OBActions::ActionType action, Window window, Py_DECREF(arglist); } + + + + + +bool python_bind_key(PyObject *keylist, PyObject *callback) +{ + if (!PyList_Check(keylist)) { + PyErr_SetString(PyExc_AssertionError, "Invalid keylist. Not a list."); + return false; + } + if (!PyCallable_Check(callback)) { + PyErr_SetString(PyExc_AssertionError, "Invalid callback function."); + return false; + } + + OBBindings::StringVect vectkeylist; + for (int i = 0, end = PyList_Size(keylist); i < end; ++i) { + PyObject *str = PyList_GetItem(keylist, i); + if (!PyString_Check(str)) { + PyErr_SetString(PyExc_AssertionError, + "Invalid keylist. It must contain only strings."); + return false; + } + vectkeylist.push_back(PyString_AsString(str)); + } + + // the id is what the binding class can call back with so it doesnt have to + // worry about the python function pointer + int id = keyfuncs.size(); + if (Openbox::instance->bindings()->add_key(vectkeylist, id)) { + Py_XINCREF(callback); // Add a reference to new callback + keyfuncs.push_back(callback); + return true; + } else { + PyErr_SetString(PyExc_AssertionError,"Unable to create binding. Invalid."); + return false; + } +} + +bool python_unbind_key(PyObject *keylist) +{ + if (!PyList_Check(keylist)) { + PyErr_SetString(PyExc_AssertionError, "Invalid keylist. Not a list."); + return false; + } + + OBBindings::StringVect vectkeylist; + for (int i = 0, end = PyList_Size(keylist); i < end; ++i) { + PyObject *str = PyList_GetItem(keylist, i); + if (!PyString_Check(str)) { + PyErr_SetString(PyExc_AssertionError, + "Invalid keylist. It must contain only strings."); + return false; + } + vectkeylist.push_back(PyString_AsString(str)); + } + + int id; + if ((id = + Openbox::instance->bindings()->remove_key(vectkeylist)) >= 0) { + assert(keyfuncs[id]); // shouldn't be able to remove it twice + Py_XDECREF(keyfuncs[id]); // Dispose of previous callback + // important note: we don't erase the item from the list cuz that would + // ruin all the id's that are in use. simply nullify it. + keyfuncs[id] = 0; + return true; + } + + return false; +} + +bool python_bind_mouse(const std::string &button, PyObject *callback) +{ + if (!PyCallable_Check(callback)) { + PyErr_SetString(PyExc_AssertionError, "Invalid callback function."); + return false; + } + + // the id is what the binding class can call back with so it doesnt have to + // worry about the python function pointer + int id = mousefuncs.size(); + if (Openbox::instance->bindings()->add_mouse(button, id)) { + Py_XINCREF(callback); // Add a reference to new callback + mousefuncs.push_back(callback); + return true; + } else { + PyErr_SetString(PyExc_AssertionError,"Unable to create binding. Invalid."); + return false; + } +} + +bool python_unbind_mouse(const std::string &button) +{ + int id; + if ((id = + Openbox::instance->bindings()->remove_mouse(button)) >= 0) { + assert(mousefuncs[id]); // shouldn't be able to remove it twice + Py_XDECREF(mousefuncs[id]); // Dispose of previous callback + // important note: we don't erase the item from the list cuz that would + // ruin all the id's that are in use. simply nullify it. + mousefuncs[id] = 0; + return true; + } + + return false; +} + +bool python_unbind_all() +{ + Openbox::instance->bindings()->remove_all(); + return true; +} + + }