-OTicker is in libqtaux so we need to link it to avoid weird crashes

-Some function names have changed in xine update them
-Start to merge video_out_fb.c changes into nullvideo and give credit
 to the source of it

4 files changed, 236 insertions, 141 deletions

diff --git a/noncore/multimedia/opieplayer2/lib.cpp b/noncore/multimedia/opieplayer2/lib.cpp
index 1ab5c96..248221b 100644
--- a/noncore/multimedia/opieplayer2/lib.cpp
+++ b/noncore/multimedia/opieplayer2/lib.cpp
@@ -41,16 +41,17 @@ using namespace Opie::Core;
 
 /* QT */
 #include <qtextstream.h>
 #include <qdir.h>
 #include <qgfx_qws.h>
 
 /* STD */
 #include <assert.h>
+#include <unistd.h>
 
 typedef void (*display_xine_frame_t) (void *user_data, uint8_t* frame,
                                       int  width, int height,int bytes );
 
 extern "C" {
     xine_vo_driver_t* init_video_out_plugin( xine_t *xine, void* video, display_xine_frame_t, void * );
     int null_is_showing_video( const xine_vo_driver_t* self );
     void null_set_show_video( const xine_vo_driver_t* self, int show );
@@ -103,20 +104,20 @@ Lib::Lib( InitializationMode initMode, XineVideoWidget* widget )
         m_initialized = true;
     }
     else
         start();
 }
 
 void Lib::run()
 {
-    odebug << "Lib::run() started" << oendl; 
+    odebug << "Lib::run() started" << oendl;
     initialize();
     m_initialized = true;
-    odebug << "Lib::run() finished" << oendl; 
+    odebug << "Lib::run() finished" << oendl;
 }
 
 void Lib::initialize()
 {
     m_duringInitialization = true;
     m_xine =  xine_new( );
 
     QString configPath = QDir::homeDirPath() + "/Settings/opiexine.cf";
@@ -141,17 +142,17 @@ void Lib::initialize()
         printf( "!0\n" );
         setWidget( m_wid );
     }
 
     m_queue = xine_event_new_queue (m_stream);
 
     xine_event_create_listener_thread (m_queue, xine_event_handler, this);
 
-      ::null_preload_decoders( m_stream );
+    ::null_preload_decoders( m_stream );
 
     m_duringInitialization = false;
 }
 
 Lib::~Lib() {
     assert( isRunning() == false );
     assert( m_initialized );
 
@@ -212,17 +213,17 @@ int Lib::play( const QString& fileName, int startPos, int start_time ) {
         return 0;
     }
     return xine_play( m_stream, startPos, start_time);
 }
 
 void Lib::stop() {
     assert( m_initialized );
 
-    odebug << "<<<<<<<< STOP IN LIB TRIGGERED >>>>>>>" << oendl; 
+    odebug << "<<<<<<<< STOP IN LIB TRIGGERED >>>>>>>" << oendl;
     xine_stop( m_stream );
 }
 
 void Lib::pause( bool toggle ) {
     assert( m_initialized );
 
     xine_set_param( m_stream, XINE_PARAM_SPEED, toggle ? XINE_SPEED_PAUSE : XINE_SPEED_NORMAL );
 }
@@ -324,19 +325,19 @@ int Lib::error() const {
     return xine_get_error( m_stream );
 };
 
 void Lib::ensureInitialized()
 {
     if ( m_initialized )
         return;
 
-    odebug << "waiting for initialization thread to finish" << oendl; 
+    odebug << "waiting for initialization thread to finish" << oendl;
     wait();
-    odebug << "initialization thread finished!" << oendl; 
+    odebug << "initialization thread finished!" << oendl;
 }
 
 void Lib::setWidget( XineVideoWidget *widget )
 {
     m_wid = widget;
     resize ( m_wid-> size ( ) );
     ::null_set_mode( m_videoOutput, qt_screen->depth(), qt_screen->pixelType() );
     m_wid->repaint();
@@ -419,16 +420,16 @@ 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 );
 }
 
 void Lib::drawFrame( uint8_t* frame,  int width,  int height,  int bytes ) {
     assert( m_initialized );
 
     if ( !m_video ) {
-        owarn << "not showing video now" << oendl; 
+        owarn << "not showing video now" << oendl;
         return;
     }
 
     assert( m_wid );
 
     m_wid-> setVideoFrame ( frame, width, height, bytes );
 }
diff --git a/noncore/multimedia/opieplayer2/nullvideo.c b/noncore/multimedia/opieplayer2/nullvideo.c
index 378bbd4..6769a37 100644
--- a/noncore/multimedia/opieplayer2/nullvideo.c
+++ b/noncore/multimedia/opieplayer2/nullvideo.c
@@ -1,14 +1,15 @@
 /*
                             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>
+                             Copyright (c)  2002-2003 Miguel Freitas of xine
               =.
             .=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,
@@ -108,17 +109,17 @@ struct opie_frame_s {
   null_driver_t     *output;
 };
 
 static uint32_t null_get_capabilities( vo_driver_t *self ){
   null_driver_t* this = (null_driver_t*)self;
   return this->m_capabilities;
 }
 
-static void null_frame_copy (vo_frame_t *vo_img, uint8_t **src) {
+static void null_frame_proc_slice (vo_frame_t *vo_img, uint8_t **src) {
   opie_frame_t  *frame = (opie_frame_t *) vo_img ;
 
   vo_img->proc_called = 1;
 
   if (!frame->output->m_show_video) {
     /* printf("nullvideo: no video\n");  */
     return;
   }
@@ -170,84 +171,66 @@ static vo_frame_t* null_alloc_frame( vo_driver_t* self ){
 
   null_driver_t* this = (null_driver_t*)self;
   opie_frame_t* frame;
 
 #ifdef LOG
   fprintf (stderr, "nullvideo: alloc_frame\n");
 #endif
 
-  frame = (opie_frame_t*)malloc ( sizeof(opie_frame_t) );
+  frame = (opie_frame_t*)xine_xmalloc ( sizeof(opie_frame_t) );
 
-  memset( frame, 0, sizeof( opie_frame_t) );
   memcpy (&frame->sc, &this->sc, sizeof(vo_scale_t));
 
   pthread_mutex_init (&frame->frame.mutex, NULL);
 
   frame->output = this;
 
   /* initialize the frame*/
   frame->frame.driver = self;
-  frame->frame.proc_slice = null_frame_copy;
+  frame->frame.proc_slice = null_frame_proc_slice;
   frame->frame.field = null_frame_field;
   frame->frame.dispose = null_frame_dispose;
 
   /*
    *    colorspace converter for this frame
    */
   frame->yuv2rgb = this->yuv2rgb_factory->create_converter (this->yuv2rgb_factory);
 
   return (vo_frame_t*) frame;
 }
 
-static void null_update_frame_format( vo_driver_t* self, vo_frame_t* img,
-                                      uint32_t width, uint32_t height,
-                                      double ratio_code,  int format, 
-                                      int flags ){
-  null_driver_t* this = (null_driver_t*) self;
-  opie_frame_t*  frame = (opie_frame_t*)img;
-  /* not needed now */
 
-#ifdef LOG
-  fprintf (stderr, "nullvideo: update_frame_format\n");
-#endif
+static void null_frame_compute_ideal_size( null_driver_t *this, 
+                                           opie_frame_t *frame ) {
+    this = this;
 
-  flags &= VO_BOTH_FIELDS;
+    _x_vo_scale_compute_ideal_size(&frame->sc);
+}
 
-  /* find out if we need to adapt this frame */
+static void null_frame_compute_rgb_size( null_driver_t *this,
+                                         opie_frame_t  *frame ){
+    this = this;
 
-  if ((width != frame->sc.delivered_width)
-      || (height != frame->sc.delivered_height)
-      || (ratio_code != frame->sc.delivered_ratio)
-      || (flags != frame->flags)
-      || (format != frame->format)
-      || (this->sc.user_ratio != frame->sc.user_ratio)
-      || (this->gui_width != frame->sc.gui_width)
-      || (this->gui_height != frame->sc.gui_height)) {
+    _x_vo_scale_compute_output_size(&frame->sc);
 
-    frame->sc.delivered_width      = width;
-    frame->sc.delivered_height     = height;
-    frame->sc.delivered_ratio      = ratio_code;
-    frame->flags                   = flags;
-    frame->format                  = format;
-    frame->sc.user_ratio           = this->sc.user_ratio;
-    frame->sc.gui_width            = this->gui_width;
-    frame->sc.gui_height           = this->gui_height;
-    frame->sc.gui_pixel_aspect     = 1.0;
+    /* avoid problems in yuv2rgb */
+    if(frame->sc.output_height < (frame->sc.delivered_height+15) >> 4)
+        frame->sc.output_height = (frame->sc.delivered_height+15) >> 4;
 
-    vo_scale_compute_ideal_size ( &frame->sc );
-    vo_scale_compute_output_size( &frame->sc );
+    if (frame->sc.output_width < 8)
+        frame->sc.output_width = 8;
 
-#ifdef LOG
-    fprintf (stderr, "nullvideo: gui %dx%d delivered %dx%d output %dx%d\n",
-       frame->sc.gui_width, frame->sc.gui_height,
-       frame->sc.delivered_width, frame->sc.delivered_height,
-       frame->sc.output_width, frame->sc.output_height);
-#endif
+    /* yuv2rgb_mlib needs an even YUV2 width */
+    if (frame->sc.output_width & 1)
+        frame->sc.output_width++;
+}
 
+static void null_frame_reallocate( null_driver_t *this, opie_frame_t *frame,
+                                   uint32_t width, uint32_t height, int format){
     /*
      * (re-) allocate
      */
     if( frame->data ) {
       if( frame->chunk[0] ){
           free( frame->chunk[0] );
           frame->chunk[0] = NULL;
       }
@@ -257,17 +240,17 @@ static void null_update_frame_format( vo_driver_t* self, vo_frame_t* img,
       }
       if( frame->chunk[2] ){
           free ( frame->chunk[2] );
           frame->chunk[2] = NULL;
       }
       free ( frame->data );
     }
 
-    frame->data = xine_xmalloc (frame->sc.output_width
+      frame->data = xine_xmalloc (frame->sc.output_width
                                 * frame->sc.output_height
                                 * this->bytes_per_pixel );
 
     if( format == XINE_IMGFMT_YV12 ) {
       frame->frame.pitches[0] = 8*((width + 7) / 8);
       frame->frame.pitches[1] = 8*((width + 15) / 16);
       frame->frame.pitches[2] = 8*((width + 15) / 16);
       frame->frame.base[0] = xine_xmalloc_aligned (16, frame->frame.pitches[0] * height,(void **)&frame->chunk[0]);
@@ -278,73 +261,126 @@ static void null_update_frame_format( vo_driver_t* self, vo_frame_t* img,
       frame->frame.pitches[0] = 8*((width + 3) / 4);
 
       frame->frame.base[0] = xine_xmalloc_aligned (16, frame->frame.pitches[0] * height,
                                                    (void **)&frame->chunk[0]);
       frame->chunk[1] = NULL;
       frame->chunk[2] = NULL;
     }
 
+}
+
+static void null_setup_colorspace_converter(opie_frame_t *frame, int flags ) {
+    switch (flags) {
+        case VO_TOP_FIELD:
+        case VO_BOTTOM_FIELD:
+            frame->yuv2rgb->configure (frame->yuv2rgb,
+                                       frame->sc.delivered_width,
+                                       16,
+                                       2*frame->frame.pitches[0],
+                                       2*frame->frame.pitches[1],
+                                       frame->sc.output_width,
+                                       frame->stripe_height,
+                                       frame->bytes_per_line*2);
+            frame->yuv_stride = frame->bytes_per_line*2;
+            break;
+        case VO_BOTH_FIELDS:
+            frame->yuv2rgb->configure (frame->yuv2rgb,
+                                       frame->sc.delivered_width,
+                                       16,
+                                       frame->frame.pitches[0],
+                                       frame->frame.pitches[1],
+                                       frame->sc.output_width,
+                                       frame->stripe_height,
+                                       frame->bytes_per_line);
+            frame->yuv_stride = frame->bytes_per_line;
+            break;
+    }
+#ifdef LOG
+    fprintf (stderr, "nullvideo: colorspace converter configured.\n");
+#endif
+}
+
+static void null_update_frame_format( vo_driver_t* self, vo_frame_t* img,
+                                      uint32_t width, uint32_t height,
+                                      double ratio_code,  int format, 
+                                      int flags ){
+  null_driver_t* this = (null_driver_t*) self;
+  opie_frame_t*  frame = (opie_frame_t*)img;
+
+#ifdef LOG
+  fprintf (stderr, "nullvideo: update_frame_format\n");
+#endif
+
+  flags &= VO_BOTH_FIELDS;
+
+  /* find out if we need to adapt this frame */
+
+  if ((width != frame->sc.delivered_width)
+      || (height != frame->sc.delivered_height)
+      || (ratio_code != frame->sc.delivered_ratio)
+      || (flags != frame->flags)
+      || (format != frame->format)
+      || (this->sc.user_ratio != frame->sc.user_ratio)
+      || (this->gui_width != frame->sc.gui_width)
+      || (this->gui_height != frame->sc.gui_height)) {
+
+    frame->sc.delivered_width      = width;
+    frame->sc.delivered_height     = height;
+    frame->sc.delivered_ratio      = ratio_code;
+    frame->flags                   = flags;
+    frame->format                  = format;
+    frame->sc.user_ratio           = this->sc.user_ratio;
+    frame->sc.gui_width            = this->gui_width;
+    frame->sc.gui_height           = this->gui_height;
+    frame->sc.gui_pixel_aspect     = 1.0;
+
+
+    null_frame_compute_ideal_size(this, frame);
+    null_frame_compute_rgb_size(this, frame);
+    null_frame_reallocate(this, frame, width, height, format);
+
+#ifdef LOG
+    fprintf (stderr, "nullvideo: gui %dx%d delivered %dx%d output %dx%d\n",
+       frame->sc.gui_width, frame->sc.gui_height,
+       frame->sc.delivered_width, frame->sc.delivered_height,
+       frame->sc.output_width, frame->sc.output_height);
+#endif
+
+
+
     frame->stripe_height = 16 * frame->sc.output_height / frame->sc.delivered_height;
     frame->bytes_per_line = frame->sc.output_width * this->bytes_per_pixel;
 
     /*
      * set up colorspace converter
      */
+    null_setup_colorspace_converter(frame, flags);
 
-    switch (flags) {
-    case VO_TOP_FIELD:
-    case VO_BOTTOM_FIELD:
-      frame->yuv2rgb->configure (frame->yuv2rgb,
-                                 frame->sc.delivered_width,
-                                 16,
-                                 2*frame->frame.pitches[0],
-                                 2*frame->frame.pitches[1],
-                                 frame->sc.output_width,
-                                 frame->stripe_height,
-                                 frame->bytes_per_line*2);
-      frame->yuv_stride = frame->bytes_per_line*2;
-      break;
-    case VO_BOTH_FIELDS:
-      frame->yuv2rgb->configure (frame->yuv2rgb,
-                                 frame->sc.delivered_width,
-                                 16,
-                                 frame->frame.pitches[0],
-                                 frame->frame.pitches[1],
-                                 frame->sc.output_width,
-                                 frame->stripe_height,
-                                 frame->bytes_per_line);
-      frame->yuv_stride = frame->bytes_per_line;
-      break;
-    }
-#ifdef LOG
-    fprintf (stderr, "nullvideo: colorspace converter configured.\n");
-#endif
   }
-
   /*
    * reset dest pointers
    */
 
-  if (frame->data) {
-    switch (flags) {
-    case VO_TOP_FIELD:
-      frame->rgb_dst    = (uint8_t *)frame->data;
-      frame->stripe_inc = 2 * frame->stripe_height * frame->bytes_per_line;
-      break;
-    case VO_BOTTOM_FIELD:
-      frame->rgb_dst    = (uint8_t *)frame->data + frame->bytes_per_line ;
-      frame->stripe_inc = 2 * frame->stripe_height * frame->bytes_per_line;
-      break;
-    case VO_BOTH_FIELDS:
-      frame->rgb_dst    = (uint8_t *)frame->data;
-      frame->stripe_inc = frame->stripe_height * frame->bytes_per_line;
-      break;
+    if (frame->data) {
+        switch (flags) {
+            case VO_TOP_FIELD:
+                frame->rgb_dst    = (uint8_t *)frame->data;
+                frame->stripe_inc = 2 * frame->stripe_height * frame->bytes_per_line;
+                break;
+            case VO_BOTTOM_FIELD:
+                frame->rgb_dst    = (uint8_t *)frame->data + frame->bytes_per_line ;
+                frame->stripe_inc = 2 * frame->stripe_height * frame->bytes_per_line;
+                break;
+            case VO_BOTH_FIELDS:
+                frame->rgb_dst    = (uint8_t *)frame->data;
+                frame->stripe_inc = frame->stripe_height * frame->bytes_per_line;
+                break;
+        }
     }
-  }
 }
 
 static void null_display_frame( vo_driver_t* self, vo_frame_t *frame_gen ){
   null_driver_t* this = (null_driver_t*) self;
   opie_frame_t* frame = (opie_frame_t*)frame_gen;
   display_xine_frame_t display = this->frameDis;
 
   if (!this->m_show_video)
@@ -359,24 +395,28 @@ static void null_display_frame( vo_driver_t* self, vo_frame_t *frame_gen ){
   frame->frame.free(&frame->frame);
 }
 
 
 /* blending related */
 static void null_overlay_clut_yuv2rgb (null_driver_t  *this,
                                        vo_overlay_t *overlay,
                                        opie_frame_t *frame) {
+  this = this;
+
+
   int i;
   clut_t* clut = (clut_t*) overlay->color;
   if (!overlay->rgb_clut) {
     for (i = 0; i < sizeof(overlay->color)/sizeof(overlay->color[0]); i++) {
       *((uint32_t *)&clut[i]) =
-          frame->yuv2rgb->yuv2rgb_single_pixel_fun (frame->yuv2rgb,
-                                                    clut[i].y, clut[i].cb,
-                                                    clut[i].cr);
+          frame->yuv2rgb->
+          yuv2rgb_single_pixel_fun (frame->yuv2rgb,
+                                    clut[i].y, clut[i].cb,
+                                    clut[i].cr);
     }
     overlay->rgb_clut++;
   }
   if (!overlay->clip_rgb_clut) {
     clut = (clut_t*) overlay->clip_color;
     for (i = 0; i < sizeof(overlay->color)/sizeof(overlay->color[0]); i++) {
       *((uint32_t *)&clut[i]) =
   frame->yuv2rgb->yuv2rgb_single_pixel_fun(frame->yuv2rgb,
@@ -396,80 +436,143 @@ static void null_overlay_blend ( vo_driver_t *this_gen, vo_frame_t *frame_gen,
     return;
 
   /* Alpha Blend here */
   if (overlay->rle) {
     if( !overlay->rgb_clut || !overlay->clip_rgb_clut)
       null_overlay_clut_yuv2rgb(this,overlay,frame);
 
     switch(this->bpp) {
-    case 16:
-      blend_rgb16( (uint8_t *)frame->data, overlay,
-       frame->sc.output_width, frame->sc.output_height,
-       frame->sc.delivered_width, frame->sc.delivered_height);
-      break;
-    case 24:
-      blend_rgb24( (uint8_t *)frame->data, overlay,
-       frame->sc.output_width, frame->sc.output_height,
-       frame->sc.delivered_width, frame->sc.delivered_height);
-      break;
-    case 32:
-      blend_rgb32( (uint8_t *)frame->data, overlay,
-       frame->sc.output_width, frame->sc.output_height,
-       frame->sc.delivered_width, frame->sc.delivered_height);
-      break;
-    default:
-      /* It should never get here */
-      break;
+        case 16:
+            blend_rgb16((uint8_t *)frame->data,
+                        overlay,
+                        frame->sc.output_width,
+                        frame->sc.output_height,
+                        frame->sc.delivered_width,
+                        frame->sc.delivered_height);
+        break;
+        case 24:
+            blend_rgb24((uint8_t *)frame->data,
+                        overlay,
+                        frame->sc.output_width,
+                        frame->sc.output_height,
+                        frame->sc.delivered_width,
+                        frame->sc.delivered_height);
+        break;
+        case 32:
+            blend_rgb32((uint8_t *)frame->data,
+                        overlay,
+                        frame->sc.output_width,
+                        frame->sc.output_height,
+                        frame->sc.delivered_width,
+                        frame->sc.delivered_height);
+        break;
+        default:
+        /* It should never get here */
+            break;
     }
   }
 }
 
 
-static int null_get_property( vo_driver_t* self,
-                              int property ){
+static int null_get_property( vo_driver_t* self, int property ){
+#if 0
+  null_driver_t *this = (null_driver_t *)self;
+
+  switch(property)
+  {
+      case VO_PROP_ASPECT_RATIO:
+          return this->sc.user_ratio;
+      case VO_PROP_BRIGHTNESS:
+          return this->yuv2rgb_brightness;
+      case VO_PROP_WINDOW_WIDTH:
+          return this->sc.gui_width;
+      case VO_PROP_WINDOW_HEIGHT:
+          return this->sc.gui_height;
+    default:
+        break;
+  }
+#else
+  property = property;
+  self = self;
+#endif
+
   return 0;
 }
-static int null_set_property( vo_driver_t* self,
-                              int property,
+static int null_set_property( vo_driver_t* self, int property,
                               int value ){
+#if 0
+  null_driver_t *this = (null_driver_t *)self;
+
+  switch(property)
+  {
+      case VO_PROP_ASPECT_RATIO:
+          if(value>=XINE_VO_ASPECT_NUM_RATIOS)
+              value = XINE_VO_ASPECT_AUTO;
+          this->sc.user_ratio = value;
+          break;
+      case VO_PROP_BRIGHTNESS:
+          this->yuv2rgb_brightness = value;
+          this->yuv2rgb_factory->
+              set_csc_levels(this->yuv2rgb_factory, value, 128, 128);
+          break;
+      default:
+          break;
+  }
+#else
+  self = self;
+  property = property;
+#endif
+
   return value;
 }
 static void null_get_property_min_max( vo_driver_t* self,
                                        int property, int *min,
                                        int *max ){
+  self = self;
+  property = property;
+
   *max = 0;
   *min = 0;
 }
 static int null_gui_data_exchange( vo_driver_t* self,
                                    int data_type,
                                    void *data ){
+  self = self;
+  data_type = data_type;
+  data = data;
+
   return 0;
 }
 
 static void null_dispose ( vo_driver_t* self ){
   null_driver_t* this = (null_driver_t*)self;
   free ( this );
 }
 static int null_redraw_needed( vo_driver_t* self ){
-  return 0;
+    self = self;
+
+    return 0;
 }
 
 
-xine_vo_driver_t* init_video_out_plugin( xine_t *xine,
+xine_video_port_t* init_video_out_plugin( xine_t *xine,
                                          void* video,
                                          display_xine_frame_t frameDisplayFunc,
                                          void *userData ){
+  video = video;
+
+
   null_driver_t *vo;
   vo = (null_driver_t*)malloc( sizeof(null_driver_t ) );
 
   /* memset? */
   memset(vo,0, sizeof(null_driver_t ) );
 
-  vo_scale_init (&vo->sc, 0, 0, xine->config);
+  _x_vo_scale_init (&vo->sc, 0, 0, xine->config);
 
   vo->sc.gui_pixel_aspect = 1.0;
 
   vo->m_show_video = 0; // false
   vo->m_video_fullscreen = 0;
   vo->m_is_scaling = 0;
   vo->display_ratio = 1.0;
   vo->gui_width = 16;
@@ -486,39 +589,26 @@ xine_vo_driver_t* init_video_out_plugin( xine_t *xine,
   vo->vo_driver.set_property          = null_set_property;
   vo->vo_driver.get_property_min_max  = null_get_property_min_max;
   vo->vo_driver.gui_data_exchange     = null_gui_data_exchange;
   vo->vo_driver.dispose               = null_dispose;
   vo->vo_driver.redraw_needed         = null_redraw_needed;
     
 
   /* capabilities */
-  vo->m_capabilities = /* VO_CAP_COPIES_IMAGE | */ VO_CAP_YUY2 | VO_CAP_YV12;
+  vo->m_capabilities = VO_CAP_YUY2 | VO_CAP_YV12;
   vo->yuv2rgb_factory = yuv2rgb_factory_init (MODE_16_RGB, vo->yuv2rgb_swap,
                                               vo->yuv2rgb_cmap);
 
   vo->caller = userData;
   vo->frameDis = frameDisplayFunc;
 
-  /* return ( vo_driver_t*) vo; */
-  return vo_new_port( xine, ( vo_driver_t* )vo, 0 );
+  return _x_vo_new_port(xine, &vo->vo_driver, 0);
 }
 
-#if 0
-static vo_info_t vo_info_null = {
-  5,
-  XINE_VISUAL_TYPE_FB
-};
-
-vo_info_t *get_video_out_plugin_info(){
-  vo_info_null.description = ("xine video output plugin using null device");
-  return &vo_info_null;
-}
-
-#endif
 
 /* this is special for this device */
 /**
  * We know that we will be controled by the XINE LIB++
  */
 
 /**
  *
@@ -559,18 +649,17 @@ void null_set_gui_height( xine_vo_driver_t* self, int height ) {
 
 
 void null_set_mode( xine_vo_driver_t* self, int depth,  int rgb  ) {
   null_driver_t* this = (null_driver_t*)self->driver;
 
   this->bytes_per_pixel = (depth + 7 ) / 8;
   this->bpp = this->bytes_per_pixel * 8;
   this->depth = depth;
-  printf("depth %d %d\n", depth, this->bpp);
-  printf("pixeltype %d\n", rgb );
+ 
   switch ( this->depth ) {
   case 32:
     if( rgb == 0 )
       this->yuv2rgb_mode = MODE_32_RGB;
     else
       this->yuv2rgb_mode = MODE_32_BGR;
   case 24:
     if( this->bpp == 32 ) {
@@ -623,20 +712,24 @@ void null_display_handler( xine_vo_driver_t* self, display_xine_frame_t t,
 void null_preload_decoders( xine_stream_t *stream )
 {
     static const uint32_t preloadedAudioDecoders[] = { BUF_AUDIO_MPEG, BUF_AUDIO_VORBIS };
     static const uint8_t preloadedAudioDecoderCount = sizeof( preloadedAudioDecoders ) / sizeof( preloadedAudioDecoders[ 0 ] );
     static const uint32_t preloadedVideoDecoders[] = { BUF_VIDEO_MPEG, BUF_VIDEO_MPEG4, BUF_VIDEO_DIVX5 };
     static const uint8_t preloadedVideoDecoderCount = sizeof( preloadedVideoDecoders ) / sizeof( preloadedVideoDecoders[ 0 ] );
 
     uint8_t i;
+#if 0
 
     for ( i = 0; i < preloadedAudioDecoderCount; ++i ) {
-        audio_decoder_t *decoder = get_audio_decoder( stream, ( preloadedAudioDecoders[ i ] >> 16 ) & 0xff );
+        audio_decoder_t *decoder = get_audio_decoder( stream, ( preloadedAudioDecoders[ i ] >> 16 ) & 0xff );
+        decoder = decoder;
 /*        free_audio_decoder( stream, decoder ); */
     }
 
     for ( i = 0; i < preloadedVideoDecoderCount; ++i ) {
-         video_decoder_t *decoder = get_video_decoder( stream, ( preloadedVideoDecoders[ i ] >> 16 ) & 0xff );
+        video_decoder_t *decoder = get_video_decoder( stream, ( preloadedVideoDecoders[ i ] >> 16 ) & 0xff );
+         decoder = decoder;
 /*         free_video_decoder( stream, decoder ); */
     }
+#endif
 }
 
diff --git a/noncore/multimedia/opieplayer2/opieplayer2.pro b/noncore/multimedia/opieplayer2/opieplayer2.pro
index b9a8d6d..8166658 100644
--- a/noncore/multimedia/opieplayer2/opieplayer2.pro
+++ b/noncore/multimedia/opieplayer2/opieplayer2.pro
@@ -10,17 +10,17 @@ SOURCES   = main.cpp \
             frame.cpp lib.cpp nullvideo.c xinevideowidget.cpp volumecontrol.cpp \
         playlistwidgetgui.cpp\
             alphablend.c yuv2rgb.c yuv2rgb_arm2.c yuv2rgb_arm4l.S \
         threadutil.cpp mediawidget.cpp playlistview.cpp playlistfileview.cpp \
         skin.cpp
 TARGET    = opieplayer2
 INCLUDEPATH += $(OPIEDIR)/include
 DEPENDPATH  += $(OPIEDIR)/include
-LIBS        += -lqpe -lpthread -lopiecore2 -lopieui2 -lxine -lstdc++
+LIBS        += -lqpe -lpthread -lopiecore2 -lopieui2 -lqtaux2 -lxine 
 MOC_DIR   = qpeobj
 OBJECTS_DIR = qpeobj
 
 include ( $(OPIEDIR)/include.pro )
 
 !isEmpty( LIBXINE_INC_DIR ) {
     INCLUDEPATH = $$LIBXINE_INC_DIR $$INCLUDEPATH
 }
diff --git a/noncore/multimedia/opieplayer2/xinevideowidget.cpp b/noncore/multimedia/opieplayer2/xinevideowidget.cpp
index 15c611f..1ac9277 100644
--- a/noncore/multimedia/opieplayer2/xinevideowidget.cpp
+++ b/noncore/multimedia/opieplayer2/xinevideowidget.cpp
@@ -46,16 +46,17 @@
 
 // 180 deg rot: copy a line from src to dst reversed
 
 /*
  * This code relies the len be a multiply of 16bit
  */
 static inline void memcpy_rev ( void *_dst, void *_src, size_t len )
 {
+
     /*
      * move the source to the end
      */
     char *src_c = static_cast<char*>(_src) + len;
 
     /*
      * as we copy by 16bit and not 8bit
      * devide the length by two