- buttonPixUp/Down are now vectors, too

- removed unused and uninitialized pixmaps array

2 files changed, 10 insertions, 18 deletions

diff --git a/noncore/multimedia/opieplayer2/audiowidget.cpp b/noncore/multimedia/opieplayer2/audiowidget.cpp
index fe7ddc1..2081b4f 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.cpp
+++ b/noncore/multimedia/opieplayer2/audiowidget.cpp
@@ -131,6 +131,4 @@ AudioWidget::AudioWidget( PlayListWidget &playList, MediaPlayerState &mediaPlaye
 
-    for ( int i = 0; i < 10; i++ ) {
-        buttonPixUp[i] = 0l;
-        buttonPixDown[i] = 0l;
-    }
+    buttonPixUp.resize( masks.size(), QPixmap() );
+    buttonPixDown.resize( masks.size(), QPixmap() );
 
@@ -182,6 +180,2 @@ AudioWidget::~AudioWidget() {
 
-    for ( int i = 0; i < 10; i++ ) {
-        delete buttonPixUp[i];
-        delete buttonPixDown[i];
-    }
 //    mediaPlayerState->setPlaying(false);
@@ -200,5 +194,5 @@ QPixmap combineImageWithBackground( QImage img, QPixmap bg, QPoint offset ) {
 
-QPixmap *maskPixToMask( QPixmap pix, QBitmap mask ) {
-    QPixmap *pixmap = new QPixmap( pix );
-    pixmap->setMask( mask );
+QPixmap maskPixToMask( QPixmap pix, QBitmap mask ) {
+    QPixmap pixmap( pix );
+    pixmap.setMask( mask );
     return pixmap;
@@ -227,4 +221,2 @@ void AudioWidget::resizeEvent( QResizeEvent * ) {
         if ( !masks[i].isNull() ) {
-            delete buttonPixUp[i];
-            delete buttonPixDown[i];
             buttonPixUp[i] = maskPixToMask( pixUp, masks[i] );
@@ -334,5 +326,5 @@ void AudioWidget::paintButton( QPainter &p, int i ) {
     if ( buttons[i].isDown ) {
-        p.drawPixmap( upperLeftOfButtonMask, *buttonPixDown[i] );
+        p.drawPixmap( upperLeftOfButtonMask, buttonPixDown[i] );
     } else {
-        p.drawPixmap( upperLeftOfButtonMask, *buttonPixUp[i] );
+        p.drawPixmap( upperLeftOfButtonMask, buttonPixUp[i] );
     }
diff --git a/noncore/multimedia/opieplayer2/audiowidget.h b/noncore/multimedia/opieplayer2/audiowidget.h
index bc7262c..73ce1e5 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.h
+++ b/noncore/multimedia/opieplayer2/audiowidget.h
@@ -94,2 +94,3 @@ private:
     typedef std::vector<QBitmap> MaskVector;
+    typedef std::vector<QPixmap> PixmapVector;
 
@@ -103,6 +104,5 @@ private:
     MaskVector masks;
-    QPixmap *buttonPixUp[10];
-    QPixmap *buttonPixDown[10];
+    PixmapVector buttonPixUp;
+    PixmapVector buttonPixDown;
 
-    QPixmap *pixmaps[4];
     OTicker  songInfo;