]> git.sesse.net Git - vlc/blobdiff - src/video_output/video_output.c
* Mandatory step for video output IV and the audio output quality
[vlc] / src / video_output / video_output.c
index 32bf673ed61b143e28eb78075113c8a1d1fa1830..fcae25b41492a2738a6bd159ff0039f8287d7269 100644 (file)
@@ -1,30 +1,57 @@
 /*****************************************************************************
  * video_output.c : video output thread
- * (c)2000 VideoLAN
- *****************************************************************************
  * This module describes the programming interface for video output threads.
  * It includes functions allowing to open a new thread, send pictures to a
- * thread, and destroy a previously oppenned video output thread.
+ * thread, and destroy a previously oppened video output thread.
+ *****************************************************************************
+ * Copyright (C) 2000 VideoLAN
+ * $Id: video_output.c,v 1.122 2001/05/01 04:18:18 sam Exp $
+ *
+ * Authors: Vincent Seguin <seguin@via.ecp.fr>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * 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 General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
  *****************************************************************************/
 
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <errno.h>
-#include <stdlib.h>
-#include <stdio.h>
-#include <string.h>
+#include "defs.h"
+
+#include <errno.h>                                                 /* ENOMEM */
+#include <stdlib.h>                                                /* free() */
+#include <stdio.h>                                              /* sprintf() */
+#include <string.h>                                            /* strerror() */
+
+#ifdef STATS
+#   include <sys/times.h>
+#endif
 
-#include "common.h"
 #include "config.h"
+#include "common.h"
+#include "threads.h"
 #include "mtime.h"
-#include "vlc_thread.h"
+#include "modules.h"
+
 #include "video.h"
 #include "video_output.h"
 #include "video_text.h"
-#include "video_sys.h"
+#include "video_spu.h"
 #include "video_yuv.h"
+
 #include "intf_msg.h"
+
 #include "main.h"
 
 /*****************************************************************************
@@ -49,6 +76,7 @@ static void     RenderSubPicture  ( vout_thread_t *p_vout,
                                     subpicture_t *p_subpic );
 static void     RenderInterface   ( vout_thread_t *p_vout );
 static int      RenderIdle        ( vout_thread_t *p_vout );
+static int      RenderSplash      ( vout_thread_t *p_vout );
 static void     RenderInfo        ( vout_thread_t *p_vout );
 static int      Manage            ( vout_thread_t *p_vout );
 static int      Align             ( vout_thread_t *p_vout, int *pi_x,
@@ -65,22 +93,46 @@ static void     SetPalette        ( p_vout_thread_t p_vout, u16 *red,
  * If pi_status is NULL, then the function will block until the thread is ready.
  * If not, it will be updated using one of the THREAD_* constants.
  *****************************************************************************/
-vout_thread_t * vout_CreateThread               ( char *psz_display, int i_root_window,
-                                                  int i_width, int i_height, int *pi_status )
+vout_thread_t * vout_CreateThread   ( int *pi_status )
 {
     vout_thread_t * p_vout;                             /* thread descriptor */
     int             i_status;                               /* thread status */
     int             i_index;               /* index for array initialization */
 
     /* Allocate descriptor */
-    intf_DbgMsg("\n");
     p_vout = (vout_thread_t *) malloc( sizeof(vout_thread_t) );
     if( p_vout == NULL )
     {
-        intf_ErrMsg("error: %s\n", strerror(ENOMEM));
+        intf_ErrMsg( "vout error: vout thread creation returned %s",
+                     strerror(ENOMEM) );
+        return( NULL );
+    }
+
+    /* Choose the best module */
+    p_vout->p_module = module_Need( MODULE_CAPABILITY_VOUT, NULL );
+
+    if( p_vout->p_module == NULL )
+    {
+        intf_ErrMsg( "vout error: no suitable vout module" );
+        free( p_vout );
         return( NULL );
     }
 
+#define f p_vout->p_module->p_functions->vout.functions.vout
+    p_vout->pf_create     = f.pf_create;
+    p_vout->pf_init       = f.pf_init;
+    p_vout->pf_end        = f.pf_end;
+    p_vout->pf_destroy    = f.pf_destroy;
+    p_vout->pf_manage     = f.pf_manage;
+    p_vout->pf_display    = f.pf_display;
+    p_vout->pf_setpalette = f.pf_setpalette;
+#undef f
+
+    if( p_vout->pf_setpalette == NULL )
+    {
+        p_vout->pf_setpalette = SetPalette;
+    }
+
     /* Initialize thread properties - thread id and locks will be initialized
      * later */
     p_vout->b_die               = 0;
@@ -89,37 +141,39 @@ vout_thread_t * vout_CreateThread               ( char *psz_display, int i_root_
     p_vout->pi_status           = (pi_status != NULL) ? pi_status : &i_status;
     *p_vout->pi_status          = THREAD_CREATE;
 
-    /* Initialize some fields used by the system-dependant method - these fields will
-     * probably be modified by the method, and are only preferences */
-    p_vout->i_changes           = 0;
-    p_vout->i_width             = i_width;
-    p_vout->i_height            = i_height;
-    p_vout->i_bytes_per_line    = i_width * 2;
-    p_vout->i_screen_depth      = 15;
-    p_vout->i_bytes_per_pixel   = 2;
-    p_vout->f_gamma             = VOUT_GAMMA;
-
-    p_vout->b_grayscale         = main_GetIntVariable( VOUT_GRAYSCALE_VAR, VOUT_GRAYSCALE_DEFAULT );
-    p_vout->b_info              = 0;
-    p_vout->b_interface         = 0;
-    p_vout->b_scale             = 0;
-
-    p_vout->p_set_palette       = SetPalette;
-
-    intf_DbgMsg("wished configuration: %dx%d, %d/%d bpp (%d Bpl)\n",
-                p_vout->i_width, p_vout->i_height, p_vout->i_screen_depth,
-                p_vout->i_bytes_per_pixel * 8, p_vout->i_bytes_per_line );
+    /* Initialize some fields used by the system-dependant method - these
+     * fields will probably be modified by the method, and are only
+     * preferences */
+    p_vout->i_changes             = 0;
+    p_vout->i_width               = main_GetIntVariable( VOUT_WIDTH_VAR,
+                                                         VOUT_WIDTH_DEFAULT );
+    p_vout->i_height              = main_GetIntVariable( VOUT_HEIGHT_VAR,
+                                                         VOUT_HEIGHT_DEFAULT );
+    p_vout->i_bytes_per_line      = p_vout->i_width * 2;
+    p_vout->i_screen_depth        = 15;
+    p_vout->i_bytes_per_pixel     = 2;
+    p_vout->f_gamma               = VOUT_GAMMA;
+    p_vout->b_need_render         = 1;
+    p_vout->b_YCbr                = 0;
+    
+    p_vout->b_grayscale           = main_GetIntVariable( VOUT_GRAYSCALE_VAR,
+                                                     VOUT_GRAYSCALE_DEFAULT );
+    p_vout->b_info                = 0;
+    p_vout->b_interface           = 0;
+    p_vout->b_scale               = 1;
+
+    intf_WarnMsg( 1, "wished configuration: %dx%d, %d/%d bpp (%d Bpl)",
+                  p_vout->i_width, p_vout->i_height, p_vout->i_screen_depth,
+                  p_vout->i_bytes_per_pixel * 8, p_vout->i_bytes_per_line );
 
     /* Initialize idle screen */
-    p_vout->last_display_date   = mdate();
     p_vout->last_display_date   = 0;
     p_vout->last_idle_date      = 0;
+    p_vout->init_display_date   = mdate();
+    p_vout->render_time         = 10000;
 
-#ifdef STATS
     /* Initialize statistics fields */
-    p_vout->render_time         = 0;
     p_vout->c_fps_samples       = 0;
-#endif
 
     /* Initialize buffer index */
     p_vout->i_buffer_index      = 0;
@@ -130,26 +184,36 @@ vout_thread_t * vout_CreateThread               ( char *psz_display, int i_root_
     {
         p_vout->p_picture[i_index].i_type   =   EMPTY_PICTURE;
         p_vout->p_picture[i_index].i_status =   FREE_PICTURE;
+    }
+    for( i_index = 0; i_index < VOUT_MAX_SUBPICTURES; i_index++)
+    {
         p_vout->p_subpicture[i_index].i_type  = EMPTY_SUBPICTURE;
         p_vout->p_subpicture[i_index].i_status= FREE_SUBPICTURE;
     }
+    p_vout->i_pictures = 0;
 
     /* Create and initialize system-dependant method - this function issues its
      * own error messages */
-    if( vout_SysCreate( p_vout, psz_display, i_root_window ) )
+    if( p_vout->pf_create( p_vout ) )
     {
-      free( p_vout );
-      return( NULL );
+        module_Unneed( p_vout->p_module );
+        free( p_vout );
+        return( NULL );
     }
-    intf_DbgMsg("actual configuration: %dx%d, %d/%d bpp (%d Bpl), masks: 0x%x/0x%x/0x%x\n",
-                p_vout->i_width, p_vout->i_height, p_vout->i_screen_depth,
-                p_vout->i_bytes_per_pixel * 8, p_vout->i_bytes_per_line,
-                p_vout->i_red_mask, p_vout->i_green_mask, p_vout->i_blue_mask );
+    intf_WarnMsg( 1, "actual configuration: %dx%d, %d/%d bpp (%d Bpl), "
+                  "masks: 0x%x/0x%x/0x%x",
+                  p_vout->i_width, p_vout->i_height, p_vout->i_screen_depth,
+                  p_vout->i_bytes_per_pixel * 8, p_vout->i_bytes_per_line,
+                  p_vout->i_red_mask, p_vout->i_green_mask,
+                  p_vout->i_blue_mask );
 
     /* Calculate shifts from system-updated masks */
-    MaskToShift( &p_vout->i_red_lshift,   &p_vout->i_red_rshift,   p_vout->i_red_mask );
-    MaskToShift( &p_vout->i_green_lshift, &p_vout->i_green_rshift, p_vout->i_green_mask );
-    MaskToShift( &p_vout->i_blue_lshift,  &p_vout->i_blue_rshift,  p_vout->i_blue_mask );
+    MaskToShift( &p_vout->i_red_lshift, &p_vout->i_red_rshift,
+                 p_vout->i_red_mask );
+    MaskToShift( &p_vout->i_green_lshift, &p_vout->i_green_rshift,
+                 p_vout->i_green_mask );
+    MaskToShift( &p_vout->i_blue_lshift, &p_vout->i_blue_rshift,
+                 p_vout->i_blue_mask );
 
     /* Set some useful colors */
     p_vout->i_white_pixel = RGB2PIXEL( p_vout, 255, 255, 255 );
@@ -157,41 +221,39 @@ vout_thread_t * vout_CreateThread               ( char *psz_display, int i_root_
     p_vout->i_gray_pixel  = RGB2PIXEL( p_vout, 128, 128, 128 );
     p_vout->i_blue_pixel  = RGB2PIXEL( p_vout, 0, 0, 50 );
 
-    /* Load fonts - fonts must be initialized after the systme method since
-     * they may be dependant of screen depth and other thread properties */
-    p_vout->p_default_font      = vout_LoadFont( VOUT_DEFAULT_FONT );
+    /* Load fonts - fonts must be initialized after the system method since
+     * they may be dependant on screen depth and other thread properties */
+    p_vout->p_default_font = vout_LoadFont( VOUT_DEFAULT_FONT );
     if( p_vout->p_default_font == NULL )
     {
-        vout_SysDestroy( p_vout );
-        free( p_vout );
-        return( NULL );
+        intf_ErrMsg( "vout error: could not load default font" );
     }
-    p_vout->p_large_font        = vout_LoadFont( VOUT_LARGE_FONT );
+
+    p_vout->p_large_font = vout_LoadFont( VOUT_LARGE_FONT );
     if( p_vout->p_large_font == NULL )
     {
-        vout_UnloadFont( p_vout->p_default_font );
-        vout_SysDestroy( p_vout );
-        free( p_vout );
-        return( NULL );
+        intf_ErrMsg( "vout error: could not load large font" );
     }
 
     /* Create thread and set locks */
     vlc_mutex_init( &p_vout->picture_lock );
     vlc_mutex_init( &p_vout->subpicture_lock );
     vlc_mutex_init( &p_vout->change_lock );
-    vlc_mutex_lock( &p_vout->change_lock );
-    if( vlc_thread_create( &p_vout->thread_id, "video output", (void *) RunThread, (void *) p_vout) )
+
+    if( vlc_thread_create( &p_vout->thread_id, "video output",
+                           (void *) RunThread, (void *) p_vout) )
     {
-        intf_ErrMsg("error: %s\n", strerror(ENOMEM));
+        intf_ErrMsg("vout error: %s", strerror(ENOMEM));
         vout_UnloadFont( p_vout->p_default_font );
         vout_UnloadFont( p_vout->p_large_font );
-       vout_SysDestroy( p_vout );
+        p_vout->pf_destroy( p_vout );
         free( p_vout );
         return( NULL );
     }
 
-    intf_Msg("Video display initialized (%dx%d, %d/%d bpp)\n", p_vout->i_width,
-             p_vout->i_height, p_vout->i_screen_depth, p_vout->i_bytes_per_pixel * 8 );
+    intf_Msg( "vout: video display initialized (%dx%d, %d/%d bpp)",
+              p_vout->i_width, p_vout->i_height, p_vout->i_screen_depth,
+              p_vout->i_bytes_per_pixel * 8 );
 
     /* If status is NULL, wait until the thread is created */
     if( pi_status == NULL )
@@ -222,7 +284,7 @@ void vout_DestroyThread( vout_thread_t *p_vout, int *pi_status )
     int     i_status;                                       /* thread status */
 
     /* Set status */
-    intf_DbgMsg("\n");
+    intf_DbgMsg("");
     p_vout->pi_status = (pi_status != NULL) ? pi_status : &i_status;
     *p_vout->pi_status = THREAD_DESTROY;
 
@@ -249,7 +311,7 @@ void vout_DestroyThread( vout_thread_t *p_vout, int *pi_status )
  *****************************************************************************/
 void  vout_DisplaySubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
 {
-#ifdef DEBUG_VIDEO
+#ifdef TRACE_VOUT
     char        psz_begin_date[MSTRTIME_MAX_SIZE]; /* buffer for date string */
     char        psz_end_date[MSTRTIME_MAX_SIZE];   /* buffer for date string */
 #endif
@@ -258,7 +320,7 @@ void  vout_DisplaySubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
     /* Check if status is valid */
     if( p_subpic->i_status != RESERVED_SUBPICTURE )
     {
-        intf_DbgMsg("error: subpicture %p has invalid status %d\n", p_subpic,
+        intf_ErrMsg("error: subpicture %p has invalid status %d", p_subpic,
                     p_subpic->i_status );
     }
 #endif
@@ -266,9 +328,9 @@ void  vout_DisplaySubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
     /* Remove reservation flag */
     p_subpic->i_status = READY_SUBPICTURE;
 
-#ifdef DEBUG_VIDEO
-    /* Send subpicture informations */
-    intf_DbgMsg("subpicture %p: type=%d, begin date=%s, end date=%s\n",
+#ifdef TRACE_VOUT
+    /* Send subpicture information */
+    intf_DbgMsg("subpicture %p: type=%d, begin date=%s, end date=%s",
                 p_subpic, p_subpic->i_type,
                 mstrtime( psz_begin_date, p_subpic->begin_date ),
                 mstrtime( psz_end_date, p_subpic->end_date ) );
@@ -276,7 +338,7 @@ void  vout_DisplaySubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
 }
 
 /*****************************************************************************
- * vout_CreateSubPicture: allocate an subpicture in the video output heap.
+ * vout_CreateSubPicture: allocate a subpicture in the video output heap.
  *****************************************************************************
  * This function create a reserved subpicture in the video output heap.
  * A null pointer is returned if the function fails. This method provides an
@@ -296,45 +358,46 @@ subpicture_t *vout_CreateSubPicture( vout_thread_t *p_vout, int i_type,
     /*
      * Look for an empty place
      */
-    for( i_subpic = 0; i_subpic < VOUT_MAX_PICTURES; i_subpic++ )
+    for( i_subpic = 0; i_subpic < VOUT_MAX_SUBPICTURES; i_subpic++ )
     {
-       if( p_vout->p_subpicture[i_subpic].i_status == DESTROYED_SUBPICTURE )
-       {
-           /* Subpicture is marked for destruction, but is still allocated */
-           if( (p_vout->p_subpicture[i_subpic].i_type  == i_type)   &&
-               (p_vout->p_subpicture[i_subpic].i_size  >= i_size) )
-           {
-               /* Memory size do match or is smaller : memory will not be reallocated,
-                 * and function can end immediately - this is the best possible case,
-                 * since no memory allocation needs to be done */
-               p_vout->p_subpicture[i_subpic].i_status = RESERVED_SUBPICTURE;
-#ifdef DEBUG_VIDEO
-                intf_DbgMsg("subpicture %p (in destroyed subpicture slot)\n",
+        if( p_vout->p_subpicture[i_subpic].i_status == DESTROYED_SUBPICTURE )
+        {
+            /* Subpicture is marked for destruction, but is still allocated */
+            if( (p_vout->p_subpicture[i_subpic].i_type  == i_type)   &&
+                (p_vout->p_subpicture[i_subpic].i_size  >= i_size) )
+            {
+                /* Memory size do match or is smaller : memory will not be
+                 * reallocated, and function can end immediately - this is
+                 * the best possible case, since no memory allocation needs
+                 * to be done */
+                p_vout->p_subpicture[i_subpic].i_status = RESERVED_SUBPICTURE;
+#ifdef TRACE_VOUT
+                intf_DbgMsg("subpicture %p (in destroyed subpicture slot)",
                             &p_vout->p_subpicture[i_subpic] );
 #endif
-               vlc_mutex_unlock( &p_vout->subpicture_lock );
-               return( &p_vout->p_subpicture[i_subpic] );
-           }
-           else if( p_destroyed_subpic == NULL )
-           {
-               /* Memory size do not match, but subpicture index will be kept in
-                * case no other place are left */
-               p_destroyed_subpic = &p_vout->p_subpicture[i_subpic];
-           }   
-       }
+                vlc_mutex_unlock( &p_vout->subpicture_lock );
+                return( &p_vout->p_subpicture[i_subpic] );
+            }
+            else if( p_destroyed_subpic == NULL )
+            {
+                /* Memory size do not match, but subpicture index will be kept in
+                 * case no other place are left */
+                p_destroyed_subpic = &p_vout->p_subpicture[i_subpic];
+            }
+        }
         else if( (p_free_subpic == NULL) &&
                  (p_vout->p_subpicture[i_subpic].i_status == FREE_SUBPICTURE ))
         {
-           /* Subpicture is empty and ready for allocation */
+            /* Subpicture is empty and ready for allocation */
             p_free_subpic = &p_vout->p_subpicture[i_subpic];
         }
     }
 
-    /* If no free subpicture is available, use a destroyed subpicture */
+    /* If no free subpictures are available, use a destroyed subpicture */
     if( (p_free_subpic == NULL) && (p_destroyed_subpic != NULL ) )
     {
-       /* No free subpicture or matching destroyed subpicture has been found, but
-        * a destroyed subpicture is still avalaible */
+        /* No free subpicture or matching destroyed subpictures have been
+         * found, but a destroyed subpicture is still avalaible */
         free( p_destroyed_subpic->p_data );
         p_free_subpic = p_destroyed_subpic;
     }
@@ -350,16 +413,20 @@ subpicture_t *vout_CreateSubPicture( vout_thread_t *p_vout, int i_type,
         case TEXT_SUBPICTURE:                             /* text subpicture */
             p_free_subpic->p_data = malloc( i_size + 1 );
             break;
+        case DVD_SUBPICTURE:                          /* DVD subpicture unit */
+            p_free_subpic->p_data = malloc( i_size );
+            break;
 #ifdef DEBUG
         default:
-            intf_DbgMsg("error: unknown subpicture type %d\n", i_type );
+            intf_ErrMsg("error: unknown subpicture type %d", i_type );
             p_free_subpic->p_data   =  NULL;
             break;
 #endif
         }
 
         if( p_free_subpic->p_data != NULL )
-        {                    /* Copy subpicture informations, set some default values */
+        {
+            /* Copy subpicture information, set some default values */
             p_free_subpic->i_type                      = i_type;
             p_free_subpic->i_status                    = RESERVED_SUBPICTURE;
             p_free_subpic->i_size                      = i_size;
@@ -376,18 +443,19 @@ subpicture_t *vout_CreateSubPicture( vout_thread_t *p_vout, int i_type,
             p_free_subpic->i_type   =  EMPTY_SUBPICTURE;
             p_free_subpic->i_status =  FREE_SUBPICTURE;
             p_free_subpic =            NULL;
-            intf_ErrMsg("warning: %s\n", strerror( ENOMEM ) );
+            intf_ErrMsg( "vout error: spu allocation returned %s",
+                         strerror( ENOMEM ) );
         }
 
-#ifdef DEBUG_VIDEO
-        intf_DbgMsg("subpicture %p (in free subpicture slot)\n", p_free_subpic );
+#ifdef TRACE_VOUT
+        intf_DbgMsg("subpicture %p (in free subpicture slot)", p_free_subpic );
 #endif
         vlc_mutex_unlock( &p_vout->subpicture_lock );
         return( p_free_subpic );
     }
 
     /* No free or destroyed subpicture could be found */
-    intf_DbgMsg( "warning: heap is full\n" );
+    intf_DbgMsg( "warning: subpicture heap is full" );
     vlc_mutex_unlock( &p_vout->subpicture_lock );
     return( NULL );
 }
@@ -406,15 +474,15 @@ void vout_DestroySubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
    /* Check if status is valid */
    if( p_subpic->i_status != RESERVED_SUBPICTURE )
    {
-       intf_DbgMsg("error: subpicture %p has invalid status %d\n",
+       intf_ErrMsg("error: subpicture %p has invalid status %d",
                    p_subpic, p_subpic->i_status );
    }
 #endif
 
     p_subpic->i_status = DESTROYED_SUBPICTURE;
 
-#ifdef DEBUG_VIDEO
-    intf_DbgMsg("subpicture %p\n", p_subpic);
+#ifdef TRACE_VOUT
+    intf_DbgMsg("subpicture %p", p_subpic);
 #endif
 }
 
@@ -438,13 +506,13 @@ void  vout_DisplayPicture( vout_thread_t *p_vout, picture_t *p_pic )
         break;
 #ifdef DEBUG
     default:
-        intf_DbgMsg("error: picture %p has invalid status %d\n", p_pic, p_pic->i_status );
+        intf_ErrMsg("error: picture %p has invalid status %d", p_pic, p_pic->i_status );
         break;
 #endif
     }
 
-#ifdef DEBUG_VIDEO
-    intf_DbgMsg("picture %p\n", p_pic);
+#ifdef TRACE_VOUT
+    intf_DbgMsg("picture %p", p_pic);
 #endif
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
@@ -458,7 +526,7 @@ void  vout_DisplayPicture( vout_thread_t *p_vout, picture_t *p_pic )
  *****************************************************************************/
 void  vout_DatePicture( vout_thread_t *p_vout, picture_t *p_pic, mtime_t date )
 {
-#ifdef DEBUG_VIDEO
+#ifdef TRACE_VOUT
     char        psz_date[MSTRTIME_MAX_SIZE];                         /* date */
 #endif
 
@@ -474,13 +542,13 @@ void  vout_DatePicture( vout_thread_t *p_vout, picture_t *p_pic, mtime_t date )
         break;
 #ifdef DEBUG
     default:
-        intf_DbgMsg("error: picture %p has invalid status %d\n", p_pic, p_pic->i_status );
+        intf_ErrMsg("error: picture %p has invalid status %d", p_pic, p_pic->i_status );
         break;
 #endif
     }
 
-#ifdef DEBUG_VIDEO
-    intf_DbgMsg("picture %p, display date: %s\n", p_pic, mstrtime( psz_date, p_pic->date) );
+#ifdef TRACE_VOUT
+    intf_DbgMsg("picture %p, display date: %s", p_pic, mstrtime( psz_date, p_pic->date) );
 #endif
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
@@ -494,7 +562,7 @@ void  vout_DatePicture( vout_thread_t *p_vout, picture_t *p_pic, mtime_t date )
  * since several pictures can be created by several producers threads.
  *****************************************************************************/
 picture_t *vout_CreatePicture( vout_thread_t *p_vout, int i_type,
-                              int i_width, int i_height )
+                               int i_width, int i_height )
 {
     int         i_picture;                                  /* picture index */
     int         i_chroma_width = 0;                          /* chroma width */
@@ -509,37 +577,38 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout, int i_type,
      */
     for( i_picture = 0; i_picture < VOUT_MAX_PICTURES; i_picture++ )
     {
-       if( p_vout->p_picture[i_picture].i_status == DESTROYED_PICTURE )
-       {
-           /* Picture is marked for destruction, but is still allocated - note
+        if( p_vout->p_picture[i_picture].i_status == DESTROYED_PICTURE )
+        {
+            /* Picture is marked for destruction, but is still allocated - note
              * that if width and type are the same for two pictures, chroma_width
              * should also be the same */
-           if( (p_vout->p_picture[i_picture].i_type           == i_type)   &&
-               (p_vout->p_picture[i_picture].i_height         == i_height) &&
-               (p_vout->p_picture[i_picture].i_width          == i_width) )
-           {
-               /* Memory size do match : memory will not be reallocated, and function
+            if( (p_vout->p_picture[i_picture].i_type           == i_type)   &&
+                (p_vout->p_picture[i_picture].i_height         == i_height) &&
+                (p_vout->p_picture[i_picture].i_width          == i_width) )
+            {
+                /* Memory size do match : memory will not be reallocated, and function
                  * can end immediately - this is the best possible case, since no
                  * memory allocation needs to be done */
-               p_vout->p_picture[i_picture].i_status = RESERVED_PICTURE;
-#ifdef DEBUG_VIDEO
-                intf_DbgMsg("picture %p (in destroyed picture slot)\n",
+                p_vout->p_picture[i_picture].i_status = RESERVED_PICTURE;
+                p_vout->i_pictures++;
+#ifdef TRACE_VOUT
+                intf_DbgMsg("picture %p (in destroyed picture slot)",
                             &p_vout->p_picture[i_picture] );
 #endif
-               vlc_mutex_unlock( &p_vout->picture_lock );
-               return( &p_vout->p_picture[i_picture] );
-           }
-           else if( p_destroyed_picture == NULL )
-           {
-               /* Memory size do not match, but picture index will be kept in
-                * case no other place are left */
-               p_destroyed_picture = &p_vout->p_picture[i_picture];
-           }   
-       }
+                vlc_mutex_unlock( &p_vout->picture_lock );
+                return( &p_vout->p_picture[i_picture] );
+            }
+            else if( p_destroyed_picture == NULL )
+            {
+                /* Memory size do not match, but picture index will be kept in
+                 * case no other place are left */
+                p_destroyed_picture = &p_vout->p_picture[i_picture];
+            }
+        }
         else if( (p_free_picture == NULL) &&
                  (p_vout->p_picture[i_picture].i_status == FREE_PICTURE ))
         {
-           /* Picture is empty and ready for allocation */
+            /* Picture is empty and ready for allocation */
             p_free_picture = &p_vout->p_picture[i_picture];
         }
     }
@@ -547,8 +616,8 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout, int i_type,
     /* If no free picture is available, use a destroyed picture */
     if( (p_free_picture == NULL) && (p_destroyed_picture != NULL ) )
     {
-       /* No free picture or matching destroyed picture has been found, but
-        * a destroyed picture is still avalaible */
+        /* No free picture or matching destroyed picture has been found, but
+         * a destroyed picture is still avalaible */
         free( p_destroyed_picture->p_data );
         p_free_picture = p_destroyed_picture;
     }
@@ -584,7 +653,7 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout, int i_type,
             break;
 #ifdef DEBUG
         default:
-            intf_DbgMsg("error: unknown picture type %d\n", i_type );
+            intf_ErrMsg("error: unknown picture type %d", i_type );
             p_free_picture->p_data   =  NULL;
             break;
 #endif
@@ -592,19 +661,23 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout, int i_type,
 
         if( p_free_picture->p_data != NULL )
         {
-            /* Copy picture informations, set some default values */
+            /* Copy picture information, set some default values */
             p_free_picture->i_type                      = i_type;
             p_free_picture->i_status                    = RESERVED_PICTURE;
             p_free_picture->i_matrix_coefficients       = 1;
             p_free_picture->i_width                     = i_width;
             p_free_picture->i_height                    = i_height;
             p_free_picture->i_chroma_width              = i_chroma_width;
+            p_free_picture->i_size                      = i_width * i_height;
+            p_free_picture->i_chroma_size               = i_chroma_width
+                                                            * i_height;
             p_free_picture->i_display_horizontal_offset = 0;
             p_free_picture->i_display_vertical_offset   = 0;
             p_free_picture->i_display_width             = i_width;
             p_free_picture->i_display_height            = i_height;
             p_free_picture->i_aspect_ratio              = AR_SQUARE_PICTURE;
             p_free_picture->i_refcount                  = 0;
+            p_vout->i_pictures++;
         }
         else
         {
@@ -612,18 +685,23 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout, int i_type,
             p_free_picture->i_type   =  EMPTY_PICTURE;
             p_free_picture->i_status =  FREE_PICTURE;
             p_free_picture =            NULL;
-            intf_ErrMsg("warning: %s\n", strerror( ENOMEM ) );
+            intf_ErrMsg( "vout error: picture allocation returned %s",
+                         strerror( ENOMEM ) );
         }
 
-#ifdef DEBUG_VIDEO
-        intf_DbgMsg("picture %p (in free picture slot)\n", p_free_picture );
+#ifdef TRACE_VOUT
+        intf_DbgMsg("picture %p (in free picture slot)", p_free_picture );
 #endif
         vlc_mutex_unlock( &p_vout->picture_lock );
+
+        /* Initialize mutex */
+        vlc_mutex_init( &(p_free_picture->lock_deccount) );
+        
         return( p_free_picture );
     }
 
     /* No free or destroyed picture could be found */
-    intf_DbgMsg( "warning: heap is full\n" );
+    intf_DbgMsg( "warning: picture heap is full" );
     vlc_mutex_unlock( &p_vout->picture_lock );
     return( NULL );
 }
@@ -634,26 +712,32 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout, int i_type,
  * This function frees a previously reserved picture or a permanent
  * picture. It is meant to be used when the construction of a picture aborted.
  * Note that the picture will be destroyed even if it is linked !
- * This function does not need locking since reserved pictures are ignored by
- * the output thread.
  *****************************************************************************/
 void vout_DestroyPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
+    vlc_mutex_lock( &p_vout->picture_lock );
+
 #ifdef DEBUG
-   /* Check if picture status is valid */
-   if( (p_pic->i_status != RESERVED_PICTURE) &&
-       (p_pic->i_status != RESERVED_DATED_PICTURE) &&
-       (p_pic->i_status != RESERVED_DISP_PICTURE) )
-   {
-       intf_DbgMsg("error: picture %p has invalid status %d\n", p_pic, p_pic->i_status );
-   }
+    /* Check if picture status is valid */
+    if( (p_pic->i_status != RESERVED_PICTURE) &&
+        (p_pic->i_status != RESERVED_DATED_PICTURE) &&
+        (p_pic->i_status != RESERVED_DISP_PICTURE) )
+    {
+        intf_ErrMsg("error: picture %p has invalid status %d", p_pic, p_pic->i_status );
+    }
 #endif
 
     p_pic->i_status = DESTROYED_PICTURE;
+    p_vout->i_pictures--;
 
-#ifdef DEBUG_VIDEO
-    intf_DbgMsg("picture %p\n", p_pic);
+#ifdef TRACE_VOUT
+    intf_DbgMsg("picture %p", p_pic);
 #endif
+
+    /* destroy the lock that had been initialized in CreatePicture */
+    vlc_mutex_destroy( &(p_pic->lock_deccount) );
+   
+    vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
 /*****************************************************************************
@@ -667,8 +751,8 @@ void vout_LinkPicture( vout_thread_t *p_vout, picture_t *p_pic )
     vlc_mutex_lock( &p_vout->picture_lock );
     p_pic->i_refcount++;
 
-#ifdef DEBUG_VIDEO
-    intf_DbgMsg("picture %p refcount=%d\n", p_pic, p_pic->i_refcount );
+#ifdef TRACE_VOUT
+    intf_DbgMsg("picture %p refcount=%d", p_pic, p_pic->i_refcount );
 #endif
 
     vlc_mutex_unlock( &p_vout->picture_lock );
@@ -684,21 +768,22 @@ void vout_UnlinkPicture( vout_thread_t *p_vout, picture_t *p_pic )
     vlc_mutex_lock( &p_vout->picture_lock );
     p_pic->i_refcount--;
 
-#ifdef DEBUG_VIDEO
+#ifdef TRACE_VOUT
     if( p_pic->i_refcount < 0 )
     {
-        intf_DbgMsg("error: refcount < 0\n");
+        intf_DbgMsg("error: refcount < 0");
         p_pic->i_refcount = 0;
     }
 #endif
 
     if( (p_pic->i_refcount == 0) && (p_pic->i_status == DISPLAYED_PICTURE) )
     {
-       p_pic->i_status = DESTROYED_PICTURE;
+        p_pic->i_status = DESTROYED_PICTURE;
+        p_vout->i_pictures--;
     }
 
-#ifdef DEBUG_VIDEO
-    intf_DbgMsg("picture %p refcount=%d\n", p_pic, p_pic->i_refcount );
+#ifdef TRACE_VOUT
+    intf_DbgMsg("picture %p refcount=%d", p_pic, p_pic->i_refcount );
 #endif
 
     vlc_mutex_unlock( &p_vout->picture_lock );
@@ -758,32 +843,32 @@ void vout_Pixel2RGB( vout_thread_t *p_vout, u32 i_pixel, int *pi_red, int *pi_gr
  *****************************************************************************/
 static int BinaryLog(u32 i)
 {
-    int i_log;
+    int i_log = 0;
 
-    i_log = 0;
-    if (i & 0xffff0000)
+    if(i & 0xffff0000)
     {
-        i_log = 16;
+        i_log += 16;
     }
-    if (i & 0xff00ff00)
+    if(i & 0xff00ff00)
     {
         i_log += 8;
     }
-    if (i & 0xf0f0f0f0)
+    if(i & 0xf0f0f0f0)
     {
         i_log += 4;
     }
-    if (i & 0xcccccccc)
+    if(i & 0xcccccccc)
     {
         i_log += 2;
     }
-    if (i & 0xaaaaaaaa)
+    if(i & 0xaaaaaaaa)
     {
-        i_log++;
+        i_log += 1;
     }
+
     if (i != ((u32)1 << i_log))
     {
-       intf_ErrMsg("internal error: binary log overflow\n");
+        intf_DbgMsg("internal error: binary log overflow");
     }
 
     return( i_log );
@@ -821,11 +906,16 @@ static void MaskToShift( int *pi_left, int *pi_right, u32 i_mask )
 static int InitThread( vout_thread_t *p_vout )
 {
     /* Update status */
-    intf_DbgMsg("\n");
     *p_vout->pi_status = THREAD_START;
 
+    vlc_mutex_lock( &p_vout->change_lock );
+
+#ifdef STATS
+    p_vout->c_loops = 0;
+#endif
+
    /* Initialize output method - this function issues its own error messages */
-    if( vout_SysInit( p_vout ) )
+    if( p_vout->pf_init( p_vout ) )
     {
         return( 1 );
     }
@@ -833,14 +923,16 @@ static int InitThread( vout_thread_t *p_vout )
     /* Initialize convertion tables and functions */
     if( vout_InitYUV( p_vout ) )
     {
-        intf_ErrMsg("error: can't allocate YUV translation tables\n");
+        intf_ErrMsg("vout error: can't allocate YUV translation tables");
         return( 1 );
     }
 
     /* Mark thread as running and return */
     p_vout->b_active =          1;
     *p_vout->pi_status =        THREAD_READY;
-    intf_DbgMsg("thread ready\n");
+
+    
+    intf_DbgMsg("thread ready");
     return( 0 );
 }
 
@@ -869,7 +961,6 @@ static void RunThread( vout_thread_t *p_vout)
         DestroyThread( p_vout, THREAD_ERROR );
         return;
     }
-    intf_DbgMsg("\n");
 
     /*
      * Main loop - it is not executed if an error occured during
@@ -882,55 +973,81 @@ static void RunThread( vout_thread_t *p_vout)
         p_subpic =      NULL;
         display_date =  0;
         current_date =  mdate();
+#ifdef STATS
+        p_vout->c_loops++;
+        if( !(p_vout->c_loops % VOUT_STATS_NB_LOOPS) )
+        {
+            intf_Msg("vout stats: picture heap: %d/%d",
+                     p_vout->i_pictures, VOUT_MAX_PICTURES);
+        }
+#endif
 
         /*
-        * Find the picture to display - this operation does not need lock,
+         * Find the picture to display - this operation does not need lock,
          * since only READY_PICTUREs are handled
          */
         for( i_index = 0; i_index < VOUT_MAX_PICTURES; i_index++ )
-       {
-           if( (p_vout->p_picture[i_index].i_status == READY_PICTURE) &&
-               ( (p_pic == NULL) ||
-                 (p_vout->p_picture[i_index].date < display_date) ) )
-           {
+        {
+            if( (p_vout->p_picture[i_index].i_status == READY_PICTURE) &&
+            ( (p_pic == NULL) ||
+              (p_vout->p_picture[i_index].date < display_date) ) )
+            {
                 p_pic = &p_vout->p_picture[i_index];
                 display_date = p_pic->date;
-           }
-       }
+            }
+        }
 
         if( p_pic )
         {
-#ifdef STATS
             /* Computes FPS rate */
             p_vout->p_fps_sample[ p_vout->c_fps_samples++ % VOUT_FPS_SAMPLES ] = display_date;
-#endif 
-           if( display_date < current_date )
-           {
-               /* Picture is late: it will be destroyed and the thread will sleep and
-                 * go to next picture */
+
+            if( display_date < current_date - p_vout->render_time )
+            {
+                /* Picture is late: it will be destroyed and the thread
+                 * will sleep and go to next picture */
+
                 vlc_mutex_lock( &p_vout->picture_lock );
-                p_pic->i_status = p_pic->i_refcount ? DISPLAYED_PICTURE : DESTROYED_PICTURE;
-               intf_DbgMsg( "warning: late picture %p skipped refcount=%d\n", p_pic, p_pic->i_refcount );
+                if( p_pic->i_refcount )
+                {
+                    p_pic->i_status = DISPLAYED_PICTURE;
+                }
+                else
+                {
+                    p_pic->i_status = DESTROYED_PICTURE;
+                    p_vout->i_pictures--;
+                }
+                intf_WarnMsg( 3,
+                        "warning: late picture skipped (%p)", p_pic );
                 vlc_mutex_unlock( &p_vout->picture_lock );
+
+                continue;
+            }
+            else if( display_date > current_date + VOUT_DISPLAY_DELAY )
+            {
+                /* A picture is ready to be rendered, but its rendering date
+                 * is far from the current one so the thread will perform an
+                 * empty loop as if no picture were found. The picture state
+                 * is unchanged */
                 p_pic =         NULL;
                 display_date =  0;
-           }
-           else if( display_date > current_date + VOUT_DISPLAY_DELAY )
-           {
-               /* A picture is ready to be rendered, but its rendering date is
-                * far from the current one so the thread will perform an empty loop
-                * as if no picture were found. The picture state is unchanged */
-                p_pic =         NULL;
-                display_date =  0;
-           }
+            }
         }
-
         /*
-         * Find the subpicture to display - this operation does not need lock, since
-         * only READY_SUBPICTURES are handled. If no picture has been selected,
-         * display_date will depend on the subpicture
+         * Find the subpictures to display - this operation does not need
+         * lock, since only READY_SUBPICTURE are handled. If no picture
+         * has been selected, display_date will depend on the subpicture.
+         * We get an easily parsable chained list of subpictures which
+         * ends with NULL since p_subpic was initialized to NULL.
          */
-        //??
+        for( i_index = 0; i_index < VOUT_MAX_SUBPICTURES; i_index++ )
+        {
+            if( p_vout->p_subpicture[i_index].i_status == READY_SUBPICTURE )
+            {
+                p_vout->p_subpicture[i_index].p_next = p_subpic;
+                p_subpic = &p_vout->p_subpicture[i_index];
+            }
+        }
 
         /*
          * Perform rendering, sleep and display rendered picture
@@ -939,75 +1056,45 @@ static void RunThread( vout_thread_t *p_vout)
         {
             b_display = p_vout->b_active;
             p_vout->last_display_date = display_date;
+            p_vout->p_rendered_pic = p_pic;
 
-            if( b_display )
-            {
-                /* Set picture dimensions and clear buffer */
-                SetBufferPicture( p_vout, p_pic );
 
-                /* Render picture and informations */
+
+
+            /* Set picture dimensions and clear buffer */
+            SetBufferPicture( p_vout, p_pic );
+
+            /* FIXME: if b_need_render == 0 we need to do something with
+             * the subpictures one day. */
+
+            if( p_vout->b_need_render && b_display )
+            {
+                /* Render picture and information */
                 RenderPicture( p_vout, p_pic );
                 if( p_vout->b_info )
                 {
                     RenderPictureInfo( p_vout, p_pic );
                     RenderInfo( p_vout );
                 }
-            }
-
-            /* Remove picture from heap */
-            vlc_mutex_lock( &p_vout->picture_lock );
-            p_pic->i_status = p_pic->i_refcount ? DISPLAYED_PICTURE : DESTROYED_PICTURE;
-            vlc_mutex_unlock( &p_vout->picture_lock );
-
-            /* Render interface and subpicture */
-            if( b_display && p_vout->b_interface )
-            {
-                RenderInterface( p_vout );
-            }
-            if( p_subpic )
-            {
-                if( b_display )
+                if( p_subpic )
                 {
                     RenderSubPicture( p_vout, p_subpic );
                 }
-
-                /* Remove subpicture from heap */
-                vlc_mutex_lock( &p_vout->subpicture_lock );
-                p_subpic->i_status = DESTROYED_SUBPICTURE;
-                vlc_mutex_unlock( &p_vout->subpicture_lock );
             }
 
-        }
-        else if( p_subpic )                              /* subpicture alone */
-        {
-            b_display = p_vout->b_active;
-            p_vout->last_display_date = display_date;
-
-            if( b_display )
+            /* Render interface and subpicture */
+            if( b_display && p_vout->b_interface && p_vout->b_need_render )
             {
-                /* Clear buffer */
-                SetBufferPicture( p_vout, NULL );
-
-                /* Render informations, interface and subpicture */
-                if( p_vout->b_info )
-                {
-                    RenderInfo( p_vout );
-                }
-                if( p_vout->b_interface )
-                {
-                    RenderInterface( p_vout );
-                }
-                RenderSubPicture( p_vout, p_subpic );
+                RenderInterface( p_vout );
             }
 
-            /* Remove subpicture from heap */
-            vlc_mutex_lock( &p_vout->subpicture_lock );
-            p_subpic->i_status = DESTROYED_SUBPICTURE;
-            vlc_mutex_unlock( &p_vout->subpicture_lock );
         }
-        else if( p_vout->b_active )        /* idle or interface screen alone */
+        else if( p_vout->b_active && p_vout->b_need_render 
+                  && p_vout->init_display_date == 0)
         {
-            if( p_vout->b_interface && 0 /* && ?? intf_change */ )
+            /* Idle or interface screen alone */
+
+            if( p_vout->b_interface && 0 /* && XXX?? intf_change */ )
             {
                 /* Interface has changed, so a new rendering is required - force
                  * it by setting last idle date to 0 */
@@ -1032,14 +1119,37 @@ static void RunThread( vout_thread_t *p_vout)
             b_display = 0;
         }
 
+
+        /*
+         * Check for the current time and
+         * display splash screen if everything is on time
+         */
+        if( p_vout->init_display_date > 0 && p_vout->b_need_render ) 
+        {
+            if( p_vout->b_active && 
+                    mdate()-p_vout->init_display_date < 5000000)
+            {
+                /* there is something to display ! */
+                    b_display = 1;
+                    RenderSplash( p_vout );
+                
+            } else {
+                /* no splash screen ! */
+                p_vout->init_display_date=0;
+            }
+        }
+            
+
         /*
          * Sleep, wake up and display rendered picture
          */
 
-#ifdef STATS
-        /* Store render time */
-        p_vout->render_time = mdate() - current_date;
-#endif
+        if( display_date != 0 )
+        {
+            /* Store render time using Bresenham algorithm */
+            p_vout->render_time += mdate() - current_date;
+            p_vout->render_time >>= 1;
+        }
 
         /* Give back change lock */
         vlc_mutex_unlock( &p_vout->change_lock );
@@ -1047,7 +1157,7 @@ static void RunThread( vout_thread_t *p_vout)
         /* Sleep a while or until a given date */
         if( display_date != 0 )
         {
-            mwait( display_date );
+            mwait( display_date - VOUT_MWAIT_TOLERANCE );
         }
         else
         {
@@ -1057,26 +1167,45 @@ static void RunThread( vout_thread_t *p_vout)
         /* On awakening, take back lock and send immediately picture to display,
          * then swap buffers */
         vlc_mutex_lock( &p_vout->change_lock );
-#ifdef DEBUG_VIDEO
-        intf_DbgMsg( "picture %p, subpicture %p in buffer %d, display=%d\n", p_pic, p_subpic,
-                     p_vout->i_buffer_index, b_display && !(p_vout->i_changes & VOUT_NODISPLAY_CHANGE) );
+#ifdef TRACE_VOUT
+        intf_DbgMsg( "picture %p, subpicture %p in buffer %d, display=%d", p_pic, p_subpic,
+                     p_vout->i_buffer_index, b_display /* && !(p_vout->i_changes & VOUT_NODISPLAY_CHANGE) */ );
 #endif
-        if( b_display && !(p_vout->i_changes & VOUT_NODISPLAY_CHANGE) )
+        if( b_display /* && !(p_vout->i_changes & VOUT_NODISPLAY_CHANGE) */ )
         {
-            vout_SysDisplay( p_vout );
+            p_vout->pf_display( p_vout );
+#ifndef SYS_BEOS
             p_vout->i_buffer_index = ++p_vout->i_buffer_index & 1;
+#endif
         }
 
+        if( p_pic )
+        {
+            /* Remove picture from heap */
+            vlc_mutex_lock( &p_vout->picture_lock );
+            if( p_pic->i_refcount )
+            {
+                p_pic->i_status = DISPLAYED_PICTURE;
+            }
+            else
+            {
+                p_pic->i_status = DESTROYED_PICTURE;
+                p_vout->i_pictures--;
+            }
+            vlc_mutex_unlock( &p_vout->picture_lock );
+        }
+
+
         /*
          * Check events and manage thread
-        */
-        if( vout_SysManage( p_vout ) | Manage( p_vout ) )
-       {
-           /* A fatal error occured, and the thread must terminate immediately,
-            * without displaying anything - setting b_error to 1 cause the
-            * immediate end of the main while() loop. */
-           p_vout->b_error = 1;
-       }
+         */
+        if( p_vout->pf_manage( p_vout ) | Manage( p_vout ) )
+        {
+            /* A fatal error occured, and the thread must terminate immediately,
+             * without displaying anything - setting b_error to 1 cause the
+             * immediate end of the main while() loop. */
+            p_vout->b_error = 1;
+        }
     }
 
     /*
@@ -1090,7 +1219,7 @@ static void RunThread( vout_thread_t *p_vout)
     /* End of thread */
     EndThread( p_vout );
     DestroyThread( p_vout, THREAD_OVER );
-    intf_DbgMsg( "thread end\n" );
+    intf_DbgMsg( "thread end" );
 }
 
 /*****************************************************************************
@@ -1103,7 +1232,6 @@ static void RunThread( vout_thread_t *p_vout)
 static void ErrorThread( vout_thread_t *p_vout )
 {
     /* Wait until a `die' order */
-    intf_DbgMsg("\n");
     while( !p_vout->b_die )
     {
         /* Sleep a while */
@@ -1114,7 +1242,7 @@ static void ErrorThread( vout_thread_t *p_vout )
 /*****************************************************************************
  * EndThread: thread destruction
  *****************************************************************************
- * This function is called when the thread ends after a sucessfull
+ * This function is called when the thread ends after a sucessful
  * initialization. It frees all ressources allocated by InitThread.
  *****************************************************************************/
 static void EndThread( vout_thread_t *p_vout )
@@ -1122,16 +1250,29 @@ static void EndThread( vout_thread_t *p_vout )
     int     i_index;                                        /* index in heap */
 
     /* Store status */
-    intf_DbgMsg("\n");
     *p_vout->pi_status = THREAD_END;
 
+#ifdef STATS
+    {
+        struct tms cpu_usage;
+        times( &cpu_usage );
+
+        intf_Msg( "vout stats: cpu usage (user: %d, system: %d)",
+                  cpu_usage.tms_utime, cpu_usage.tms_stime );
+    }
+#endif
+
     /* Destroy all remaining pictures and subpictures */
     for( i_index = 0; i_index < VOUT_MAX_PICTURES; i_index++ )
     {
-       if( p_vout->p_picture[i_index].i_status != FREE_PICTURE )
-       {
+        if( p_vout->p_picture[i_index].i_status != FREE_PICTURE )
+        {
             free( p_vout->p_picture[i_index].p_data );
         }
+    }
+
+    for( i_index = 0; i_index < VOUT_MAX_SUBPICTURES; i_index++ )
+    {
         if( p_vout->p_subpicture[i_index].i_status != FREE_SUBPICTURE )
         {
             free( p_vout->p_subpicture[i_index].p_data );
@@ -1140,7 +1281,10 @@ static void EndThread( vout_thread_t *p_vout )
 
     /* Destroy translation tables */
     vout_EndYUV( p_vout );
-    vout_SysEnd( p_vout );
+    p_vout->pf_end( p_vout );
+
+    /* Release the change lock */
+    vlc_mutex_unlock( &p_vout->change_lock );
 }
 
 /*****************************************************************************
@@ -1154,13 +1298,22 @@ static void DestroyThread( vout_thread_t *p_vout, int i_status )
     int *pi_status;                                         /* status adress */
 
     /* Store status adress */
-    intf_DbgMsg("\n");
     pi_status = p_vout->pi_status;
 
     /* Destroy thread structures allocated by Create and InitThread */
     vout_UnloadFont( p_vout->p_default_font );
     vout_UnloadFont( p_vout->p_large_font );
-    vout_SysDestroy( p_vout );
+    p_vout->pf_destroy( p_vout );
+
+    /* Destroy the locks */
+    vlc_mutex_destroy( &p_vout->picture_lock );
+    vlc_mutex_destroy( &p_vout->subpicture_lock );
+    vlc_mutex_destroy( &p_vout->change_lock );
+                
+    /* Release the module */
+    module_Unneed( p_vout->p_module );
+
+    /* Free structure */
     free( p_vout );
     *pi_status = i_status;
 }
@@ -1169,7 +1322,7 @@ static void DestroyThread( vout_thread_t *p_vout, int i_status )
  * Print: print simple text on a picture
  *****************************************************************************
  * This function will print a simple text on the picture. It is designed to
- * print debugging or general informations.
+ * print debugging or general information.
  *****************************************************************************/
 void Print( vout_thread_t *p_vout, int i_x, int i_y, int i_h_align, int i_v_align, unsigned char *psz_text )
 {
@@ -1186,7 +1339,7 @@ void Print( vout_thread_t *p_vout, int i_x, int i_y, int i_h_align, int i_v_alig
                     i_y * p_vout->i_bytes_per_line + i_x * p_vout->i_bytes_per_pixel,
                     p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
                     p_vout->i_white_pixel, 0, 0,
-                    0, psz_text );
+                    0, psz_text, 100 );
     }
 }
 
@@ -1248,7 +1401,7 @@ static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int
     if( i_area == p_buffer->i_areas )
     {
         /* New area is below all existing ones: just add it at the end of the
-         * array, if possible - else, append it to the last one */
+         * array, if possible - otherwise, append it to the last one */
         if( i_area < VOUT_MAX_AREAS )
         {
             p_buffer->pi_area_begin[i_area] = i_y;
@@ -1257,8 +1410,8 @@ static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int
         }
         else
         {
-#ifdef DEBUG_VIDEO
-            intf_DbgMsg("areas overflow\n");
+#ifdef TRACE_VOUT
+            intf_DbgMsg("area overflow");
 #endif
             p_buffer->pi_area_end[VOUT_MAX_AREAS - 1] = i_h;
         }
@@ -1281,8 +1434,8 @@ static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int
                  * move all old areas down */
                 if( p_buffer->i_areas == VOUT_MAX_AREAS )
                 {
-#ifdef DEBUG_VIDEO
-                    intf_DbgMsg("areas overflow\n");
+#ifdef TRACE_VOUT
+                    intf_DbgMsg("areas overflow");
 #endif
                     p_buffer->pi_area_end[VOUT_MAX_AREAS - 2] = p_buffer->pi_area_end[VOUT_MAX_AREAS - 1];
                 }
@@ -1315,7 +1468,7 @@ static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int
             if( i_area_copy != i_area )
             {
                 /* Merge with last possible areas */
-                p_buffer->pi_area_end[i_area] = MAX( i_h, p_buffer->pi_area_end[i_area_copy] );
+                //p_buffer->pi_area_end[i_area] = MAX( i_h, p_buffer->pi_area_end[i_area_copy] );
 
                 /* Shift lower areas upward */
                 i_area_shift = i_area_copy - i_area;
@@ -1418,6 +1571,7 @@ static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
         /* Set picture position */
         i_pic_x = (p_vout->i_width - i_pic_width) / 2;
         i_pic_y = (p_vout->i_height - i_pic_height) / 2;
+
     }
     else
     {
@@ -1429,7 +1583,7 @@ static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
     }
 
     /*
-     * Set new picture size - if is is smaller than the previous one, clear
+     * Set new picture size - if it is smaller than the previous one, clear
      * around it. Since picture are centered, only their size is tested.
      */
     if( (p_buffer->i_pic_width > i_pic_width) || (p_buffer->i_pic_height > i_pic_height) )
@@ -1455,12 +1609,13 @@ static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
      */
     for( i_area = 0; i_area < p_buffer->i_areas; i_area++ )
     {
-#ifdef DEBUG_VIDEO
-        intf_DbgMsg("clearing picture %p area in buffer %d: %d-%d\n", p_pic,
+#ifdef TRACE_VOUT
+        intf_DbgMsg("clearing picture %p area in buffer %d: %d-%d", p_pic,
                     p_vout->i_buffer_index, p_buffer->pi_area_begin[i_area], p_buffer->pi_area_end[i_area] );
 #endif
         i_data_size = (p_buffer->pi_area_end[i_area] - p_buffer->pi_area_begin[i_area] + 1) * p_vout->i_bytes_per_line;
         p_data = (u64*) (p_buffer->p_data + p_vout->i_bytes_per_line * p_buffer->pi_area_begin[i_area]);
+
         for( i_data_index = i_data_size / 256; i_data_index-- ; )
         {
             /* Clear 256 bytes block */
@@ -1495,106 +1650,107 @@ static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
 /*****************************************************************************
  * RenderPicture: render a picture
  *****************************************************************************
- * This function convert a picture from a video heap to a pixel-encoded image
- * and copy it to the current rendering buffer. No lock is required, since the
- * rendered picture has been determined as existant, and will only be destroyed
- * by the vout thread later.
+ * This function converts a picture from a video heap to a pixel-encoded image
+ * and copies it to the current rendering buffer. No lock is required, since
+ * the * rendered picture has been determined as existant, and will only be
+ * destroyed by the vout thread later.
  *****************************************************************************/
 static void RenderPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
-#ifdef DEBUG_VIDEO
+#ifdef TRACE_VOUT
     char                psz_date[MSTRTIME_MAX_SIZE];         /* picture date */
     mtime_t             render_time;               /* picture rendering time */
 #endif
     vout_buffer_t *     p_buffer;                        /* rendering buffer */
     byte_t *            p_pic_data;                /* convertion destination */
 
-    /* Get and set rendering informations */
+    /* Get and set rendering information */
     p_buffer =          &p_vout->p_buffer[ p_vout->i_buffer_index ];
     p_pic_data =        p_buffer->p_data +
-        p_buffer->i_pic_x * p_vout->i_bytes_per_pixel +
-        p_buffer->i_pic_y * p_vout->i_bytes_per_line;
-#ifdef DEBUG_VIDEO
+                        p_buffer->i_pic_x * p_vout->i_bytes_per_pixel +
+                        p_buffer->i_pic_y * p_vout->i_bytes_per_line;
+#ifdef TRACE_VOUT
     render_time = mdate();
 #endif
 
+    
     /*
      * Choose appropriate rendering function and render picture
      */
     switch( p_pic->i_type )
     {
     case YUV_420_PICTURE:
-        p_vout->yuv.p_Convert420( p_vout, p_pic_data,
-                                  p_pic->p_y, p_pic->p_u, p_pic->p_v,
-                                  p_pic->i_width, p_pic->i_height,
-                                  p_buffer->i_pic_width, p_buffer->i_pic_height,
-                                  p_vout->i_bytes_per_line / p_vout->i_bytes_per_pixel,
-                                  p_pic->i_matrix_coefficients );
+        p_vout->yuv.pf_yuv420( p_vout, p_pic_data,
+                               p_pic->p_y, p_pic->p_u, p_pic->p_v,
+                               p_pic->i_width, p_pic->i_height,
+                               p_buffer->i_pic_width, p_buffer->i_pic_height,
+                               p_vout->i_bytes_per_line / p_vout->i_bytes_per_pixel,
+                               p_pic->i_matrix_coefficients );
         break;
     case YUV_422_PICTURE:
-        p_vout->yuv.p_Convert422( p_vout, p_pic_data,
-                                  p_pic->p_y, p_pic->p_u, p_pic->p_v,
-                                  p_pic->i_width, p_pic->i_height,
-                                  p_buffer->i_pic_width, p_buffer->i_pic_height,
-                                  p_vout->i_bytes_per_line / p_vout->i_bytes_per_pixel,
-                                  p_pic->i_matrix_coefficients );
+        p_vout->yuv.pf_yuv422( p_vout, p_pic_data,
+                               p_pic->p_y, p_pic->p_u, p_pic->p_v,
+                               p_pic->i_width, p_pic->i_height,
+                               p_buffer->i_pic_width, p_buffer->i_pic_height,
+                               p_vout->i_bytes_per_line / p_vout->i_bytes_per_pixel,
+                               p_pic->i_matrix_coefficients );
         break;
     case YUV_444_PICTURE:
-        p_vout->yuv.p_Convert444( p_vout, p_pic_data,
-                                  p_pic->p_y, p_pic->p_u, p_pic->p_v,
-                                  p_pic->i_width, p_pic->i_height,
-                                  p_buffer->i_pic_width, p_buffer->i_pic_height,
-                                  p_vout->i_bytes_per_line / p_vout->i_bytes_per_pixel,
-                                  p_pic->i_matrix_coefficients );
+        p_vout->yuv.pf_yuv444( p_vout, p_pic_data,
+                               p_pic->p_y, p_pic->p_u, p_pic->p_v,
+                               p_pic->i_width, p_pic->i_height,
+                               p_buffer->i_pic_width, p_buffer->i_pic_height,
+                               p_vout->i_bytes_per_line / p_vout->i_bytes_per_pixel,
+                               p_pic->i_matrix_coefficients );
         break;
 #ifdef DEBUG
     default:
-        intf_DbgMsg("error: unknown picture type %d\n", p_pic->i_type );
+        intf_ErrMsg("error: unknown picture type %d", p_pic->i_type );
         break;
 #endif
     }
 
-#ifdef DEBUG_VIDEO
+#ifdef TRACE_VOUT
     /* Print picture date and rendering time */
-    intf_DbgMsg("picture %p rendered in buffer %d (%ld us), display date: %s\n", p_pic,
+    intf_DbgMsg("picture %p rendered in buffer %d (%ld us), display date: %s", p_pic,
                 p_vout->i_buffer_index, (long) (mdate() - render_time),
                 mstrtime( psz_date, p_pic->date ));
 #endif
 }
 
 /*****************************************************************************
- * RenderPictureInfo: print additionnal informations on a picture
+ * RenderPictureInfo: print additionnal information on a picture
  *****************************************************************************
- * This function will print informations such as fps and other picture
- * dependant informations.
+ * This function will print information such as fps and other picture
+ * dependant information.
  *****************************************************************************/
 static void RenderPictureInfo( vout_thread_t *p_vout, picture_t *p_pic )
 {
-#if defined(STATS) || defined(DEBUG)
     char        psz_buffer[256];                            /* string buffer */
-#endif
 
-#ifdef STATS
     /*
      * Print FPS rate in upper right corner
      */
     if( p_vout->c_fps_samples > VOUT_FPS_SAMPLES )
     {
-        sprintf( psz_buffer, "%.2f fps", (double) VOUT_FPS_SAMPLES * 1000000 /
-                 ( p_vout->p_fps_sample[ (p_vout->c_fps_samples - 1) % VOUT_FPS_SAMPLES ] -
-                   p_vout->p_fps_sample[ p_vout->c_fps_samples % VOUT_FPS_SAMPLES ] ) );
+        long i_fps = VOUT_FPS_SAMPLES * 1000000 * 10 /
+                           ( p_vout->p_fps_sample[ (p_vout->c_fps_samples - 1)
+                                                   % VOUT_FPS_SAMPLES ] -
+                             p_vout->p_fps_sample[ p_vout->c_fps_samples
+                                                   % VOUT_FPS_SAMPLES ] );
+        sprintf( psz_buffer, "%li.%i fps", i_fps / 10, (int)i_fps % 10 );
         Print( p_vout, 0, 0, RIGHT_RALIGN, TOP_RALIGN, psz_buffer );
     }
 
     /*
      * Print frames count and loop time in upper left corner
      */
-    sprintf( psz_buffer, "%ld frames   rendering: %ld us",
+    sprintf( psz_buffer, "%ld frames, render: %ldus",
              (long) p_vout->c_fps_samples, (long) p_vout->render_time );
     Print( p_vout, 0, 0, LEFT_RALIGN, TOP_RALIGN, psz_buffer );
-#endif
 
-#ifdef DEBUG
+#ifdef STATS
     /*
      * Print picture information in lower right corner
      */
@@ -1617,6 +1773,42 @@ static void RenderPictureInfo( vout_thread_t *p_vout, picture_t *p_pic )
 #endif
 }
 
+/*****************************************************************************
+ * RenderSplash: render splash picture
+ *****************************************************************************
+ * This function will print something on the screen. It will return 0 if
+ * nothing has been rendered, or 1 if something has been changed on the screen.
+ * Note that if you absolutely want something to be printed, you will have
+ * to force it by setting the last idle date to 0.
+ * Unlike other rendering functions, this one calls the SetBufferPicture
+ * function when needed.
+ *****************************************************************************/
+int RenderSplash( vout_thread_t *p_vout )
+{
+    int         i_x = 0, i_y = 0;                           /* text position */
+    int         i_width, i_height;                              /* text size */
+    char *psz_text =    "VideoLAN Client (" VERSION ")";  /* text to display */
+
+    memset( p_vout->p_buffer[ p_vout->i_buffer_index ].p_data,
+                  p_vout->i_bytes_per_line * p_vout->i_height, 12);
+
+ //   SetBufferPicture( p_vout, NULL );
+    vout_TextSize( p_vout->p_large_font, WIDE_TEXT | OUTLINED_TEXT, psz_text,
+                   &i_width, &i_height );
+    if( !Align( p_vout, &i_x, &i_y, i_width, i_height, CENTER_RALIGN, CENTER_RALIGN ) )
+    {
+        vout_Print( p_vout->p_large_font,
+                    p_vout->p_buffer[ p_vout->i_buffer_index ].p_data +
+                    i_x * p_vout->i_bytes_per_pixel + (i_y - 16 ) * p_vout->i_bytes_per_line,
+                    p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
+                    p_vout->i_white_pixel, p_vout->i_gray_pixel, 0,
+                    WIDE_TEXT | OUTLINED_TEXT, psz_text, 100);
+        SetBufferArea( p_vout, i_x, i_y, i_width, i_height);
+    }
+    return( 1 );
+}
+
+
 /*****************************************************************************
  * RenderIdle: render idle picture
  *****************************************************************************
@@ -1627,41 +1819,63 @@ static void RenderPictureInfo( vout_thread_t *p_vout, picture_t *p_pic )
  * Unlike other rendering functions, this one calls the SetBufferPicture
  * function when needed.
  *****************************************************************************/
-static int RenderIdle( vout_thread_t *p_vout )
+int RenderIdle( vout_thread_t *p_vout )
 {
+#if 0
     int         i_x = 0, i_y = 0;                           /* text position */
     int         i_width, i_height;                              /* text size */
+    int         i_amount = 0;                             /*  amount to draw */
+    char *psz_text =    "Waiting for stream";            /* text to display */
+    char *psz_wtext =   "[................]";
+#endif
     mtime_t     current_date;                                /* current date */
-    const char *psz_text = "waiting for stream ...";      /* text to display */
+
+
+    memset( p_vout->p_buffer[ p_vout->i_buffer_index ].p_data,
+                    p_vout->i_bytes_per_line * p_vout->i_height, 12);
 
 
     current_date = mdate();
-    if( (current_date - p_vout->last_display_date) > VOUT_IDLE_DELAY &&
-        (current_date - p_vout->last_idle_date) > VOUT_IDLE_DELAY )
+    if( (current_date - p_vout->last_display_date) > VOUT_IDLE_DELAY 
+//            && (current_date - p_vout->last_idle_date) > VOUT_IDLE_DELAY 
+    )
     {
+        /* FIXME: idle screen disabled */
+#if 0
         SetBufferPicture( p_vout, NULL );
         vout_TextSize( p_vout->p_large_font, WIDE_TEXT | OUTLINED_TEXT, psz_text,
                        &i_width, &i_height );
-        if( !Align( p_vout, &i_x, &i_y, i_width, i_height, CENTER_RALIGN, CENTER_RALIGN ) )
+        if( !Align( p_vout, &i_x, &i_y, i_width, i_height * 2, CENTER_RALIGN, CENTER_RALIGN ) )
         {
+            i_amount = (int) ((current_date - p_vout->last_display_date ) / 5000LL);            
             vout_Print( p_vout->p_large_font,
                         p_vout->p_buffer[ p_vout->i_buffer_index ].p_data +
                         i_x * p_vout->i_bytes_per_pixel + i_y * p_vout->i_bytes_per_line,
                         p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
                         p_vout->i_white_pixel, p_vout->i_gray_pixel, 0,
-                        WIDE_TEXT | OUTLINED_TEXT, psz_text );
-            SetBufferArea( p_vout, i_x, i_y, i_width, i_height );
+                        WIDE_TEXT | OUTLINED_TEXT, psz_text,  (i_amount / 3 ) %110);
+
+            vout_Print( p_vout->p_large_font,
+                    p_vout->p_buffer[ p_vout->i_buffer_index ].p_data +
+                    i_x * p_vout->i_bytes_per_pixel + (i_y + 16) * p_vout->i_bytes_per_line,
+                    p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
+                    p_vout->i_white_pixel, p_vout->i_gray_pixel, 0,
+                    WIDE_TEXT | OUTLINED_TEXT, psz_wtext,  (i_amount/5)%110 );
+            
+
+            SetBufferArea( p_vout, i_x, i_y, i_width, i_height * 2 );
         }
+#endif
         return( 1 );
     }
     return( 0 );
 }
 
 /*****************************************************************************
- * RenderInfo: render additionnal informations
+ * RenderInfo: render additionnal information
  *****************************************************************************
- * This function render informations which do not depend of the current picture
- * rendered.
+ * This function renders information which do not depend on the current
+ * picture rendered.
  *****************************************************************************/
 static void RenderInfo( vout_thread_t *p_vout )
 {
@@ -1690,8 +1904,8 @@ static void RenderInfo( vout_thread_t *p_vout )
             break;
         }
     }
-    sprintf( psz_buffer, "pic: %d/%d/%d",
-             i_reserved_pic, i_ready_pic, VOUT_MAX_PICTURES );
+    sprintf( psz_buffer, "pic: %d (%d/%d)/%d",
+             p_vout->i_pictures, i_reserved_pic, i_ready_pic, VOUT_MAX_PICTURES );
     Print( p_vout, 0, 0, LEFT_RALIGN, BOTTOM_RALIGN, psz_buffer );
 #endif
 }
@@ -1699,50 +1913,79 @@ static void RenderInfo( vout_thread_t *p_vout )
 /*****************************************************************************
  * RenderSubPicture: render a subpicture
  *****************************************************************************
- * This function render a sub picture unit.
+ * This function renders a sub picture unit.
  *****************************************************************************/
 static void RenderSubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
 {
     p_vout_font_t       p_font;                                 /* text font */
     int                 i_width, i_height;          /* subpicture dimensions */
 
-    switch( p_subpic->i_type )
+    while( p_subpic != NULL )
     {
-    case TEXT_SUBPICTURE:                                /* single line text */
-        /* Select default font if not specified */
-        p_font = p_subpic->type.text.p_font;
-        if( p_font == NULL )
+        switch( p_subpic->i_type )
         {
-            p_font = p_vout->p_default_font;
-        }
+        case DVD_SUBPICTURE:                          /* DVD subpicture unit */
+            /* test if the picture really has to be displayed */
+            if( mdate() < p_subpic->begin_date )
+            {
+                /* not yet, see you later */
+                break;
+            }
+            if( mdate() > p_subpic->end_date )
+            {
+                /* too late, destroying the subpic */
+                vout_DestroySubPicture( p_vout, p_subpic );
+                break;
+            }
+            vout_RenderSPU( &p_vout->p_buffer[ p_vout->i_buffer_index ],
+                            p_subpic, p_vout->i_bytes_per_pixel,
+                            p_vout->i_bytes_per_line );
+            break;
+        case TEXT_SUBPICTURE:                            /* single line text */
+            /* Select default font if not specified */
+            p_font = p_subpic->type.text.p_font;
+            if( p_font == NULL )
+            {
+                p_font = p_vout->p_default_font;
+            }
 
-        /* Computes text size (width and height fields are ignored) and print it */
-        vout_TextSize( p_font, p_subpic->type.text.i_style, p_subpic->p_data, &i_width, &i_height );
-        if( !Align( p_vout, &p_subpic->i_x, &p_subpic->i_y, i_width, i_height,
-                    p_subpic->i_horizontal_align, p_subpic->i_vertical_align ) )
-        {
-            vout_Print( p_font, p_vout->p_buffer[ p_vout->i_buffer_index ].p_data +
-                        p_subpic->i_x * p_vout->i_bytes_per_pixel +
-                        p_subpic->i_y * p_vout->i_bytes_per_line,
-                        p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
-                        p_subpic->type.text.i_char_color, p_subpic->type.text.i_border_color,
-                        p_subpic->type.text.i_bg_color, p_subpic->type.text.i_style,
-                        p_subpic->p_data );
-            SetBufferArea( p_vout, p_subpic->i_x, p_subpic->i_y, i_width, i_height );
-        }
-        break;
+            /* Compute text size (width and height fields are ignored)
+             * and print it */
+            vout_TextSize( p_font, p_subpic->type.text.i_style,
+                           p_subpic->p_data, &i_width, &i_height );
+            if( !Align( p_vout, &p_subpic->i_x, &p_subpic->i_y,
+                        i_width, i_height, p_subpic->i_horizontal_align,
+                        p_subpic->i_vertical_align ) )
+            {
+                vout_Print( p_font,
+                            p_vout->p_buffer[ p_vout->i_buffer_index ].p_data +
+                            p_subpic->i_x * p_vout->i_bytes_per_pixel +
+                            p_subpic->i_y * p_vout->i_bytes_per_line,
+                            p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
+                            p_subpic->type.text.i_char_color,
+                            p_subpic->type.text.i_border_color,
+                            p_subpic->type.text.i_bg_color,
+                            p_subpic->type.text.i_style, p_subpic->p_data, 100 );
+                SetBufferArea( p_vout, p_subpic->i_x, p_subpic->i_y,
+                               i_width, i_height );
+            }
+            break;
 
 #ifdef DEBUG
-    default:
-        intf_DbgMsg("error: unknown subpicture %p type %d\n", p_subpic, p_subpic->i_type );
+        default:
+            intf_ErrMsg( "error: unknown subpicture %p type %d",
+                         p_subpic, p_subpic->i_type );
 #endif
+        }
+
+        p_subpic = p_subpic->p_next;
     }
 }
 
 /*****************************************************************************
  * RenderInterface: render the interface
  *****************************************************************************
- * This function render the interface, if any.
+ * This function renders the interface, if any.
  *****************************************************************************/
 static void RenderInterface( vout_thread_t *p_vout )
 {
@@ -1762,7 +2005,7 @@ static void RenderInterface( vout_thread_t *p_vout )
          i_byte < p_vout->i_height * p_vout->i_bytes_per_line;
          i_byte++ )
     {
-        //?? noooo !
+        /* XXX?? noooo ! */
         p_vout->p_buffer[ p_vout->i_buffer_index ].p_data[ i_byte ] = p_vout->i_blue_pixel;
     }
 
@@ -1773,7 +2016,7 @@ static void RenderInterface( vout_thread_t *p_vout )
                     (p_vout->i_height - i_height) * p_vout->i_bytes_per_line,
                     p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
                     p_vout->i_white_pixel, p_vout->i_black_pixel, 0,
-                    OUTLINED_TEXT, psz_text_1 );
+                    OUTLINED_TEXT, psz_text_1, 100 );
     }
     if( i_width_2 < p_vout->i_width )
     {
@@ -1781,7 +2024,7 @@ static void RenderInterface( vout_thread_t *p_vout )
                     (p_vout->i_height - i_height + i_text_height) * p_vout->i_bytes_per_line,
                     p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
                     p_vout->i_white_pixel, p_vout->i_black_pixel, 0,
-                    OUTLINED_TEXT, psz_text_2 );
+                    OUTLINED_TEXT, psz_text_2, 100 );
     }
 
     /* Activate modified area */
@@ -1795,11 +2038,11 @@ static void RenderInterface( vout_thread_t *p_vout )
  *****************************************************************************/
 static int Manage( vout_thread_t *p_vout )
 {
-#ifdef DEBUG_VIDEO
+#ifdef TRACE_VOUT
     if( p_vout->i_changes )
     {
-        intf_DbgMsg("changes: 0x%x (no display: 0x%x)\n", p_vout->i_changes,
-                    p_vout->i_changes & VOUT_NODISPLAY_CHANGE );
+        intf_DbgMsg("changes: 0x%x (no display: 0x%x)", p_vout->i_changes,
+                    0 /* p_vout->i_changes & VOUT_NODISPLAY_CHANGE */ );
     }
 #endif
 
@@ -1809,23 +2052,24 @@ static int Manage( vout_thread_t *p_vout )
     {
         if( vout_ResetYUV( p_vout ) )
         {
-            intf_ErrMsg("error: can't rebuild convertion tables\n");
+            intf_ErrMsg( "vout error: can't rebuild conversion tables" );
             return( 1 );
         }
     }
 
     /* Clear changes flags which does not need management or have been
      * handled */
-    p_vout->i_changes &= ~(VOUT_GAMMA_CHANGE | VOUT_GRAYSCALE_CHANGE |
-                           VOUT_YUV_CHANGE   | VOUT_INFO_CHANGE |
-                           VOUT_INTF_CHANGE  | VOUT_SCALE_CHANGE );
+    p_vout->i_changes &= ~(VOUT_GAMMA_CHANGE  | VOUT_GRAYSCALE_CHANGE |
+                           VOUT_YUV_CHANGE    | VOUT_INFO_CHANGE |
+                           VOUT_INTF_CHANGE   | VOUT_SCALE_CHANGE |
+                           VOUT_CURSOR_CHANGE | VOUT_FULLSCREEN_CHANGE );
 
     /* Detect unauthorized changes */
     if( p_vout->i_changes )
     {
-        /* Some changes were not acknowledged by vout_SysManage or this function,
-         * it means they should not be authorized */
-        intf_ErrMsg( "error: unauthorized changes in the video output thread\n" );
+        /* Some changes were not acknowledged by p_vout->pf_manage or this
+         * function, it means they should not be authorized */
+        intf_ErrMsg( "vout error: unauthorized changes in the vout thread" );
         return( 1 );
     }
 
@@ -1894,6 +2138,6 @@ static int Align( vout_thread_t *p_vout, int *pi_x, int *pi_y,
 static void     SetPalette        ( p_vout_thread_t p_vout, u16 *red,
                                     u16 *green, u16 *blue, u16 *transp )
 {
-    intf_ErrMsg( "SetPalette: method does not support palette changing\n" );
+    intf_ErrMsg( "vout error: method does not support palette changing" );
 }