mediadetect no longer needed, also beginning of id3 etc tag info support along with some other small fixes

8 files changed, 229 insertions, 101 deletions

diff --git a/noncore/multimedia/opieplayer2/audiowidget.cpp b/noncore/multimedia/opieplayer2/audiowidget.cpp
index 620c71f..a6fd334 100644
--- a/noncore/multimedia/opieplayer2/audiowidget.cpp
+++ b/noncore/multimedia/opieplayer2/audiowidget.cpp
@@ -60,3 +60,3 @@ Ticker::Ticker( QWidget* parent=0 ) : QFrame( parent ) {
         setFrameStyle( WinPanel | Sunken );
-        setText( "No Song" );
+        //setText( "No Song" );
 }
@@ -462,3 +462,2 @@ void AudioWidget::mouseMoveEvent( QMouseEvent *event ) {
                     if( mediaPlayerState->isPaused ) {
-//                          setToggleButton( i, FALSE );
                         mediaPlayerState->setPaused( FALSE );
@@ -466,12 +465,8 @@ void AudioWidget::mouseMoveEvent( QMouseEvent *event ) {
                     } else if( !mediaPlayerState->isPaused ) {
-//                            setToggleButton( i, TRUE );
                         mediaPlayerState->setPaused( TRUE );
                         return;
-                    } else {
-                        //  setToggleButton( i, TRUE );
-                        // mediaPlayerState->setPlaying( videoButtons[i].isDown );
                     }
                 case AudioStop:       mediaPlayerState->setPlaying(FALSE); return;
-                  case AudioNext:  if(playList->whichList() ==0) mediaPlayerState->setNext(); return;
-                case AudioPrevious:  if(playList->whichList() ==0) mediaPlayerState->setPrev(); return;
+                case AudioNext:       if( playList->whichList() ==0 ) mediaPlayerState->setNext(); return;
+                case AudioPrevious:   if( playList->whichList() ==0 ) mediaPlayerState->setPrev(); return;
                 case AudioLoop:       mediaPlayerState->setLooping(audioButtons[i].isDown); return;
@@ -510,3 +505,3 @@ void AudioWidget::closeEvent( QCloseEvent* ) {
 
-void AudioWidget::paintEvent( QPaintEvent * pe) {
+void AudioWidget::paintEvent( QPaintEvent * pe ) {
     if ( !pe->erased() ) {
@@ -582,14 +577,2 @@ void AudioWidget::keyReleaseEvent( QKeyEvent *e) {
       case Key_Escape: {
-/*
- * author pleas tell me where the i come from .-)
-   #if defined(QT_QWS_IPAQ)
-          if( mediaPlayerState->isPaused ) {
-              setToggleButton( i, FALSE );
-              mediaPlayerState->setPaused( FALSE );
-          } else if( !mediaPlayerState->isPaused ) {
-              setToggleButton( i, TRUE );
-              mediaPlayerState->setPaused( TRUE );
-          }
-#endif
-*/
       }
diff --git a/noncore/multimedia/opieplayer2/lib.cpp b/noncore/multimedia/opieplayer2/lib.cpp
index 4021d4a..85030de 100644
--- a/noncore/multimedia/opieplayer2/lib.cpp
+++ b/noncore/multimedia/opieplayer2/lib.cpp
@@ -152,3 +152,5 @@ int Lib::play( const QString& fileName, int startPos, int start_time ) {
     QString str = fileName.stripWhiteSpace();
-    xine_open( m_xine,  QFile::encodeName(str.utf8() ).data() );
+    if ( !xine_open( m_xine, QFile::encodeName(str.utf8() ).data() ) ) {
+        return 0;
+    }
     return xine_play( m_xine, startPos, start_time);
@@ -193,5 +195,11 @@ int Lib::length() {
 bool Lib::isSeekable() {
-    return xine_get_stream_info ( m_xine, XINE_STREAM_INFO_SEEKABLE );
+    return xine_get_stream_info( m_xine, XINE_STREAM_INFO_SEEKABLE );
+}
+
+void Lib::seekTo( int time ) {
+//     xine_trick_mode ( m_xine, XINE_TRICK_MODE_SEEK_TO_TIME, time ); NOT IMPLEMENTED YET IN XINE :_(
+    xine_play( m_xine, 0, time );
 }
 
+
 Frame Lib::currentFrame() {
@@ -201,4 +209,4 @@ Frame Lib::currentFrame() {
 
-QString Lib::metaInfo() {
-    xine_get_meta_info( m_xine, 0 );
+QString Lib::metaInfo( int number) {
+    return xine_get_meta_info( m_xine, number );
 }
@@ -218,3 +226,3 @@ void Lib::setShowVideo( bool video ) {
     m_video = video;
-    ::null_set_show_video( m_videoOutput,  video );
+    ::null_set_show_video( m_videoOutput, video );
 }
@@ -225,2 +233,16 @@ bool Lib::isShowingVideo() {
 
+bool Lib::hasVideo() {
+    //looks like it is not implemented yet
+    //return xine_get_stream_info( m_xine, XINE_STREAM_INFO_VIDEO_CHANNELS );
+    // ugly hack until xine is ready, look for the width of the video
+    int test =  xine_get_stream_info( m_xine, 2 );
+    if( test > 0 ) {
+        // qDebug( QString(" has video:  %1").arg( test ) );
+        return true;
+    } else {
+        //qDebug ( "does not have video ");
+        return false;
+    }
+}
+
 void Lib::showVideoFullScreen( bool fullScreen ) {
@@ -234,3 +256,3 @@ bool Lib::isVideoFullScreen() {
 void Lib::setScaling( bool scale ) {
-    ::null_set_scaling( m_videoOutput, scale  );
+    ::null_set_scaling( m_videoOutput, scale );
 }
@@ -252,3 +274,3 @@ void Lib::xine_display_frame( void* user_data, uint8_t *frame,
                               int width,  int height,  int bytes ) {
-    ( (Lib*)user_data)->drawFrame( frame,  width, height, bytes );
+    ( (Lib*)user_data)->drawFrame( frame, width, height, bytes );
 }
@@ -260,5 +282,2 @@ void Lib::drawFrame( uint8_t* frame,  int width,  int height,  int bytes ) {
     }
-
-//    qWarning( "called draw frame %d %d", width, height );
-
     m_wid-> setVideoFrame ( frame, width, height, bytes );
diff --git a/noncore/multimedia/opieplayer2/lib.h b/noncore/multimedia/opieplayer2/lib.h
index 29adc4d..6cdd9c6 100644
--- a/noncore/multimedia/opieplayer2/lib.h
+++ b/noncore/multimedia/opieplayer2/lib.h
@@ -119,5 +119,14 @@ namespace XINE {
          * Get the meta info (like author etc) from the stream
-         *
+         * XINE_META_INFO_TITLE               0
+         * XINE_META_INFO_COMMENT             1
+         * XINE_META_INFO_ARTIST              2
+         * XINE_META_INFO_GENRE               3
+         * XINE_META_INFO_ALBUM               4
+         * XINE_META_INFO_YEAR                5
+         * XINE_META_INFO_VIDEOCODEC          6
+         * XINE_META_INFO_AUDIOCODEC          7
+         * XINE_META_INFO_SYSTEMLAYER         8
+         * XINE_META_INFO_INPUT_PLUGIN        9
          */
-        QString metaInfo() ;
+        QString metaInfo( int number );
 
@@ -129,2 +138,13 @@ namespace XINE {
         /**
+         * seek to a position
+         */
+        void seekTo( int time );
+
+        /**
+         *
+         * @return is media stream has video
+         */
+        bool hasVideo();
+
+        /**
          *
@@ -146,2 +166,6 @@ namespace XINE {
          * Returns the error code
+         * XINE_ERROR_NONE                    0
+         * XINE_ERROR_NO_INPUT_PLUGIN         1
+         * XINE_ERROR_NO_DEMUXER_PLUGIN       2
+         * XINE_ERROR_DEMUXER_FAILED          3
          */
diff --git a/noncore/multimedia/opieplayer2/mediaplayer.cpp b/noncore/multimedia/opieplayer2/mediaplayer.cpp
index 553e3c1..8da7f73 100644
--- a/noncore/multimedia/opieplayer2/mediaplayer.cpp
+++ b/noncore/multimedia/opieplayer2/mediaplayer.cpp
@@ -38,3 +38,3 @@ MediaPlayer::MediaPlayer( QObject *parent, const char *name )
     playList->setCaption(tr("OpiePlayer: Initializating"));
-    
+
     qApp->processEvents();
@@ -102,27 +102,40 @@ void MediaPlayer::setPlaying( bool play ) {
         time.sprintf("%li:%02i", seconds/60, (int)seconds%60 );
-        qDebug(time);
-        
-    } else { //if playing in file list.. play in a different way
-          // random and looping settings enabled causes problems here,
-          // since there is no selected file in the playlist, but a selected file in the file list,
-          // so we remember and shutoff
+        //qDebug(time);
+
+    } else {
+        //if playing in file list.. play in a different way
+        // random and looping settings enabled causes problems here,
+        // since there is no selected file in the playlist, but a selected file in the file list,
+        // so we remember and shutoff
         l = mediaPlayerState->looping();
-        if(l)
+        if(l) {
             mediaPlayerState->setLooping( false );
+        }
         r = mediaPlayerState->shuffled();
-        mediaPlayerState->setShuffled(false);
-        
+        mediaPlayerState->setShuffled( false );
+
         fileName = playList->currentFileListPathName();
-        xineControl->play( fileName);
-        long seconds =  mediaPlayerState->length();//
+        xineControl->play( fileName );
+        long seconds =  mediaPlayerState->length();
         time.sprintf("%li:%02i", seconds/60, (int)seconds%60 );
-        qDebug(time);
-        if( fileName.left(4) != "http" )
-            fileName = QFileInfo( fileName).baseName();
+        //qDebug(time);
+        if( fileName.left(4) != "http" ) {
+            fileName = QFileInfo( fileName ).baseName();
+        }
 
     }
-        if( fileName.left(4) == "http" )
-            tickerText= tr( " File: " ) + fileName;
-        else
-            tickerText = tr( " File: " ) + fileName + tr(", Length: ") + time;
+
+    if( fileName.left(4) == "http" ) {
+       if ( xineControl->getMetaInfo().isEmpty() ) {
+           tickerText = tr( " File: " ) + fileName;
+       } else {
+           tickerText = xineControl->getMetaInfo();
+       }
+    } else {
+        if ( xineControl->getMetaInfo().isEmpty() ) {
+            tickerText = tr( " File: " ) + fileName + tr( ", Length: " ) + time + "  ";
+        } else {
+            tickerText = xineControl->getMetaInfo() + " Length: " + time + "  ";
+        }
+    }
     audioUI->setTickerText( tickerText );
@@ -132,3 +145,3 @@ void MediaPlayer::setPlaying( bool play ) {
 void MediaPlayer::prev() {
-    if(playList->whichList() == 0) { //if using the playlist
+    if( playList->whichList() == 0 ) { //if using the playlist
         if ( playList->prev() ) {
@@ -194,6 +207,6 @@ void MediaPlayer::stopChangingVolume() {
     int h=0;
-    if( !xineControl->hasVideo()) {
+    if( !xineControl->hasVideo() ) {
         w = audioUI->width();
         h = audioUI->height();
-        audioUI->repaint( (w - 200) / 2, h - yoff, 200 + 9, 70, FALSE );
+        audioUI->repaint( ( w - 200 ) / 2, h - yoff, 200 + 9, 70, FALSE );
     } else {
@@ -201,3 +214,3 @@ void MediaPlayer::stopChangingVolume() {
         h = videoUI->height();
-        videoUI->repaint( (w - 200) / 2, h - yoff, 200 + 9, 70, FALSE );
+        videoUI->repaint( ( w - 200 ) / 2, h - yoff, 200 + 9, 70, FALSE );
     }
@@ -208,5 +221,5 @@ void MediaPlayer::timerEvent( QTimerEvent * ) {
     if ( volumeDirection == +1 ) {
-        volControl->incVol(2);
-    }  else if ( volumeDirection == -1 ) {
-        volControl->decVol(2);
+        volControl->incVol( 2 );
+    } else if ( volumeDirection == -1 ) {
+        volControl->decVol( 2 );
     }
@@ -225,3 +238,3 @@ void MediaPlayer::timerEvent( QTimerEvent * ) {
     int w=0; int h=0;
-    if( !xineControl->hasVideo()) {
+    if( !xineControl->hasVideo() ) {
         w = audioUI->width();
@@ -231,3 +244,3 @@ void MediaPlayer::timerEvent( QTimerEvent * ) {
             if ( onScreenDisplayVolume > v ) {
-                audioUI->repaint( (w - 200) / 2 + v * 20 + 0, h - yoff + 40, (onScreenDisplayVolume - v) * 20 + 9, 30, FALSE );
+                audioUI->repaint( ( w - 200 ) / 2 + v * 20 + 0, h - yoff + 40, ( onScreenDisplayVolume - v ) * 20 + 9, 30, FALSE );
             }
@@ -248,5 +261,5 @@ void MediaPlayer::timerEvent( QTimerEvent * ) {
             if ( v > i ) {
-                p.drawRect( (w - 200) / 2 + i * 20 + 0, h - yoff + 40, 9, 30 );
+                p.drawRect( ( w - 200 ) / 2 + i * 20 + 0, h - yoff + 40, 9, 30 );
             } else {
-                p.drawRect( (w - 200) / 2 + i * 20 + 3, h - yoff + 50, 3, 10 );
+                p.drawRect( ( w - 200 ) / 2 + i * 20 + 3, h - yoff + 50, 3, 10 );
             }
@@ -259,3 +272,3 @@ void MediaPlayer::timerEvent( QTimerEvent * ) {
             if ( onScreenDisplayVolume > v ) {
-                videoUI->repaint( (w - 200) / 2 + v * 20 + 0, h - yoff + 40, (onScreenDisplayVolume - v) * 20 + 9, 30, FALSE );
+                videoUI->repaint( (w - 200) / 2 + v * 20 + 0, h - yoff + 40, ( onScreenDisplayVolume - v ) * 20 + 9, 30, FALSE );
             }
@@ -272,3 +285,3 @@ void MediaPlayer::timerEvent( QTimerEvent * ) {
         p.setFont( f );
-        p.drawText( (w - 200) / 2, h - yoff + 20, tr("Volume") );
+        p.drawText( (w - 200) / 2, h - yoff + 20, tr( "Volume" ) );
 
@@ -329,3 +342,3 @@ void MediaPlayer::cleanUp() {// this happens on closing
     playList->writeConfig( cfg );
-    
+
 //     QPEApplication::grabKeyboard();
diff --git a/noncore/multimedia/opieplayer2/mediaplayer.h b/noncore/multimedia/opieplayer2/mediaplayer.h
index 05be128..131db33 100644
--- a/noncore/multimedia/opieplayer2/mediaplayer.h
+++ b/noncore/multimedia/opieplayer2/mediaplayer.h
@@ -1 +1,33 @@
+ /*
+                            This file is part of the Opie Project
+
+                             Copyright (c)  2002 Max Reiss <harlekin@handhelds.org>
+                             Copyright (c)  2002 LJP <>
+                             Copyright (c)  2002 Holger Freyther <zecke@handhelds.org>
+              =.
+            .=l.
+           .>+-=
+ _;:,     .>    :=|.         This program is free software; you can
+.> <`_,   >  .   <=          redistribute it and/or  modify it under
+:`=1 )Y*s>-.--   :           the terms of the GNU General Public
+.="- .-=="i,     .._         License as published by the Free Software
+ - .   .-<_>     .<>         Foundation; either version 2 of the License,
+     ._= =}       :          or (at your option) any later version.
+    .%`+i>       _;_.
+    .i_,=:_.      -<s.       This program is distributed in the hope that
+     +  .  -:.       =       it will be useful,  but WITHOUT ANY WARRANTY;
+    : ..    .:,     . . .    without even the implied warranty of
+    =_        +     =;=|`    MERCHANTABILITY or FITNESS FOR A
+  _.=:.       :    :=>`:     PARTICULAR PURPOSE. See the GNU
+..}^=.=       =       ;      Library General Public License for more
+++=   -.     .`     .:       details.
+ :     =  ...= . :.=-
+ -.   .:....=;==+<;          You should have received a copy of the GNU
+  -_. . .   )=.  =           Library General Public License along with
+    --        :-=`           this library; see the file COPYING.LIB.
+                             If not, write to the Free Software Foundation,
+                             Inc., 59 Temple Place - Suite 330,
+                             Boston, MA 02111-1307, USA.
+
+*/
 
diff --git a/noncore/multimedia/opieplayer2/opieplayer2.pro b/noncore/multimedia/opieplayer2/opieplayer2.pro
index 1b687a3..619d36d 100644
--- a/noncore/multimedia/opieplayer2/opieplayer2.pro
+++ b/noncore/multimedia/opieplayer2/opieplayer2.pro
@@ -4,3 +4,3 @@ CONFIG         = qt warn_on debug
 DESTDIR   = $(OPIEDIR)/bin
-HEADERS   = playlistselection.h mediaplayerstate.h xinecontrol.h mediadetect.h\
+HEADERS   = playlistselection.h mediaplayerstate.h xinecontrol.h \
    	    videowidget.h audiowidget.h playlistwidget.h mediaplayer.h inputDialog.h \
@@ -9,3 +9,3 @@ HEADERS   = playlistselection.h mediaplayerstate.h xinecontrol.h mediadetect.h\
 SOURCES   = main.cpp \
-            playlistselection.cpp mediaplayerstate.cpp xinecontrol.cpp mediadetect.cpp\
+            playlistselection.cpp mediaplayerstate.cpp xinecontrol.cpp \
             videowidget.cpp audiowidget.cpp playlistwidget.cpp mediaplayer.cpp inputDialog.cpp \
diff --git a/noncore/multimedia/opieplayer2/xinecontrol.cpp b/noncore/multimedia/opieplayer2/xinecontrol.cpp
index fabc9a5..c22822c 100644
--- a/noncore/multimedia/opieplayer2/xinecontrol.cpp
+++ b/noncore/multimedia/opieplayer2/xinecontrol.cpp
@@ -35,2 +35,3 @@
 #include <qtimer.h>
+#include <qmessagebox.h>
 #include <qpe/qcopenvelope_qws.h>
@@ -45,6 +46,7 @@ XineControl::XineControl( QObject *parent, const char *name )
     : QObject( parent, name ) {
+
     libXine = new XINE::Lib(videoUI->vidWidget() );
 
-    connect ( videoUI, SIGNAL( videoResized ( const QSize & )), this, SLOT( videoResized ( const QSize & )));
-    connect( mediaPlayerState, SIGNAL( pausedToggled(bool) ),  this, SLOT( pause(bool) ) );
+    connect ( videoUI, SIGNAL( videoResized ( const QSize & )), this, SLOT( videoResized ( const QSize & ) ) );
+    connect( mediaPlayerState, SIGNAL( pausedToggled( bool ) ),  this, SLOT( pause( bool ) ) );
     connect( this, SIGNAL( positionChanged( long ) ), mediaPlayerState, SLOT( updatePosition( long ) ) );
@@ -54,3 +56,3 @@ XineControl::XineControl( QObject *parent, const char *name )
     connect( mediaPlayerState,  SIGNAL( videoGammaChanged( int ) ), this,  SLOT( setGamma( int ) ) );
-    connect( libXine, SIGNAL( stopped() ),  this, SLOT( nextMedia() ) );
+    connect( libXine, SIGNAL( stopped() ), this, SLOT( nextMedia() ) );
 
@@ -64,3 +66,3 @@ XineControl::~XineControl() {
         // Re-enable the suspend mode
-        QCopEnvelope("QPE/System", "setScreenSaverMode(int)" ) << QPEApplication::Enable;
+        QCopEnvelope( "QPE/System", "setScreenSaverMode(int)" ) << QPEApplication::Enable;
     }
@@ -71,4 +73,4 @@ XineControl::~XineControl() {
 void XineControl::play( const QString& fileName ) {
-    hasVideoChannel=FALSE;
-    hasAudioChannel=FALSE;
+    hasVideoChannel = FALSE;
+    hasAudioChannel = FALSE;
     m_fileName = fileName;
@@ -77,15 +79,20 @@ void XineControl::play( const QString& fileName ) {
 
-    libXine->play( fileName );
-    mediaPlayerState->setPlaying( true );
-    char whichGui = mdetect.videoOrAudio( fileName );
-    if (whichGui == 'f') {
-        qDebug("Nicht erkannter Dateityp");
+    if ( !libXine->play( fileName ) ) {
+        QMessageBox::warning( 0l , tr( "Failure" ), getErrorCode() );
         return;
     }
-    if (whichGui == 'a') {
+    mediaPlayerState->setPlaying( true );
+
+    char whichGui;
+    // qDebug( QString( "libXine->hasVideo() return : %1 ").arg( libXine->hasVideo() ) );
+    if ( !libXine->hasVideo() ) {
+        whichGui = 'a';
+        qDebug("HAS AUDIO");
         libXine->setShowVideo( false );
-        hasAudioChannel=TRUE;
+        hasAudioChannel = TRUE;
     } else {
+        whichGui = 'v';
+        qDebug("HAS VIDEO");
         libXine->setShowVideo( true );
-        hasVideoChannel=TRUE;
+        hasVideoChannel = TRUE;
     }
@@ -96,3 +103,2 @@ void XineControl::play( const QString& fileName ) {
 
-
 #if defined(Q_WS_QWS) && !defined(QT_NO_COP)
@@ -101,3 +107,3 @@ void XineControl::play( const QString& fileName ) {
         // Stop the screen from blanking and power saving state
-        QCopEnvelope("QPE/System", "setScreenSaverMode(int)" )
+        QCopEnvelope( "QPE/System", "setScreenSaverMode(int)" )
             << ( whichGui == 'v' ? QPEApplication::Disable : QPEApplication::DisableSuspend );
@@ -119,4 +125,4 @@ void XineControl::setGamma( int value ) {
 void XineControl::stop( bool isSet ) {
-    if ( !isSet) {
-        libXine->stop( );
+    if ( !isSet ) {
+        libXine->stop();
 
@@ -126,6 +132,5 @@ void XineControl::stop( bool isSet ) {
             // Re-enable the suspend mode
-            QCopEnvelope("QPE/System", "setScreenSaverMode(int)" ) << QPEApplication::Enable;
+            QCopEnvelope( "QPE/System", "setScreenSaverMode(int)" ) << QPEApplication::Enable;
         }
 #endif
-
     }
@@ -138,6 +143,6 @@ void XineControl::stop( bool isSet ) {
 void XineControl::pause( bool isSet) {
-    if (isSet) {
+    if ( isSet ) {
         libXine->pause();
     } else {
-        libXine->play( m_fileName, 0, m_currentTime);
+        libXine->play( m_fileName, 0, m_currentTime );
     }
@@ -151,3 +156,3 @@ long XineControl::currentTime() {
     // todo: jede sekunde überprüfen
-    m_currentTime =  libXine->currentTime();
+    m_currentTime = libXine->currentTime();
     return m_currentTime;
@@ -170,10 +175,11 @@ void  XineControl::length() {
 long XineControl::position() {
-    m_position = ( currentTime()  );
-    mediaPlayerState->updatePosition( m_position  );
+    m_position = ( currentTime() );
+    mediaPlayerState->updatePosition( m_position );
     long emitPos = (long)m_position;
     emit positionChanged( emitPos );
-    if(mediaPlayerState->isPlaying)
+    if( mediaPlayerState->isPlaying ) {
     // needs to be stopped the media is stopped
-    QTimer::singleShot( 1000, this, SLOT( position() ) );
-//    qDebug("POSITION : %d", m_position);
+        QTimer::singleShot( 1000, this, SLOT( position() ) );
+    }
+    // qDebug("POSITION : %d", m_position);
     return m_position;
@@ -186,3 +192,44 @@ long XineControl::position() {
 void XineControl::setFullscreen( bool isSet ) {
-    libXine->showVideoFullScreen( isSet);
+    libXine->showVideoFullScreen( isSet );
+}
+
+
+QString XineControl::getMetaInfo() {
+
+    QString returnString;
+
+    if ( !libXine->metaInfo( 0 ).isEmpty() ) {
+        returnString += tr( " Titel: " + libXine->metaInfo( 0 ) );
+    }
+
+    if ( !libXine->metaInfo( 1 ).isEmpty() ) {
+        returnString += tr( " Comment: " + libXine->metaInfo( 1 ) );
+    }
+
+    if ( !libXine->metaInfo( 2 ).isEmpty() ) {
+        returnString += tr( " Artist: " + libXine->metaInfo( 2 ) );
+    }
+
+    if ( !libXine->metaInfo( 3 ).isEmpty() ) {
+        returnString += tr( " Genre: " + libXine->metaInfo( 3 ) );
+    }
+
+    if ( !libXine->metaInfo( 4 ).isEmpty() ) {
+        returnString += tr( " Album: " + libXine->metaInfo( 4 ) );
+    }
+
+    if ( !libXine->metaInfo( 5 ).isEmpty() ) {
+        returnString += tr( " Year: " + libXine->metaInfo( 5 ) );
+    }
+    return returnString;
+}
+
+QString XineControl::getErrorCode() {
+    int errorCode = libXine->error();
+
+    if ( errorCode == 1 ) {
+        return tr( "No input plugin found for this media type" );
+    } else {
+        return tr( "Some other error" );
+    }
 }
@@ -194,3 +241,3 @@ void XineControl::setFullscreen( bool isSet ) {
 void XineControl::seekTo( long second ) {
-    libXine->play( m_fileName , 0, (int)second );
+    libXine->seekTo( (int)second );
 }
@@ -198,3 +245,3 @@ void XineControl::seekTo( long second ) {
 void XineControl::videoResized ( const QSize &s ) {
-  libXine-> resize ( s );
+    libXine->resize( s );
 }
diff --git a/noncore/multimedia/opieplayer2/xinecontrol.h b/noncore/multimedia/opieplayer2/xinecontrol.h
index 1de610b..b1300a8 100644
--- a/noncore/multimedia/opieplayer2/xinecontrol.h
+++ b/noncore/multimedia/opieplayer2/xinecontrol.h
@@ -37,3 +37,2 @@
 #include "lib.h"
-#include "mediadetect.h"
 #include <qobject.h>
@@ -79,2 +78,14 @@ public slots:
 
+    /**
+     * Get as much info about the stream from xine as possible
+     */
+    QString getMetaInfo();
+
+    /**
+     * get the error code and "translate" it for the user
+     *
+     */
+    QString getErrorCode();
+
+
     void videoResized ( const QSize &s );
@@ -90,3 +101,2 @@ private:
     XINE::Lib *libXine;
-    MediaDetect mdetect;
     long m_currentTime;