4 files changed, 19 insertions, 17 deletions

diff --git a/noncore/multimedia/opieplayer2/audiowidget.cpp b/noncore/multimedia/opieplayer2/audiowidget.cpp
index 714509e..c3e206c 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.cpp
+++ b/noncore/multimedia/opieplayer2/audiowidget.cpp
@@ -104,6 +104,7 @@ AudioWidget::AudioWidget( PlayListWidget &playList, MediaPlayerState &mediaPlaye
 
     for ( uint i = 0; i < buttonCount; i++ ) {
         Button button;
+        button.command = skinInfo[ i ].command;
         button.type = skinInfo[ i ].type;
 
         QString filename = QString( QPEApplication::qpeDir()  + "/pics/" + skinPath + "/skin_mask_" + skinInfo[i].fileName + ".png" );
@@ -120,7 +121,7 @@ AudioWidget::AudioWidget( PlayListWidget &playList, MediaPlayerState &mediaPlaye
             }
         }
 
-        buttons.insert( i, button );
+        buttons.push_back( button );
     }
 
     setBackgroundPixmap( backgroundPixmap );
@@ -208,7 +209,7 @@ void AudioWidget::resizeEvent( QResizeEvent * ) {
     QPixmap pixUp = combineImageWithBackground( imgUp, backgroundPixmap, p );
     QPixmap pixDn = combineImageWithBackground( imgDn, backgroundPixmap, p );
 
-    for ( uint i = 0; i < buttons.count(); i++ ) {
+    for ( uint i = 0; i < buttons.size(); i++ ) {
         if ( !buttons[i].mask.isNull() ) {
             buttons[i].pixUp = maskPixToMask( pixUp, buttons[i].mask );
             buttons[i].pixDown = maskPixToMask( pixDn, buttons[i].mask );
@@ -334,7 +335,7 @@ void AudioWidget::timerEvent( QTimerEvent * ) {
 
 
 void AudioWidget::mouseMoveEvent( QMouseEvent *event ) {
-    for ( unsigned int i = 0; i < buttons.count(); i++ ) {
+    for ( unsigned int i = 0; i < buttons.size(); i++ ) {
 
         Button &button = buttons[ i ];
 
diff --git a/noncore/multimedia/opieplayer2/mediawidget.cpp b/noncore/multimedia/opieplayer2/mediawidget.cpp
index 66129c9..7891a7e 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.cpp
+++ b/noncore/multimedia/opieplayer2/mediawidget.cpp
@@ -82,6 +82,7 @@ void MediaWidget::handleCommand( Command command, bool buttonDown )
         case PlayList:   mediaPlayerState.setList();  return;
         case Forward:    emit forwardReleased(); return;
         case Back:       emit backReleased(); return;
+        default: assert( false );
     }
 }
 
@@ -95,7 +96,7 @@ bool MediaWidget::isOverButton( const QPoint &position, int buttonId ) const
 
 void MediaWidget::paintAllButtons( QPainter &p )
 {
-    for ( ButtonMap::ConstIterator it = buttons.begin();
+    for ( ButtonVector::const_iterator it = buttons.begin();
           it != buttons.end(); ++it )
         paintButton( p, *it );
 }
diff --git a/noncore/multimedia/opieplayer2/mediawidget.h b/noncore/multimedia/opieplayer2/mediawidget.h
index 05b7b71..efba6f3 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.h
+++ b/noncore/multimedia/opieplayer2/mediawidget.h
@@ -35,12 +35,14 @@ class MediaWidget : public QWidget
 {
     Q_OBJECT
 public:
-    enum Command { Play = 0, Stop, Next, Previous, VolumeUp, VolumeDown, Loop, PlayList, Forward, Back };
+    enum Command { Play = 0, Stop, Next, Previous, VolumeUp, VolumeDown, Loop, PlayList, Forward, Back, Undefined };
     enum ButtonType { NormalButton, ToggleButton };
 
     struct Button
     {
-        Button() : type( NormalButton ), isHeld( false ), isDown( false ) {}
+        Button() : command( Undefined ), type( NormalButton ), isHeld( false ), isDown( false ) {}
+
+        Command command;
 
         ButtonType type : 1;
         bool isHeld : 1;
@@ -51,8 +53,6 @@ public:
         QPixmap pixDown;
     };
     typedef std::vector<Button> ButtonVector;
-    // when the transition is done this should be Command -> Button
-    typedef QMap<int, Button> ButtonMap;
 
     struct SkinButtonInfo
     {
@@ -97,7 +97,7 @@ protected:
     MediaPlayerState &mediaPlayerState;
     PlayListWidget &playList;
 
-    ButtonMap buttons;
+    ButtonVector buttons;
 
     QImage buttonMask;
 
diff --git a/noncore/multimedia/opieplayer2/videowidget.cpp b/noncore/multimedia/opieplayer2/videowidget.cpp
index 9782b68..f59772e 100644
--- a/noncore/multimedia/opieplayer2/videowidget.cpp
+++ b/noncore/multimedia/opieplayer2/videowidget.cpp
@@ -74,13 +74,13 @@ VideoWidget::VideoWidget( PlayListWidget &playList, MediaPlayerState &mediaPlaye
     Button toggleButton = defaultButton;
     toggleButton.type = ToggleButton;
 
-    buttons.insert( 0, toggleButton ); // play
-    buttons.insert( 1, toggleButton ); // stop
-    buttons.insert( 2, toggleButton ); // next
-    buttons.insert( 3, toggleButton ); // previous
-    buttons.insert( 4, toggleButton ); // volUp
-    buttons.insert( 5, toggleButton ); // volDown
-    buttons.insert( 6, toggleButton ); // fullscreen
+    buttons.push_back( toggleButton ); // play
+    buttons.push_back( toggleButton ); // stop
+    buttons.push_back( toggleButton ); // next
+    buttons.push_back( toggleButton ); // previous
+    buttons.push_back( toggleButton ); // volUp
+    buttons.push_back( toggleButton ); // volDown
+    buttons.push_back( toggleButton ); // fullscreen
 
     videoFrame = new XineVideoWidget ( this, "Video frame" );
 
@@ -243,7 +243,7 @@ void VideoWidget::updateSlider( long i, long max ) {
 }
 
 void VideoWidget::mouseMoveEvent( QMouseEvent *event ) {
-    for ( unsigned int i = 0; i < buttons.count(); i++ ) {
+    for ( unsigned int i = 0; i < buttons.size(); i++ ) {
 
         Button &button = buttons[ i ];