fixed sequence to floating point conversion in usals parameters

[enigma2.git] / lib / actions / action.cpp

diff --git a/lib/actions/action.cpp b/lib/actions/action.cpp
index 25db199e81277bc723ecd9ca336d315117815c21..232237cdf626f98123e7f5bdb9ad0a6d0c32c9a5 100644 (file)
--- a/lib/actions/action.cpp
+++ b/lib/actions/action.cpp
@@ -124,8 +124,19 @@ void eActionMap::bindKey(const std::string &device, int key, int flags, const st
        m_python_keys.insert(std::pair<std::string,ePythonKeyBinding>(context, bind));
 }
 
+struct call_entry
+{
+       PyObject *m_fnc, *m_arg;
+       eWidget *m_widget;
+       void *m_widget_arg;
+       call_entry(PyObject *fnc, PyObject *arg): m_fnc(fnc), m_arg(arg), m_widget(0), m_widget_arg(0) { }
+       call_entry(eWidget *widget, void *arg): m_widget(widget), m_widget_arg(arg), m_fnc(0), m_arg(0) { }
+};
+
 void eActionMap::keyPressed(int device, int key, int flags)
 {
+       std::list<call_entry> call_list;
+       
                /* iterate active contexts. */
        for (std::multimap<int,eActionBinding>::const_iterator c(m_bindings.begin()); c != m_bindings.end();)
        {
@@ -145,12 +156,9 @@ void eActionMap::keyPressed(int device, int key, int flags)
                                {
                                        if (
                                                // (k->second.m_device == m_device) &&
-                                               (k->second.m_key == key) &&
-                                               (k->second.m_flags & (1<<flags)))
-                                       {
-                                               if (i->second.m_widget->event(eWidget::evtAction, 0, (void*)k->second.m_action))
-                                                       return;
-                                       }
+                                                       (k->second.m_key == key) &&
+                                                       (k->second.m_flags & (1<<flags)))
+                                               call_list.push_back(call_entry(i->second.m_widget, (void*)k->second.m_action));
                                }
                        } else
                        {
@@ -177,8 +185,8 @@ void eActionMap::keyPressed(int device, int key, int flags)
                                                PyTuple_SetItem(pArgs, 0, PyString_FromString(k->first.c_str()));
                                                PyTuple_SetItem(pArgs, 1, PyString_FromString(k->second.m_action.c_str()));
                                                ++k;
-                                               ePython::call(i->second.m_fnc, pArgs);
-                                               Py_DECREF(pArgs);
+                                               Py_INCREF(i->second.m_fnc);
+                                               call_list.push_back(call_entry(i->second.m_fnc, pArgs));
                                        } else
                                                ++k;
                                }
@@ -187,11 +195,28 @@ void eActionMap::keyPressed(int device, int key, int flags)
                                PyObject *pArgs = PyTuple_New(2);
                                PyTuple_SetItem(pArgs, 0, PyInt_FromLong(key));
                                PyTuple_SetItem(pArgs, 1, PyInt_FromLong(flags));
-                               ePython::call(i->second.m_fnc, pArgs);
-                               Py_DECREF(pArgs);
+                               Py_INCREF(i->second.m_fnc);
+                               call_list.push_back(call_entry(i->second.m_fnc, pArgs));
                        }
                }
        }
+
+       int res = 0;
+                       /* we need to iterate over all to not loose a reference */
+       for (std::list<call_entry>::iterator i(call_list.begin()); i != call_list.end(); ++i)
+       {
+               if (i->m_fnc)
+               {
+                       if (!res)
+                               res = ePython::call(i->m_fnc, i->m_arg);
+                       Py_DECREF(i->m_fnc);
+                       Py_DECREF(i->m_arg);
+               } else if (i->m_widget)
+               {
+                       if (!res)
+                               res = i->m_widget->event(eWidget::evtAction, 0, (void*)i->m_widget_arg);
+               }
+       }
 }
 
 eAutoInitPtr<eActionMap> init_eActionMap(eAutoInitNumbers::actions, "eActionMap");