much more code cleanup, make if look more like c++ code

7 files changed, 65 insertions, 125 deletions

diff --git a/noncore/multimedia/opieplayer2/audiowidget.cpp b/noncore/multimedia/opieplayer2/audiowidget.cpp
index 8d3963a..1b0de5d 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.cpp
+++ b/noncore/multimedia/opieplayer2/audiowidget.cpp
@@ -15,11 +15,48 @@
 
 extern MediaPlayerState *mediaPlayerState;
 
-
 static const int xo = -2; // movable x offset
 static const int yo = 22; // movable y offset
 
 
+Ticker::Ticker( QWidget* parent=0 ) : QFrame( parent ) {
+        setFrameStyle( WinPanel | Sunken );
+        setText( "No Song" );
+}
+
+Ticker::~Ticker() {
+}
+
+void Ticker::setText( const QString& text ) {
+    pos = 0; // reset it everytime the text is changed
+    scrollText = text;
+    pixelLen = fontMetrics().width( scrollText );
+    killTimers();
+    if ( pixelLen > width() ) {
+        startTimer( 50 );
+    }
+    update();
+}
+
+
+void Ticker::timerEvent( QTimerEvent * ) {
+    pos = ( pos + 1 > pixelLen ) ? 0 : pos + 1;
+    repaint( FALSE );
+}
+
+void Ticker::drawContents( QPainter *p ) {
+    QPixmap pm( width(), height() );
+    pm.fill( colorGroup().base() );
+    QPainter pmp( &pm );
+    for ( int i = 0; i - pos < width() && (i < 1 || pixelLen > width()); i += pixelLen ) {
+        pmp.drawText( i - pos, 0, INT_MAX, height(), AlignVCenter, scrollText );
+    }
+    p->drawPixmap( 0, 0, pm );
+}
+
+
+
+
 struct MediaButton {
     int  xPos, yPos;
     int  color;
@@ -27,6 +64,8 @@ struct MediaButton {
 };
 
 
+
+
 // Layout information for the audioButtons (and if it is a toggle button or not)
 MediaButton audioButtons[] = {
     { 3*30-15+xo, 3*30-13+yo, 0,  TRUE,  TRUE, FALSE, FALSE }, // play
@@ -40,7 +79,6 @@ MediaButton audioButtons[] = {
     {    1*30+xo,    1*30+yo, 0, FALSE, FALSE, FALSE, FALSE }  // playlist
 };
 
-
 static const int numButtons = (sizeof(audioButtons)/sizeof(MediaButton));
 
 
@@ -145,7 +183,6 @@ void AudioWidget::setView( char view ) {
 
     if ( view == 'a' ) {
         startTimer( 150 );
-        // show();
         showMaximized();
     } else {
         killTimers();
diff --git a/noncore/multimedia/opieplayer2/audiowidget.h b/noncore/multimedia/opieplayer2/audiowidget.h
index eab5df4..441eb6e 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.h
+++ b/noncore/multimedia/opieplayer2/audiowidget.h
@@ -30,45 +30,15 @@ enum AudioButtons {
 
 class Ticker : public QFrame {
     Q_OBJECT
+
 public:
-    Ticker( QWidget* parent=0 ) : QFrame( parent ) {
-        setFrameStyle( WinPanel | Sunken );
-        setText( "No Song" );
-    }
-    ~Ticker() { }
-    void setText( const QString& text ) {
-        pos = 0; // reset it everytime the text is changed
-        scrollText = text;
-        pixelLen = fontMetrics().width( scrollText );
-        killTimers();
-        if ( pixelLen > width() )
-            startTimer( 50 );
-        update();
-    }
+    Ticker( QWidget* parent=0 );
+    ~Ticker();
+    void setText( const QString& text ) ;
+
 protected:
-    void timerEvent( QTimerEvent * ) {
-  pos = ( pos + 1 > pixelLen ) ? 0 : pos + 1;
-#ifndef USE_DBLBUF
-  scroll( -1, 0, contentsRect() );
-#else
-  repaint( FALSE );
-#endif
-    }
-    void drawContents( QPainter *p ) {
-#ifndef USE_DBLBUF
-  for ( int i = 0; i - pos < width() && (i < 1 || pixelLen > width()); i += pixelLen )
-      p->drawText( i - pos, 0, INT_MAX, height(), AlignVCenter, scrollText );
-#else
-  // Double buffering code.
-  // Looks like qvfb makes it look like it flickers but I don't think it really is
-  QPixmap pm( width(), height() );
-  pm.fill( colorGroup().base() );
-  QPainter pmp( &pm );
-  for ( int i = 0; i - pos < width() && (i < 1 || pixelLen > width()); i += pixelLen )
-      pmp.drawText( i - pos, 0, INT_MAX, height(), AlignVCenter, scrollText );
-  p->drawPixmap( 0, 0, pm );
-#endif
-    }
+    void timerEvent( QTimerEvent * );
+    void drawContents( QPainter *p );
 private:
     QString scrollText;
     int pos, pixelLen;
diff --git a/noncore/multimedia/opieplayer2/inputDialog.cpp b/noncore/multimedia/opieplayer2/inputDialog.cpp
index da8e276..687aff6 100644
--- a/noncore/multimedia/opieplayer2/inputDialog.cpp
+++ b/noncore/multimedia/opieplayer2/inputDialog.cpp
@@ -1,11 +1,3 @@
-/****************************************************************************
-** Form implementation generated from reading ui file 'inputDialog.ui'
-**
-** Created: Sat Mar 2 07:55:03 2002
-**      by:  The User Interface Compiler (uic)
-**
-** WARNING! All changes made in this file will be lost!
-****************************************************************************/
 #include "inputDialog.h"
 
 #include <qpe/resource.h>
diff --git a/noncore/multimedia/opieplayer2/inputDialog.h b/noncore/multimedia/opieplayer2/inputDialog.h
index 3e3e36f..d4f5e12 100644
--- a/noncore/multimedia/opieplayer2/inputDialog.h
+++ b/noncore/multimedia/opieplayer2/inputDialog.h
@@ -1,11 +1,4 @@
-/****************************************************************************
-** Form interface generated from reading ui file 'inputDialog.ui'
-**
-** Created: Sat Mar 2 07:54:46 2002
-**      by:  The User Interface Compiler (uic)
-**
-** WARNING! All changes made in this file will be lost!
-****************************************************************************/
+
 #ifndef INPUTDIALOG_H
 #define INPUTDIALOG_H
 
@@ -14,8 +7,7 @@
 
 class QLineEdit;
 
-class InputDialog : public QDialog
-{ 
+class InputDialog : public QDialog {
     Q_OBJECT
 
 public:
diff --git a/noncore/multimedia/opieplayer2/mediaplayer.cpp b/noncore/multimedia/opieplayer2/mediaplayer.cpp
index e6d0525..87184ba 100644
--- a/noncore/multimedia/opieplayer2/mediaplayer.cpp
+++ b/noncore/multimedia/opieplayer2/mediaplayer.cpp
@@ -54,69 +54,29 @@ void MediaPlayer::play() {
 
 void MediaPlayer::setPlaying( bool play ) {
     if ( !play ) {
-  mediaPlayerState->setPaused( FALSE );
-//  loopControl->stop( FALSE );
-  return;
+        mediaPlayerState->setPaused( FALSE );
+        return;
     }
 
     if ( mediaPlayerState->paused() ) {
-  mediaPlayerState->setPaused( FALSE );
-  return;
+        mediaPlayerState->setPaused( FALSE );
+        return;
     }
 
     const DocLnk *playListCurrent = playList->current();
     if ( playListCurrent != NULL ) {
-//  loopControl->stop( TRUE );
-  currentFile = playListCurrent;
-    }
-
-    /*
-
-    if ( currentFile == NULL ) {
-  QMessageBox::critical( 0, tr( "No file"), tr( "Error: There is no file selected" ) );
-  mediaPlayerState->setPlaying( FALSE );
-  return;
-    }
-
-    if ( ((currentFile->file()).left(4) != "http") && !QFile::exists( currentFile->file() ) ) {
-  QMessageBox::critical( 0, tr( "File not found"), tr( "The following file was not found: <i>" ) + currentFile->file() + "</i>" );
-  mediaPlayerState->setPlaying( FALSE );
-  return;
-    }
-
-    if ( !mediaPlayerState->newDecoder( currentFile->file() ) ) {
-  QMessageBox::critical( 0, tr( "No decoder found"), tr( "Sorry, no appropriate decoders found for this file: <i>" ) + currentFile->file() + "</i>" );
-  mediaPlayerState->setPlaying( FALSE );
-  return;
+        currentFile = playListCurrent;
     }
 
-//    if ( !loopControl->init( currentFile->file() ) ) {
-//  QMessageBox::critical( 0, tr( "Error opening file"), tr( "Sorry, an error occured trying to play the file: <i>" ) + currentFile->file() + "</i>" );
-//  mediaPlayerState->setPlaying( FALSE );
-//  return;
-//    }
-//   long seconds = loopControl->totalPlaytime();
-    long seconds = 120;
-    QString time;
-   time.sprintf("%li:%02i", seconds/60, (int)seconds%60 );
-    QString tickerText;
-    if( currentFile->file().left(4) == "http" )
-     tickerText= tr( " File: " ) + currentFile->name();
-    else
-    tickerText = tr( " File: " ) + currentFile->name() + tr(", Length: ") + time;
+    audioUI->setTickerText( currentFile->file() );
 
-    QString fileInfo = mediaPlayerState->curDecoder()->fileInfo();
-    if ( !fileInfo.isEmpty() )
-  tickerText += ", " + fileInfo;
-    audioUI->setTickerText( tickerText + "." );
 
+    // alles nicht nötig, xine kümmert sich drum, man muss nur den return andio oder video gui geben
 
-    */ // alles nicht nötig, xine kümmert sich drum, man muss nur den return andio oder video gui geben
+    // Ob auch video 'v' : 'a'
+    //  mediaPlayerState->setView( 'v' );
 
-
-    //   loopControl->play();
-
-    //  mediaPlayerState->setView( loopControl->hasVideo() ? 'v' : 'a' );
+    // abspielen starten.
 }
 
 
@@ -145,7 +105,7 @@ void MediaPlayer::next() {
 void MediaPlayer::startDecreasingVolume() {
     volumeDirection = -1;
     startTimer( 100 );
-    // sollte volumeapplet machen
+    // da kommt demnächst osound denk ich mal
     //  AudioDevice::decreaseVolume();
 }
 
@@ -163,9 +123,9 @@ void MediaPlayer::stopChangingVolume() {
 
 
 void MediaPlayer::timerEvent( QTimerEvent * ) {
-//    if ( volumeDirection == +1 )
-//  AudioDevice::increaseVolume();
-//    else if ( volumeDirection == -1 )
+    //    if ( volumeDirection == +1 )
+    //  AudioDevice::increaseVolume();
+    //    else if ( volumeDirection == -1 )
         //       AudioDevice::decreaseVolume();
 }
 
diff --git a/noncore/multimedia/opieplayer2/mediaplayerstate.cpp b/noncore/multimedia/opieplayer2/mediaplayerstate.cpp
index 9b5f70e..d1d30e4 100644
--- a/noncore/multimedia/opieplayer2/mediaplayerstate.cpp
+++ b/noncore/multimedia/opieplayer2/mediaplayerstate.cpp
@@ -12,20 +12,12 @@
 
 
 
-#ifdef QT_NO_COMPONENT
-// Plugins which are compiled in when no plugin architecture available
-#include "libmad/libmadpluginimpl.h"
-#include "libmpeg3/libmpeg3pluginimpl.h"
-#include "wavplugin/wavpluginimpl.h"
-#endif
-
-
 //#define MediaPlayerDebug(x) qDebug x
 #define MediaPlayerDebug(x)
 
 
 MediaPlayerState::MediaPlayerState( QObject *parent, const char *name )
-        : QObject( parent, name ), decoder( NULL ), libmpeg3decoder( NULL ) {
+        : QObject( parent, name ), decoder( NULL ) {
     Config cfg( "OpiePlayer" );
     readConfig( cfg );
 }
diff --git a/noncore/multimedia/opieplayer2/mediaplayerstate.h b/noncore/multimedia/opieplayer2/mediaplayerstate.h
index 374e780..8a64939 100644
--- a/noncore/multimedia/opieplayer2/mediaplayerstate.h
+++ b/noncore/multimedia/opieplayer2/mediaplayerstate.h
@@ -30,8 +30,7 @@ public:
 
     MediaPlayerDecoder *newDecoder( const QString& file );
     MediaPlayerDecoder *curDecoder();
-    MediaPlayerDecoder *libMpeg3Decoder(); // ### Yucky hack needed to use libmpeg3plugin to get the
-             // number of audio samples if we are using the libmad plugin
+
 public slots:
     void setFullscreen( bool b ) { if ( isFullscreen == b ) return; isFullscreen = b; emit fullscreenToggled(b); }
     void setScaled( bool b )     { if ( isScaled     == b ) return; isScaled = b;     emit scaledToggled(b); }
@@ -88,8 +87,6 @@ private:
     char curView;
 
     MediaPlayerDecoder *decoder;
-    MediaPlayerDecoder *libmpeg3decoder;
-//    MediaPlayerDecoder *libwavdecoder;
 
     void readConfig( Config& cfg );
     void writeConfig( Config& cfg ) const;