widget: don't crash when deallocating in wrong order

[enigma2.git] / lib / gui / ewidget.cpp

diff --git a/lib/gui/ewidget.cpp b/lib/gui/ewidget.cpp
index f71d7e1d503e5e85e5070277c88baeb27f57f075..3bc6e663841e940e1d7506f7baf4722c90f01a24 100644 (file)
--- a/lib/gui/ewidget.cpp
+++ b/lib/gui/ewidget.cpp
@@ -32,6 +32,9 @@ void eWidget::move(ePoint pos)
        if (m_position == pos)
                return;
 
        if (m_position == pos)
                return;
 

+                       /* ?? what about native move support? */
+       invalidate();
+

        m_position = pos;
        
        event(evtChangedPosition);
        m_position = pos;
        
        event(evtChangedPosition);


@@ -137,6 +140,8 @@ void eWidget::hide()
        while (root && !root->m_desktop)
        {
                root = root->m_parent;
        while (root && !root->m_desktop)
        {
                root = root->m_parent;

+               if (!root)
+                       return;

                abspos += root->position();
        }
        assert(root->m_desktop);
                abspos += root->position();
        }
        assert(root->m_desktop);


@@ -177,6 +182,14 @@ void eWidget::setZPosition(int z)
        insertIntoParent(); /* now at the new Z position */
 }
 
        insertIntoParent(); /* now at the new Z position */
 }
 

+void eWidget::setTransparent(int transp)
+{
+       if (transp)
+               m_vis |= wVisTransparent;
+       else
+               m_vis &=~wVisTransparent;
+}
+

 void eWidget::mayKillFocus()
 {
        setFocus(0);
 void eWidget::mayKillFocus()
 {
        setFocus(0);


@@ -194,12 +207,11 @@ eWidget::~eWidget()
 
        m_parent = 0;
 
 
        m_parent = 0;
 

-               /* destroy all childs */
+               /* tell all childs that the parent is not anymore existing */

        ePtrList<eWidget>::iterator i(m_childs.begin());
        while (i != m_childs.end())
        {
        ePtrList<eWidget>::iterator i(m_childs.begin());
        while (i != m_childs.end())
        {

-               (*i)->m_parent = 0;
-               delete *i;
+               (*i)->parentRemoved();

                i = m_childs.erase(i);
        }
 }
                i = m_childs.erase(i);
        }
 }


@@ -215,6 +227,7 @@ void eWidget::insertIntoParent()
                        m_parent->m_childs.insert(i, this);
                        return;
                }
                        m_parent->m_childs.insert(i, this);
                        return;
                }

+               ++i;

        }
 }
 
        }
 }
 


@@ -223,14 +236,11 @@ void eWidget::doPaint(gPainter &painter, const gRegion &r)
        if (m_visible_with_childs.empty())
                return;
        
        if (m_visible_with_childs.empty())
                return;
        

-       gRegion region = r;
+       gRegion region = r, childs = r;

                        /* we were in parent's space, now we are in local space */
        region.moveBy(-position());
        
        painter.moveOffset(position());
                        /* we were in parent's space, now we are in local space */
        region.moveBy(-position());
        
        painter.moveOffset(position());

-               /* walk all childs */
-       for (ePtrList<eWidget>::iterator i(m_childs.begin()); i != m_childs.end(); ++i)
-               i->doPaint(painter, region);

        
                /* check if there's anything for us to paint */
        region &= m_visible_region;
        
                /* check if there's anything for us to paint */
        region &= m_visible_region;


@@ -240,6 +250,11 @@ void eWidget::doPaint(gPainter &painter, const gRegion &r)
                painter.resetClip(region);
                event(evtPaint, &region, &painter);
        }
                painter.resetClip(region);
                event(evtPaint, &region, &painter);
        }

+
+       childs.moveBy(-position());
+               /* walk all childs */
+       for (ePtrList<eWidget>::iterator i(m_childs.begin()); i != m_childs.end(); ++i)
+               i->doPaint(painter, childs);

        
        painter.moveOffset(-position());
 }
        
        painter.moveOffset(-position());
 }


@@ -261,6 +276,11 @@ void eWidget::recalcClipRegionsWhenVisible()
        } while(1);
 }
 
        } while(1);
 }
 

+void eWidget::parentRemoved()
+{
+       m_parent = 0;
+}
+

 int eWidget::event(int event, void *data, void *data2)
 {
        switch (event)
 int eWidget::event(int event, void *data, void *data2)
 {
        switch (event)


@@ -271,15 +291,22 @@ int eWidget::event(int event, void *data, void *data2)
                
 //             eDebug("eWidget::evtPaint");
 //             dumpRegion(*(gRegion*)data);
                
 //             eDebug("eWidget::evtPaint");
 //             dumpRegion(*(gRegion*)data);

-               if (!m_have_background_color)
+               if (!isTransparent())

                {
                {

-                       ePtr<eWindowStyle> style;
-                       if (!getStyle(style))
-                               style->paintBackground(painter, ePoint(0, 0), size());
+                       if (!m_have_background_color)
+                       {
+                               ePtr<eWindowStyle> style;
+                               if (!getStyle(style))
+                                       style->paintBackground(painter, ePoint(0, 0), size());
+                       } else
+                       {
+                               painter.setBackgroundColor(m_background_color);
+                               painter.clear();
+                       }

                } else
                {
                } else
                {

-                       painter.setBackgroundColor(m_background_color);
-                       painter.clear();
+                       if (m_have_background_color)
+                               painter.setBackgroundColor(m_background_color);

                }
                break;
        }
                }
                break;
        }