speedups
[enigma2.git] / lib / gui / elistboxcontent.cpp
index 5959343a4033552e2f057b569d891c0090c3fadd..d72b21bcd05a3a1e2ea1930245a6893f4a3c62e7 100644 (file)
@@ -335,12 +335,12 @@ void eListboxPythonStringContent::paint(gPainter &painter, eWindowStyle &style,
 
        if (m_list && cursorValid())
        {
-               PyObject *item = PyList_GetItem(m_list, m_cursor); // borrowed reference!
+               PyObject *item = PyList_GET_ITEM(m_list, m_cursor); // borrowed reference!
                painter.setFont(fnt);
 
                        /* the user can supply tuples, in this case the first one will be displayed. */         
                if (PyTuple_Check(item))
-                       item = PyTuple_GetItem(item, 0);
+                       item = PyTuple_GET_ITEM(item, 0);
                
                const char *string = PyString_Check(item) ? PyString_AsString(item) : "<not-a-string>";
                
@@ -377,7 +377,7 @@ PyObject *eListboxPythonStringContent::getCurrentSelection()
                return 0;
        if (!cursorValid())
                return 0;
-       PyObject *r = PyList_GetItem(m_list, m_cursor);
+       PyObject *r = PyList_GET_ITEM(m_list, m_cursor);
        Py_XINCREF(r);
        return r;
 }
@@ -400,14 +400,15 @@ void eListboxPythonConfigContent::paint(gPainter &painter, eWindowStyle &style,
 {
        ePtr<gFont> fnt = new gFont("Regular", 20);
        ePtr<gFont> fnt2 = new gFont("Regular", 16);
-       painter.clip(eRect(offset, m_itemsize));
+       eRect itemrect(offset, m_itemsize);
+       painter.clip(itemrect);
        style.setStyle(painter, selected ? eWindowStyle::styleListboxSelected : eWindowStyle::styleListboxNormal);
        painter.clear();
 
        if (m_list && cursorValid())
        {
                        /* get current list item */
-               PyObject *item = PyList_GetItem(m_list, m_cursor); // borrowed reference!
+               PyObject *item = PyList_GET_ITEM(m_list, m_cursor); // borrowed reference!
                PyObject *text = 0, *value = 0;
                painter.setFont(fnt);
 
@@ -421,20 +422,20 @@ void eListboxPythonConfigContent::paint(gPainter &painter, eWindowStyle &style,
                {
                                /* handle left part. get item from tuple, convert to string, display. */
                                
-                       text = PyTuple_GetItem(item, 0);
-                       text = PyObject_Str(text); /* creates a new object - old object was borrowed! */
+                       text = PyTuple_GET_ITEM(item, 0);
+//                     text = PyObject_Str(text); /* creates a new object - old object was borrowed! */
                        const char *string = (text && PyString_Check(text)) ? PyString_AsString(text) : "<not-a-string>";
                        eSize item_left = eSize(m_seperation, m_itemsize.height());
                        eSize item_right = eSize(m_itemsize.width() - m_seperation, m_itemsize.height());
                        painter.renderText(eRect(offset, item_left), string, gPainter::RT_HALIGN_LEFT);
-                       Py_XDECREF(text);
+//                     Py_XDECREF(text);
                        
                                /* now, handle the value. get 2nd part from tuple*/
-                       value = PyTuple_GetItem(item, 1);
+                       value = PyTuple_GET_ITEM(item, 1);
                        if (value)
                        {
                                PyObject *args = PyTuple_New(1);
-                               PyTuple_SetItem(args, 0, PyInt_FromLong(selected));
+                               PyTuple_SET_ITEM(args, 0, PyInt_FromLong(selected));
                                
                                        /* CallObject will call __call__ which should return the value tuple */
                                value = PyObject_CallObject(value, args);
@@ -450,14 +451,14 @@ void eListboxPythonConfigContent::paint(gPainter &painter, eWindowStyle &style,
                        if (value && PyTuple_Check(value))
                        {
                                        /* convert type to string */
-                               PyObject *type = PyTuple_GetItem(value, 0);
+                               PyObject *type = PyTuple_GET_ITEM(value, 0);
                                const char *atype = (type && PyString_Check(type)) ? PyString_AsString(type) : 0;
                                
                                if (atype)
                                {
                                        if (!strcmp(atype, "text"))
                                        {
-                                               PyObject *pvalue = PyTuple_GetItem(value, 1);
+                                               PyObject *pvalue = PyTuple_GET_ITEM(value, 1);
                                                const char *value = (pvalue && PyString_Check(pvalue)) ? PyString_AsString(pvalue) : "<not-a-string>";
                                                painter.setFont(fnt2);
                                                painter.renderText(eRect(offset + eSize(m_seperation, 0), item_right), value, gPainter::RT_HALIGN_RIGHT);
@@ -465,7 +466,7 @@ void eListboxPythonConfigContent::paint(gPainter &painter, eWindowStyle &style,
                                                        /* pvalue is borrowed */
                                        } else if (!strcmp(atype, "slider"))
                                        {
-                                               PyObject *pvalue = PyTuple_GetItem(value, 1);
+                                               PyObject *pvalue = PyTuple_GET_ITEM(value, 1);
                                                
                                                        /* convert value to Long. fallback to -1 on error. */
                                                int value = (pvalue && PyInt_Check(pvalue)) ? PyInt_AsLong(pvalue) : -1;
@@ -483,7 +484,7 @@ void eListboxPythonConfigContent::paint(gPainter &painter, eWindowStyle &style,
                                                        /* pvalue is borrowed */
                                        } else if (!strcmp(atype, "mtext"))
                                        {
-                                               PyObject *pvalue = PyTuple_GetItem(value, 1);
+                                               PyObject *pvalue = PyTuple_GET_ITEM(value, 1);
                                                const char *text = (pvalue && PyString_Check(pvalue)) ? PyString_AsString(pvalue) : "<not-a-string>";
                                                
                                                ePtr<eTextPara> para = new eTextPara(eRect(offset + eSize(m_seperation, 0), item_right));
@@ -495,7 +496,7 @@ void eListboxPythonConfigContent::paint(gPainter &painter, eWindowStyle &style,
                                                PyObject *plist = 0;
                                                
                                                if (PyTuple_Size(value) >= 3)
-                                                       plist = PyTuple_GetItem(value, 2);
+                                                       plist = PyTuple_GET_ITEM(value, 2);
                                                
                                                int entries = 0;
 
@@ -504,7 +505,7 @@ void eListboxPythonConfigContent::paint(gPainter &painter, eWindowStyle &style,
                                                
                                                for (int i = 0; i < entries; ++i)
                                                {
-                                                       PyObject *entry = PyList_GetItem(plist, i);
+                                                       PyObject *entry = PyList_GET_ITEM(plist, i);
                                                        int num = PyInt_Check(entry) ? PyInt_AsLong(entry) : -1;
                                                        
                                                        if ((num < 0) || (num >= glyphs))
@@ -552,140 +553,176 @@ void eListboxPythonMultiContent::paint(gPainter &painter, eWindowStyle &style, c
 
        if (m_list && cursorValid())
        {
-               PyObject *items = PyList_GetItem(m_list, m_cursor); // borrowed reference!
+               PyObject *items = PyList_GET_ITEM(m_list, m_cursor); // borrowed reference!
                
                if (!items)
                {
                        eDebug("eListboxPythonMultiContent: error getting item %d", m_cursor);
-                       painter.clippop();
-                       return;
+                       goto error_out;
                }
                
                if (!PyList_Check(items))
                {
                        eDebug("eListboxPythonMultiContent: list entry %d is not a list", m_cursor);
-                       painter.clippop();
-                       return;
+                       goto error_out;
                }
                
                int size = PyList_Size(items);
                for (int i = 1; i < size; ++i)
                {
-                       PyObject *item = PyList_GetItem(items, i); // borrowed reference!
+                       PyObject *item = PyList_GET_ITEM(items, i); // borrowed reference!
                        
                        if (!item)
                        {
                                eDebug("eListboxPythonMultiContent: ?");
-                               painter.clippop();
-                               return;
+                               goto error_out;
                        }
                        
-                       
                        PyObject *px = 0, *py = 0, *pwidth = 0, *pheight = 0, *pfnt = 0, *pstring = 0, *pflags = 0;
                
                        /*
                                we have a list of tuples:
                                
-                               (x, y, width, height, fnt, flags, "bla" ),
-                               
+                               (0, x, y, width, height, fnt, flags, "bla" ),
+
+                               or, for a progress:
+                               (1, x, y, width, height, filled_percent )
+
                                or, for a pixmap:
                                
-                               (x, y, width, height, pixmap )
+                               (2, x, y, width, height, pixmap )
                                
                         */
                        
                        if (!PyTuple_Check(item))
                        {
                                eDebug("eListboxPythonMultiContent did not receive a tuple.");
-                               painter.clippop();
-                               return;
+                               goto error_out;
                        }
-                       
+
                        int size = PyTuple_Size(item);
-                       
-                       if (size >= 5)
+
+                       if (!size)
                        {
-                               px = PyTuple_GetItem(item, 0);
-                               py = PyTuple_GetItem(item, 1);
-                               pwidth = PyTuple_GetItem(item, 2);
-                               pheight = PyTuple_GetItem(item, 3);
-                       
-                               pfnt = PyTuple_GetItem(item, 4); /* could also be an pixmap */
-                               if (size >= 7)
-                               {
-                                       pflags = PyTuple_GetItem(item, 5);
-                                       pstring = PyTuple_GetItem(item, 6);
-                               }
+                               eDebug("eListboxPythonMultiContent receive empty tuple.");
+                               goto error_out;
                        }
-                       
-                       ePtr<gPixmap> pixmap;
-                       
-                               /* decide what type */
-                       int type = -1;
-                       if (pfnt)
+
+                       int type = PyInt_AsLong(PyTuple_GET_ITEM(item, 0));
+
+                       if (size > 5)
                        {
-                               if (PyNumber_Check(pfnt)) /* font index */
-                                       type = 0;
-                               else if (!SwigFromPython(pixmap, pfnt))
-                                       type = 1;
+                               px = PyTuple_GET_ITEM(item, 1);
+                               py = PyTuple_GET_ITEM(item, 2);
+                               pwidth = PyTuple_GET_ITEM(item, 3);
+                               pheight = PyTuple_GET_ITEM(item, 4);
+                               pfnt = PyTuple_GET_ITEM(item, 5); /* could also be an pixmap or an int (progress filled percent) */
+                               if (size > 7)
+                               {
+                                       pflags = PyTuple_GET_ITEM(item, 6);
+                                       pstring = PyTuple_GET_ITEM(item, 7);
+                               }
                        }
                        
                        switch (type)
                        {
-                       case 0: // text
+                       case TYPE_TEXT: // text
                        {
                                if (!(px && py && pwidth && pheight && pfnt && pstring))
                                {
-                                       eDebug("eListboxPythonMultiContent received too small tuple (must be (x, y, width, height, fnt, flags, string[, ...])");
-                                       painter.clippop();
-                                       return;
+                                       eDebug("eListboxPythonMultiContent received too small tuple (must be (TYPE_TEXT, x, y, width, height, fnt, flags, string[, ...])");
+                                       goto error_out;
                                }
-       
-                               pstring = PyObject_Str(pstring);
-                       
+                               
                                const char *string = (PyString_Check(pstring)) ? PyString_AsString(pstring) : "<not-a-string>";
-                       
                                int x = PyInt_AsLong(px);
                                int y = PyInt_AsLong(py);
                                int width = PyInt_AsLong(pwidth);
                                int height = PyInt_AsLong(pheight);
                                int flags = PyInt_AsLong(pflags);
-
                                int fnt = PyInt_AsLong(pfnt);
-
+                               
                                if (m_font.find(fnt) == m_font.end())
                                {
                                        eDebug("eListboxPythonMultiContent: specified font %d was not found!", fnt);
-                                       Py_XDECREF(pstring);
-                                       painter.clippop();
-                                       return;
+                                       goto error_out;
                                }
+                               
                                eRect r = eRect(x, y, width, height);
                                r.moveBy(offset);
                                r &= itemrect;
-
+                               
                                painter.setFont(m_font[fnt]);
-
+                               
                                painter.clip(r);
                                painter.renderText(r, string, flags);
+                               painter.clippop();
+                               break;
+                       }
+                       case TYPE_PROGRESS: // Progress
+                       {
+                               if (!(px && py && pwidth && pheight && pfnt))
+                               {
+                                       eDebug("eListboxPythonMultiContent received too small tuple (must be (TYPE_PROGRESS, x, y, width, height, filled percent))");
+                                       goto error_out;
+                               }
+                               int x = PyInt_AsLong(px);
+                               int y = PyInt_AsLong(py);
+                               int width = PyInt_AsLong(pwidth);
+                               int height = PyInt_AsLong(pheight);
+                               int filled = PyInt_AsLong(pfnt);
+
+                               eRect r = eRect(x, y, width, height);
+                               r.moveBy(offset);
+                               r &= itemrect;
+
+                               painter.clip(r);
+                               int bwidth=2;  // borderwidth hardcoded yet
+
+                               // border
+                               eRect rc = eRect(x, y, width, bwidth);
+                               rc.moveBy(offset);
+                               painter.fill(rc);
+
+                               rc = eRect(x, y+bwidth, bwidth, height-bwidth);
+                               rc.moveBy(offset);
+                               painter.fill(rc);
+
+                               rc = eRect(x+bwidth, y+height-bwidth, width-bwidth, bwidth);
+                               rc.moveBy(offset);
+                               painter.fill(rc);
+
+                               rc = eRect(x+width-bwidth, y+bwidth, bwidth, height-bwidth);
+                               rc.moveBy(offset);
+                               painter.fill(rc);
+
+                               // progress
+                               rc = eRect(x+bwidth, y+bwidth, (width-bwidth*2) * filled / 100, height-bwidth*2);
+                               rc.moveBy(offset);
+                               painter.fill(rc);
+
                                painter.clippop();
 
-                               Py_XDECREF(pstring);
                                break;
                        }
-                       case 1: // pixmap
+                       case TYPE_PIXMAP: // pixmap
                        {
                                if (!(px && py && pwidth && pheight && pfnt))
                                {
-                                       eDebug("eListboxPythonMultiContent received too small tuple (must be (x, y, width, height, pixmap))");
-                                       painter.clippop();
-                                       return;
+                                       eDebug("eListboxPythonMultiContent received too small tuple (must be (TYPE_PIXMAP, x, y, width, height, pixmap))");
+                                       goto error_out;
                                }
                                int x = PyInt_AsLong(px);
                                int y = PyInt_AsLong(py);
                                int width = PyInt_AsLong(pwidth);
                                int height = PyInt_AsLong(pheight);
-                               
+                               ePtr<gPixmap> pixmap;
+                               if (SwigFromPython(pixmap, pfnt))
+                               {
+                                       eDebug("eListboxPythonMultiContent (Pixmap) get pixmap failed");
+                                       goto error_out;
+                               }
+
                                eRect r = eRect(x, y, width, height);
                                r.moveBy(offset);
                                r &= itemrect;
@@ -697,9 +734,8 @@ void eListboxPythonMultiContent::paint(gPainter &painter, eWindowStyle &style, c
                                break;
                        }
                        default:
-                               eWarning("eListboxPythonMultiContent received neither text nor pixmap entry");
-                               painter.clippop();
-                               return;
+                               eWarning("eListboxPythonMultiContent received unknown type (%d)", type);
+                               goto error_out;
                        }
                }
        }
@@ -707,6 +743,7 @@ void eListboxPythonMultiContent::paint(gPainter &painter, eWindowStyle &style, c
        if (selected)
                style.drawFrame(painter, eRect(offset, m_itemsize), eWindowStyle::frameListboxEntry);
 
+error_out:
        painter.clippop();
 }