- added an m_initialized flag and a guard for it, along with an

  assertion method called from all methods that access variables depending
  on initialization (basically all :)

2 files changed, 71 insertions, 1 deletions

diff --git a/noncore/multimedia/opieplayer2/lib.cpp b/noncore/multimedia/opieplayer2/lib.cpp
index 641cbca..99d5de6 100644
--- a/noncore/multimedia/opieplayer2/lib.cpp
+++ b/noncore/multimedia/opieplayer2/lib.cpp
@@ -72,4 +72,5 @@ using namespace XINE;
 
 Lib::Lib( XineVideoWidget* widget ) {
+    m_initialized = false;
     m_video = false;
     m_wid = widget;
@@ -117,5 +118,5 @@ void Lib::initialize()
         ::null_set_mode( m_videoOutput, qt_screen->depth(), qt_screen->pixelType() );
 
-        m_wid->repaint();
+//        m_wid->repaint();
     }
 
@@ -124,7 +125,12 @@ void Lib::initialize()
     xine_event_create_listener_thread (m_queue, xine_event_handler, this);
 
+    m_initialized = true;
 }
 
 Lib::~Lib() {
+    ThreadUtil::AutoLock lock( m_initGuard );
+
+    assert( m_initialized );
+
 //    free( m_config );
 
@@ -141,4 +147,15 @@ Lib::~Lib() {
 }
 
+void Lib::assertInitialized() const
+{
+    ThreadUtil::AutoLock lock( m_initGuard );
+
+    if ( m_initialized )
+        return;
+
+    qDebug( "LibXine: xine function called while not being initialized, yet! Fix the caller!" );
+    assert( m_initialized );
+}
+
 void Lib::resize ( const QSize &s ) {
     if ( s. width ( ) && s. height ( ) ) {
@@ -167,4 +184,6 @@ int Lib::subVersion() {
 
 int Lib::play( const QString& fileName, int startPos, int start_time ) {
+    assertInitialized();
+
     QString str = fileName.stripWhiteSpace();
     if ( !xine_open( m_stream, QFile::encodeName(str.utf8() ).data() ) ) {
@@ -175,4 +194,6 @@ int Lib::play( const QString& fileName, int startPos, int start_time ) {
 
 void Lib::stop() {
+    assertInitialized();
+
     qDebug("<<<<<<<< STOP IN LIB TRIGGERED >>>>>>>");
     xine_stop( m_stream );
@@ -180,20 +201,30 @@ void Lib::stop() {
 
 void Lib::pause( bool toggle ) {
+    assertInitialized();
+
     xine_set_param( m_stream, XINE_PARAM_SPEED, toggle ? XINE_SPEED_PAUSE : XINE_SPEED_NORMAL );
 }
 
 int Lib::speed() const {
+    assertInitialized();
+
     return  xine_get_param ( m_stream, XINE_PARAM_SPEED );
 }
 
 void Lib::setSpeed( int speed ) {
+    assertInitialized();
+
     xine_set_param ( m_stream, XINE_PARAM_SPEED, speed );
 }
 
 int Lib::status() const {
+    assertInitialized();
+
     return xine_get_status( m_stream );
 }
 
 int Lib::currentPosition() const {
+    assertInitialized();
+
     int pos, time, length;
     xine_get_pos_length( m_stream, &pos, &time, &length );
@@ -202,4 +233,6 @@ int Lib::currentPosition() const {
 
 int Lib::currentTime() const {
+    assertInitialized();
+
     int pos, time, length;
     xine_get_pos_length( m_stream, &pos, &time, &length );
@@ -208,4 +241,6 @@ int Lib::currentTime() const {
 
 int Lib::length() const {
+    assertInitialized();
+
       int pos, time, length;
       xine_get_pos_length( m_stream, &pos, &time, &length );
@@ -214,8 +249,12 @@ int Lib::length() const {
 
 bool Lib::isSeekable() const {
+    assertInitialized();
+
     return xine_get_stream_info( m_stream, XINE_STREAM_INFO_SEEKABLE );
 }
 
 void Lib::seekTo( int time ) {
+    assertInitialized();
+
     //xine_trick_mode ( m_stream, XINE_TRICK_MODE_SEEK_TO_TIME, time ); NOT IMPLEMENTED YET IN XINE :_(
     // since its now milliseconds we need *1000
@@ -225,4 +264,6 @@ void Lib::seekTo( int time ) {
 
 Frame Lib::currentFrame() const {
+    assertInitialized();
+
     Frame frame;
     return frame;
@@ -230,8 +271,12 @@ Frame Lib::currentFrame() const {
 
 QString Lib::metaInfo( int number) const {
+    assertInitialized();
+
     return xine_get_meta_info( m_stream, number );
 }
 
 int Lib::error() const {
+    assertInitialized();
+
     return xine_get_error( m_stream );
 };
@@ -249,4 +294,6 @@ void Lib::handleXineEvent( const xine_event_t* t ) {
 
 void Lib::handleXineEvent( int type ) {
+    assertInitialized();
+
     if ( type == XINE_EVENT_UI_PLAYBACK_FINISHED ) {
         emit stopped();
@@ -256,4 +303,6 @@ void Lib::handleXineEvent( int type ) {
 
 void Lib::setShowVideo( bool video ) {
+    assertInitialized();
+
     m_video = video;
     ::null_set_show_video( m_videoOutput, video );
@@ -261,24 +310,36 @@ void Lib::setShowVideo( bool video ) {
 
 bool Lib::isShowingVideo() const {
+    assertInitialized();
+
     return ::null_is_showing_video( m_videoOutput );
 }
 
 bool Lib::hasVideo() const {
+    assertInitialized();
+
     return  xine_get_stream_info( m_stream, 18 );
 }
 
 void Lib::showVideoFullScreen( bool fullScreen ) {
+    assertInitialized();
+
     ::null_set_fullscreen( m_videoOutput, fullScreen );
 }
 
 bool Lib::isVideoFullScreen() const {
+    assertInitialized();
+
     return ::null_is_fullscreen( m_videoOutput );
 }
 
 void Lib::setScaling( bool scale ) {
+    assertInitialized();
+
     ::null_set_scaling( m_videoOutput, scale );
 }
 
 void Lib::setGamma( int value ) {
+    assertInitialized();
+
   //qDebug( QString( "%1").arg(value)  );
   /* int gammaValue = ( 100 + value ); */
@@ -287,4 +348,6 @@ void Lib::setGamma( int value ) {
 
 bool Lib::isScaling() const {
+    assertInitialized();
+
     return ::null_is_scaling( m_videoOutput );
 }
@@ -300,4 +363,6 @@ void Lib::xine_display_frame( void* user_data, uint8_t *frame,
 
 void Lib::drawFrame( uint8_t* frame,  int width,  int height,  int bytes ) {
+    assertInitialized();
+
     if ( !m_video ) {
         qWarning("not showing video now");
diff --git a/noncore/multimedia/opieplayer2/lib.h b/noncore/multimedia/opieplayer2/lib.h
index d546c99..4b8dc81 100644
--- a/noncore/multimedia/opieplayer2/lib.h
+++ b/noncore/multimedia/opieplayer2/lib.h
@@ -184,4 +184,9 @@ namespace XINE {
         void initialize();
 
+        void assertInitialized() const;
+
+        mutable ThreadUtil::Mutex m_initGuard;
+        bool m_initialized : 1;
+
         int m_bytes_per_pixel;
         bool m_video:1;