- collect arguments passed to the skin loading into a structure

6 files changed, 24 insertions, 24 deletions

diff --git a/noncore/multimedia/opieplayer2/audiowidget.cpp b/noncore/multimedia/opieplayer2/audiowidget.cpp
index 4db444a..cd5bea4 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.cpp
+++ b/noncore/multimedia/opieplayer2/audiowidget.cpp
@@ -110,16 +110,11 @@ AudioWidget::~AudioWidget() {
 
 //    mediaPlayerState->setPlaying(false);
 }
 
-const MediaWidget::SkinButtonInfo *AudioWidget::skinInfo()
+MediaWidget::GUIInfo AudioWidget::guiInfo()
 {
-    return ::skinInfo;
-}
-
-const uint AudioWidget::buttonCount()
-{
-    return ::buttonCount;
+    return GUIInfo( QString::null /* infix */, ::skinInfo, ::buttonCount );
 }
 
 void AudioWidget::resizeEvent( QResizeEvent *e ) {
     int h = height();
@@ -173,9 +168,9 @@ void AudioWidget::setDisplayType( MediaPlayerState::DisplayType mediaType ) {
 }
 
 void AudioWidget::loadSkin()
 {
-    loadDefaultSkin( ::skinInfo, ::buttonCount );
+    loadDefaultSkin( guiInfo() );
 
     setBackgroundPixmap( backgroundPixmap );
 
     songInfo.setFocusPolicy( QWidget::NoFocus );
diff --git a/noncore/multimedia/opieplayer2/audiowidget.h b/noncore/multimedia/opieplayer2/audiowidget.h
index 75d4857..53dbb88 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.h
+++ b/noncore/multimedia/opieplayer2/audiowidget.h
@@ -55,10 +55,9 @@ public:
     AudioWidget( PlayListWidget &playList, MediaPlayerState &mediaPlayerState, QWidget* parent=0, const char* name=0 );
     ~AudioWidget();
     void setTickerText( const QString &text ) { songInfo.setText( text ); }
 
-    static const MediaWidget::SkinButtonInfo *skinInfo();
-    static const uint buttonCount();
+    static MediaWidget::GUIInfo guiInfo();
 
 public slots:
     void updateSlider( long, long );
     void sliderPressed( );
diff --git a/noncore/multimedia/opieplayer2/mediawidget.cpp b/noncore/multimedia/opieplayer2/mediawidget.cpp
index ab2ec50..c0106d4 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.cpp
+++ b/noncore/multimedia/opieplayer2/mediawidget.cpp
@@ -63,12 +63,12 @@ MediaWidget::Button MediaWidget::setupButton( const SkinButtonInfo &buttonInfo,
 
     return button;
 }
 
-void MediaWidget::loadDefaultSkin( const SkinButtonInfo *skinInfo, uint buttonCount, const QString &fileNameInfix )
+void MediaWidget::loadDefaultSkin( const GUIInfo &guiInfo )
 {
-    Skin skin( fileNameInfix );
-    loadSkin( skinInfo, buttonCount, skin );
+    Skin skin( guiInfo.fileNameInfix );
+    loadSkin( guiInfo.buttonInfo, guiInfo.buttonCount, skin );
 }
 
 void MediaWidget::loadSkin( const SkinButtonInfo *skinInfo, uint buttonCount, const Skin &skin )
 {
diff --git a/noncore/multimedia/opieplayer2/mediawidget.h b/noncore/multimedia/opieplayer2/mediawidget.h
index 4599637..64483d8 100644
--- a/noncore/multimedia/opieplayer2/mediawidget.h
+++ b/noncore/multimedia/opieplayer2/mediawidget.h
@@ -79,8 +79,20 @@ public:
         const char *fileName;
         ButtonType type;
     };
 
+    struct GUIInfo
+    {
+        GUIInfo() : buttonInfo( 0 ), buttonCount( 0 ) {}
+        GUIInfo( const QString &_fileNameInfix, const SkinButtonInfo *_buttonInfo, const uint _buttonCount )
+            : fileNameInfix( _fileNameInfix ), buttonInfo( _buttonInfo ), buttonCount( _buttonCount )
+        {}
+
+        QString fileNameInfix;
+        const SkinButtonInfo *buttonInfo;
+        const uint buttonCount;
+    };
+
     MediaWidget( PlayListWidget &_playList, MediaPlayerState &_mediaPlayerState, QWidget *parent = 0, const char *name = 0 );
     virtual ~MediaWidget();
 
 public slots:
@@ -104,9 +116,9 @@ protected:
     void setupButtons( const SkinButtonInfo *skinInfo, uint buttonCount,
                        const Skin &skin );
     Button setupButton( const SkinButtonInfo &buttonInfo, const Skin &skin );
 
-    void loadDefaultSkin( const SkinButtonInfo *skinInfo, uint buttonCount, const QString &fileNameInfix = QString::null );
+    void loadDefaultSkin( const GUIInfo &guiInfo );
     void loadSkin( const SkinButtonInfo *skinInfo, uint buttonCount, const Skin &skin );
 
     virtual void closeEvent( QCloseEvent * );
 
diff --git a/noncore/multimedia/opieplayer2/videowidget.cpp b/noncore/multimedia/opieplayer2/videowidget.cpp
index 058730f..2a345e3 100644
--- a/noncore/multimedia/opieplayer2/videowidget.cpp
+++ b/noncore/multimedia/opieplayer2/videowidget.cpp
@@ -98,16 +98,11 @@ VideoWidget::VideoWidget( PlayListWidget &playList, MediaPlayerState &mediaPlaye
 VideoWidget::~VideoWidget() 
 {
 }
 
-const MediaWidget::SkinButtonInfo *VideoWidget::skinInfo()
+MediaWidget::GUIInfo VideoWidget::guiInfo()
 {
-    return ::skinInfo;
-}
-
-const uint VideoWidget::buttonCount()
-{
-    return ::buttonCount;
+    return GUIInfo( "V" /* infix */, ::skinInfo, ::buttonCount );
 }
 
 void VideoWidget::resizeEvent( QResizeEvent *e ) {
     int h = height();
@@ -166,9 +161,9 @@ void VideoWidget::setDisplayType( MediaPlayerState::DisplayType displayType )
 }
 
 void VideoWidget::loadSkin()
 {
-    loadDefaultSkin( ::skinInfo, ::buttonCount, "V" );
+    loadDefaultSkin( guiInfo() );
 
     setBackgroundPixmap( backgroundPixmap );
 
     delete slider;
diff --git a/noncore/multimedia/opieplayer2/videowidget.h b/noncore/multimedia/opieplayer2/videowidget.h
index 80e9356..84bc964 100644
--- a/noncore/multimedia/opieplayer2/videowidget.h
+++ b/noncore/multimedia/opieplayer2/videowidget.h
@@ -49,10 +49,9 @@ class VideoWidget : public MediaWidget {
 public:
     VideoWidget( PlayListWidget &playList, MediaPlayerState &mediaPlayerState, QWidget* parent=0, const char* name=0 );
     ~VideoWidget();
 
-    static const MediaWidget::SkinButtonInfo *skinInfo();
-    static const uint buttonCount();
+    static MediaWidget::GUIInfo guiInfo();
 
     XineVideoWidget* vidWidget();
 public slots:
     void updateSlider( long, long );