- minor cleanup in paintEvent

2 files changed, 10 insertions, 5 deletions

diff --git a/noncore/multimedia/opieplayer2/mediawidget.cpp b/noncore/multimedia/opieplayer2/mediawidget.cpp
index b73a5c0..271f788 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.cpp
+++ b/noncore/multimedia/opieplayer2/mediawidget.cpp
@@ -60,15 +60,12 @@ void MediaWidget::paintEvent( QPaintEvent *pe )
         QPainter p( &pix );
         p.translate( -pe->rect().topLeft().x(), -pe->rect().topLeft().y() );
         p.drawTiledPixmap( pe->rect(), backgroundPixmap, pe->rect().topLeft() );
-        for ( unsigned int i = 0; i < buttons.count(); i++ ) {
-            paintButton( p, buttons[ i ] );
-        }
+        paintAllButtons( p );
         QPainter p2( this );
         p2.drawPixmap( pe->rect().topLeft(), pix );
     } else {
         QPainter p( this );
-        for ( unsigned int i = 0; i < buttons.count(); i++ )
-            paintButton( p, buttons[ i ] );
+        paintAllButtons( p );
     }
 }
 
@@ -96,6 +93,13 @@ bool MediaWidget::isOverButton( const QPoint &position, int buttonId ) const
              buttonMask.pixelIndex( position.x(), position.y() ) == buttonId + 1 );
 }
 
+void MediaWidget::paintAllButtons( QPainter &p )
+{
+    for ( ButtonMap::ConstIterator it = buttons.begin();
+          it != buttons.end(); ++it )
+        paintButton( *it );
+}
+
 void MediaWidget::paintButton( const Button &button )
 {
     QPainter p( this );
diff --git a/noncore/multimedia/opieplayer2/mediawidget.h b/noncore/multimedia/opieplayer2/mediawidget.h
index dcf7fb8..34910fb 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.h
+++ b/noncore/multimedia/opieplayer2/mediawidget.h
@@ -87,6 +87,7 @@ protected:
 
     bool isOverButton( const QPoint &position, int buttonId ) const;
 
+    void paintAllButtons( QPainter &p );
     void paintButton( const Button &button );
     void paintButton( QPainter &p, const Button &button );