]> git.sesse.net Git - vlc/blobdiff - src/video_output/video_output.c
. nouveaux plugins - ne fonctionnent pas encore tous
[vlc] / src / video_output / video_output.c
index ec4fa366996a3485a10b1453c25230a8c8844db3..2514f4d7f465e304909e81573d9587c12792d27d 100644 (file)
@@ -1,35 +1,55 @@
-/******************************************************************************
+/*****************************************************************************
  * 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
+ *
+ * Authors:
+ *
+ * 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() */
 
-#include "common.h"
 #include "config.h"
+#include "common.h"
+#include "threads.h"
 #include "mtime.h"
-#include "vlc_thread.h"
+#include "plugins.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"
 
-/******************************************************************************
+/*****************************************************************************
  * Local prototypes
- ******************************************************************************/
+ *****************************************************************************/
 static int      BinaryLog         ( u32 i );
 static void     MaskToShift       ( int *pi_left, int *pi_right, u32 i_mask );
 static int      InitThread        ( vout_thread_t *p_vout );
@@ -37,7 +57,7 @@ static void     RunThread         ( vout_thread_t *p_vout );
 static void     ErrorThread       ( vout_thread_t *p_vout );
 static void     EndThread         ( vout_thread_t *p_vout );
 static void     DestroyThread     ( vout_thread_t *p_vout, int i_status );
-static void     Print             ( vout_thread_t *p_vout, int i_x, int i_y, 
+static 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 );
 static void     SetBufferArea     ( vout_thread_t *p_vout, int i_x, int i_y,
@@ -50,91 +70,132 @@ static void     RenderSubPicture  ( vout_thread_t *p_vout,
 static void     RenderInterface   ( vout_thread_t *p_vout );
 static int      RenderIdle        ( vout_thread_t *p_vout );
 static void     RenderInfo        ( vout_thread_t *p_vout );
+static void     Synchronize       ( vout_thread_t *p_vout, s64 i_delay );
 static int      Manage            ( vout_thread_t *p_vout );
 static int      Align             ( vout_thread_t *p_vout, int *pi_x,
                                     int *pi_y, int i_width, int i_height,
                                     int i_h_align, int i_v_align );
+static void     SetPalette        ( p_vout_thread_t p_vout, u16 *red,
+                                    u16 *green, u16 *blue, u16 *transp );
 
-/******************************************************************************
+/*****************************************************************************
  * vout_CreateThread: creates a new video output thread
- ******************************************************************************
+ *****************************************************************************
  * This function creates a new video output thread, and returns a pointer
  * to its description. On error, it returns NULL.
  * 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   ( char *psz_display, int i_root_window,
+                          int i_width, int i_height, int *pi_status,
+                          int i_method, void *p_data )
 {
-    vout_thread_t * p_vout;                              /* thread descriptor */
-    int             i_status;                                /* thread status */
-    int             i_index;                /* index for array initialization */
+    vout_thread_t * p_vout;                             /* thread descriptor */
+    int             i_status;                               /* thread status */
+    int             i_index;               /* index for array initialization */
+    char *          psz_method;
 
     /* Allocate descriptor */
-    intf_DbgMsg("\n");    
+    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( "error: %s\n", strerror(ENOMEM) );
+        return( NULL );
+    }
+
+    /* Request an interface plugin */
+    psz_method = main_GetPszVariable( VOUT_METHOD_VAR, VOUT_DEFAULT_METHOD );
+
+    if( RequestPlugin( &p_vout->vout_plugin, psz_method ) < 0 )
+    {
+        intf_ErrMsg( "error: could not open video plugin %s.so\n", psz_method );
+        free( p_vout );
         return( NULL );
     }
 
-    /* Initialize thread properties - thread id and locks will be initialized 
+    /* Get plugins */
+    p_vout->p_sys_create = 
+        GetPluginFunction( p_vout->vout_plugin, "vout_SysCreate" );
+    p_vout->p_sys_init =
+        GetPluginFunction( p_vout->vout_plugin, "vout_SysInit" );
+    p_vout->p_sys_end =
+        GetPluginFunction( p_vout->vout_plugin, "vout_SysEnd" );
+    p_vout->p_sys_destroy =
+        GetPluginFunction( p_vout->vout_plugin, "vout_SysDestroy" );
+    p_vout->p_sys_manage =
+        GetPluginFunction( p_vout->vout_plugin, "vout_SysManage" );
+    p_vout->p_sys_display =
+        GetPluginFunction( p_vout->vout_plugin, "vout_SysDisplay" );
+
+    /* Initialize thread properties - thread id and locks will be initialized
      * later */
     p_vout->b_die               = 0;
-    p_vout->b_error             = 0;    
+    p_vout->b_error             = 0;
     p_vout->b_active            = 0;
     p_vout->pi_status           = (pi_status != NULL) ? pi_status : &i_status;
-    *p_vout->pi_status          = THREAD_CREATE;    
+    *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_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->f_gamma             = VOUT_GAMMA;
 
-    p_vout->b_grayscale         = main_GetIntVariable( VOUT_GRAYSCALE_VAR, VOUT_GRAYSCALE_DEFAULT );
-    p_vout->b_info              = 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             = 0;
-    
-    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 );
+
+    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 idle screen */
     p_vout->last_display_date   = mdate();
+    p_vout->last_display_date   = 0;
     p_vout->last_idle_date      = 0;
 
 #ifdef STATS
     /* Initialize statistics fields */
-    p_vout->render_time         = 0;    
-    p_vout->c_fps_samples       = 0;    
-#endif      
+    p_vout->render_time         = 0;
+    p_vout->c_fps_samples       = 0;
+#endif
 
     /* Initialize buffer index */
     p_vout->i_buffer_index      = 0;
 
     /* Initialize pictures and subpictures - translation tables and functions
-     * will be initialized later in InitThread */    
+     * will be initialized later in InitThread */
     for( i_index = 0; i_index < VOUT_MAX_PICTURES; i_index++)
     {
         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;
+
+    /* Initialize synchronization informations */
+    p_vout->i_synchro_level     = VOUT_SYNCHRO_LEVEL_START;
+
     /* 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->p_sys_create( p_vout, psz_display, i_root_window, p_data ) )
     {
-      free( p_vout );
-      return( NULL );
+        TrashPlugin( p_vout->vout_plugin );
+        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,
@@ -146,81 +207,92 @@ vout_thread_t * vout_CreateThread               ( char *psz_display, int i_root_
     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 usefull colors */
+    /* Set some useful colors */
     p_vout->i_white_pixel = RGB2PIXEL( p_vout, 255, 255, 255 );
     p_vout->i_black_pixel = RGB2PIXEL( p_vout, 0, 0, 0 );
     p_vout->i_gray_pixel  = RGB2PIXEL( p_vout, 128, 128, 128 );
-    p_vout->i_blue_pixel  = RGB2PIXEL( p_vout, 0, 0, 50 );    
+    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( DATA_PATH "/" VOUT_DEFAULT_FONT );
     if( p_vout->p_default_font == NULL )
     {
-        vout_SysDestroy( p_vout );        
-        free( p_vout );        
-        return( NULL );        
-    }    
-    p_vout->p_large_font        = vout_LoadFont( VOUT_LARGE_FONT );        
+        p_vout->p_default_font  = vout_LoadFont( "share/" VOUT_DEFAULT_FONT );
+    }
+    if( p_vout->p_default_font == NULL )
+    {
+        p_vout->p_sys_destroy( p_vout );
+        TrashPlugin( p_vout->vout_plugin );
+        free( p_vout );
+        return( NULL );
+    }
+    p_vout->p_large_font        = vout_LoadFont( DATA_PATH "/" 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 );        
-    }     
+        p_vout->p_large_font    = vout_LoadFont( "share/" VOUT_LARGE_FONT );
+    }
+    if( p_vout->p_large_font == NULL )
+    {
+        vout_UnloadFont( p_vout->p_default_font );
+        p_vout->p_sys_destroy( p_vout );
+        TrashPlugin( p_vout->vout_plugin );
+        free( p_vout );
+        return( NULL );
+    }
 
     /* 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 );    
+    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) )
     {
         intf_ErrMsg("error: %s\n", strerror(ENOMEM));
         vout_UnloadFont( p_vout->p_default_font );
-        vout_UnloadFont( p_vout->p_large_font );        
-       vout_SysDestroy( p_vout );
+        vout_UnloadFont( p_vout->p_large_font );
+        p_vout->p_sys_destroy( p_vout );
+        TrashPlugin( p_vout->vout_plugin );
         free( p_vout );
         return( NULL );
-    }   
+    }
 
-    intf_Msg("Video display initialized (%dx%d, %d/%d bpp)\n", p_vout->i_width, 
+    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 );
 
     /* If status is NULL, wait until the thread is created */
     if( pi_status == NULL )
     {
         do
-        {            
+        {
             msleep( THREAD_SLEEP );
-        }while( (i_status != THREAD_READY) && (i_status != THREAD_ERROR) 
+        }while( (i_status != THREAD_READY) && (i_status != THREAD_ERROR)
                 && (i_status != THREAD_FATAL) );
         if( i_status != THREAD_READY )
         {
-            return( NULL );            
-        }        
+            return( NULL );
+        }
     }
     return( p_vout );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_DestroyThread: destroys a previously created thread
- ******************************************************************************
- * Destroy a terminated thread. 
+ *****************************************************************************
+ * Destroy a terminated thread.
  * The function will request a destruction of the specified thread. If pi_error
- * is NULL, it will return once the thread is destroyed. Else, it will be 
+ * is NULL, it will return once the thread is destroyed. Else, it will be
  * update using one of the THREAD_* constants.
- ******************************************************************************/
+ *****************************************************************************/
 void vout_DestroyThread( vout_thread_t *p_vout, int *pi_status )
-{  
-    int     i_status;                                        /* thread status */
+{
+    int     i_status;                                       /* thread status */
 
     /* Set status */
     intf_DbgMsg("\n");
     p_vout->pi_status = (pi_status != NULL) ? pi_status : &i_status;
-    *p_vout->pi_status = THREAD_DESTROY;    
-     
+    *p_vout->pi_status = THREAD_DESTROY;
+
     /* Request thread destruction */
     p_vout->b_die = 1;
 
@@ -230,32 +302,32 @@ void vout_DestroyThread( vout_thread_t *p_vout, int *pi_status )
         do
         {
             msleep( THREAD_SLEEP );
-        }while( (i_status != THREAD_OVER) && (i_status != THREAD_ERROR) 
-                && (i_status != THREAD_FATAL) );   
+        }while( (i_status != THREAD_OVER) && (i_status != THREAD_ERROR)
+                && (i_status != THREAD_FATAL) );
     }
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_DisplaySubPicture: display a subpicture unit
- ******************************************************************************
- * Remove the reservation flag of an subpicture, which will cause it to be ready 
+ *****************************************************************************
+ * Remove the reservation flag of an subpicture, which will cause it to be ready
  * for display. The picture does not need to be locked, since it is ignored by
  * the output thread if is reserved.
- ******************************************************************************/
+ *****************************************************************************/
 void  vout_DisplaySubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
 {
 #ifdef DEBUG_VIDEO
-    char        psz_begin_date[MSTRTIME_MAX_SIZE];  /* buffer for date string */
-    char        psz_end_date[MSTRTIME_MAX_SIZE];    /* buffer for date string */
+    char        psz_begin_date[MSTRTIME_MAX_SIZE]; /* buffer for date string */
+    char        psz_end_date[MSTRTIME_MAX_SIZE];   /* buffer for date string */
 #endif
 
 #ifdef DEBUG
     /* Check if status is valid */
     if( p_subpic->i_status != RESERVED_SUBPICTURE )
     {
-        intf_DbgMsg("error: subpicture %p has invalid status %d\n", p_subpic, 
-                    p_subpic->i_status );       
-    }   
+        intf_DbgMsg("error: subpicture %p has invalid status %d\n", p_subpic,
+                    p_subpic->i_status );
+    }
 #endif
 
     /* Remove reservation flag */
@@ -263,74 +335,75 @@ void  vout_DisplaySubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
 
 #ifdef DEBUG_VIDEO
     /* Send subpicture informations */
-    intf_DbgMsg("subpicture %p: type=%d, begin date=%s, end date=%s\n", 
-                p_subpic, p_subpic->i_type, 
-                mstrtime( psz_begin_date, p_subpic->begin_date ), 
-                mstrtime( psz_end_date, p_subpic->end_date ) );    
+    intf_DbgMsg("subpicture %p: type=%d, begin date=%s, end date=%s\n",
+                p_subpic, p_subpic->i_type,
+                mstrtime( psz_begin_date, p_subpic->begin_date ),
+                mstrtime( psz_end_date, p_subpic->end_date ) );
 #endif
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_CreateSubPicture: allocate an subpicture in the video output heap.
- ******************************************************************************
- * This function create a reserved 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
  * already allocated zone of memory in the spu data fields. It needs locking
- * since several pictures can be created by several producers threads. 
- ******************************************************************************/
-subpicture_t *vout_CreateSubPicture( vout_thread_t *p_vout, int i_type, 
+ * since several pictures can be created by several producers threads.
+ *****************************************************************************/
+subpicture_t *vout_CreateSubPicture( vout_thread_t *p_vout, int i_type,
                                      int i_size )
 {
-    int                 i_subpic;                          /* subpicture index */
-    subpicture_t *      p_free_subpic = NULL;         /* first free subpicture */    
-    subpicture_t *      p_destroyed_subpic = NULL;   /* first destroyed subpic */
+    int                 i_subpic;                        /* subpicture index */
+    subpicture_t *      p_free_subpic = NULL;       /* first free subpicture */
+    subpicture_t *      p_destroyed_subpic = NULL; /* first destroyed subpic */
 
     /* Get lock */
     vlc_mutex_lock( &p_vout->subpicture_lock );
 
-    /* 
-     * Look for an empty place 
+    /*
+     * Look for an empty place
      */
-    for( i_subpic = 0; i_subpic < VOUT_MAX_PICTURES; 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;
+    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", 
-                            &p_vout->p_subpicture[i_subpic] );                
+                intf_DbgMsg("subpicture %p (in destroyed subpicture slot)\n",
+                            &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];                
-           }       
-       }
-        else if( (p_free_subpic == NULL) && 
+                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( (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 */
-        free( p_destroyed_subpic->p_data );        
+    {
+        /* No free subpicture or matching destroyed subpicture has been
+         * found, but a destroyed subpicture is still avalaible */
+        free( p_destroyed_subpic->p_data );
         p_free_subpic = p_destroyed_subpic;
     }
 
@@ -342,99 +415,102 @@ subpicture_t *vout_CreateSubPicture( vout_thread_t *p_vout, int i_type,
         /* Allocate memory */
         switch( i_type )
         {
-        case TEXT_SUBPICTURE:                               /* text subpicture */
-            p_free_subpic->p_data = malloc( i_size + 1 );             
+        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 );
-            p_free_subpic->p_data   =  NULL;            
-            break;            
-#endif    
+            p_free_subpic->p_data   =  NULL;
+            break;
+#endif
         }
 
         if( p_free_subpic->p_data != NULL )
-        {                    /* Copy subpicture informations, set some default values */
+        {           /* Copy subpicture informations, 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;            
+            p_free_subpic->i_size                      = i_size;
             p_free_subpic->i_x                         = 0;
-            p_free_subpic->i_y                         = 0;             
+            p_free_subpic->i_y                         = 0;
             p_free_subpic->i_width                     = 0;
             p_free_subpic->i_height                    = 0;
-            p_free_subpic->i_horizontal_align          = CENTER_RALIGN;             
-            p_free_subpic->i_vertical_align            = CENTER_RALIGN;             
+            p_free_subpic->i_horizontal_align          = CENTER_RALIGN;
+            p_free_subpic->i_vertical_align            = CENTER_RALIGN;
         }
         else
         {
             /* Memory allocation failed : set subpicture as empty */
-            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 ) );            
+            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 ) );
         }
-        
+
 #ifdef DEBUG_VIDEO
-        intf_DbgMsg("subpicture %p (in free subpicture slot)\n", p_free_subpic );        
+        intf_DbgMsg("subpicture %p (in free subpicture slot)\n", 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" );
     vlc_mutex_unlock( &p_vout->subpicture_lock );
     return( NULL );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_DestroySubPicture: remove a subpicture from the heap
- ******************************************************************************
+ *****************************************************************************
  * This function frees a previously reserved subpicture.
  * It is meant to be used when the construction of a picture aborted.
- * This function does not need locking since reserved subpictures are ignored 
+ * This function does not need locking since reserved subpictures are ignored
  * by the output thread.
- ******************************************************************************/
+ *****************************************************************************/
 void vout_DestroySubPicture( vout_thread_t *p_vout, subpicture_t *p_subpic )
 {
 #ifdef DEBUG
    /* Check if status is valid */
    if( p_subpic->i_status != RESERVED_SUBPICTURE )
    {
-       intf_DbgMsg("error: subpicture %p has invalid status %d\n", 
-                   p_subpic, p_subpic->i_status );       
-   }   
+       intf_DbgMsg("error: subpicture %p has invalid status %d\n",
+                   p_subpic, p_subpic->i_status );
+   }
 #endif
 
     p_subpic->i_status = DESTROYED_SUBPICTURE;
 
 #ifdef DEBUG_VIDEO
-    intf_DbgMsg("subpicture %p\n", p_subpic);    
+    intf_DbgMsg("subpicture %p\n", p_subpic);
 #endif
 }
 
-/*******************************************************************************
+/*****************************************************************************
  * vout_DisplayPicture: display a picture
- *******************************************************************************
+ *****************************************************************************
  * Remove the reservation flag of a picture, which will cause it to be ready for
- * display. The picture won't be displayed until vout_DatePicture has been 
+ * display. The picture won't be displayed until vout_DatePicture has been
  * called.
- *******************************************************************************/
+ *****************************************************************************/
 void  vout_DisplayPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
     vlc_mutex_lock( &p_vout->picture_lock );
     switch( p_pic->i_status )
     {
-    case RESERVED_PICTURE:        
+    case RESERVED_PICTURE:
         p_pic->i_status = RESERVED_DISP_PICTURE;
-        break;        
+        break;
     case RESERVED_DATED_PICTURE:
         p_pic->i_status = READY_PICTURE;
-        break;        
+        break;
 #ifdef DEBUG
-    default:        
-        intf_DbgMsg("error: picture %p has invalid status %d\n", p_pic, p_pic->i_status );    
-        break;        
+    default:
+        intf_DbgMsg("error: picture %p has invalid status %d\n", p_pic, p_pic->i_status );
+        break;
 #endif
     }
 
@@ -444,33 +520,33 @@ void  vout_DisplayPicture( vout_thread_t *p_vout, picture_t *p_pic )
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
-/*******************************************************************************
+/*****************************************************************************
  * vout_DatePicture: date a picture
- *******************************************************************************
+ *****************************************************************************
  * Remove the reservation flag of a picture, which will cause it to be ready for
- * display. The picture won't be displayed until vout_DisplayPicture has been 
+ * display. The picture won't be displayed until vout_DisplayPicture has been
  * called.
- *******************************************************************************/
+ *****************************************************************************/
 void  vout_DatePicture( vout_thread_t *p_vout, picture_t *p_pic, mtime_t date )
 {
 #ifdef DEBUG_VIDEO
-    char        psz_date[MSTRTIME_MAX_SIZE];                           /* date */
+    char        psz_date[MSTRTIME_MAX_SIZE];                         /* date */
 #endif
 
     vlc_mutex_lock( &p_vout->picture_lock );
-    p_pic->date = date;    
+    p_pic->date = date;
     switch( p_pic->i_status )
     {
-    case RESERVED_PICTURE:        
+    case RESERVED_PICTURE:
         p_pic->i_status = RESERVED_DATED_PICTURE;
-        break;        
+        break;
     case RESERVED_DISP_PICTURE:
         p_pic->i_status = READY_PICTURE;
-        break;        
+        break;
 #ifdef DEBUG
-    default:        
-        intf_DbgMsg("error: picture %p has invalid status %d\n", p_pic, p_pic->i_status );    
-        break;        
+    default:
+        intf_DbgMsg("error: picture %p has invalid status %d\n", p_pic, p_pic->i_status );
+        break;
 #endif
     }
 
@@ -480,72 +556,73 @@ void  vout_DatePicture( vout_thread_t *p_vout, picture_t *p_pic, mtime_t date )
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_CreatePicture: allocate a picture in the video output heap.
- ******************************************************************************
- * This function create a reserved image in the video output heap. 
+ *****************************************************************************
+ * This function create a reserved image in the video output heap.
  * A null pointer is returned if the function fails. This method provides an
  * already allocated zone of memory in the picture data fields. It needs locking
- * 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 )
+ * 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_picture;                                   /* picture index */
-    int         i_chroma_width = 0;                           /* chroma width */    
-    picture_t * p_free_picture = NULL;                  /* first free picture */    
-    picture_t * p_destroyed_picture = NULL;        /* first destroyed picture */    
+    int         i_picture;                                  /* picture index */
+    int         i_chroma_width = 0;                          /* chroma width */
+    picture_t * p_free_picture = NULL;                 /* first free picture */
+    picture_t * p_destroyed_picture = NULL;       /* first destroyed picture */
 
     /* Get lock */
     vlc_mutex_lock( &p_vout->picture_lock );
 
-    /* 
-     * Look for an empty place 
+    /*
+     * Look for an empty place
      */
     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
-             * that if width and type are the same for two pictures, chroma_width 
+        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;
+                p_vout->p_picture[i_picture].i_status = RESERVED_PICTURE;
+                p_vout->i_pictures++;
 #ifdef DEBUG_VIDEO
-                intf_DbgMsg("picture %p (in destroyed picture slot)\n", 
-                            &p_vout->p_picture[i_picture] );                
+                intf_DbgMsg("picture %p (in destroyed picture slot)\n",
+                            &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];                
-           }       
-       }
-        else if( (p_free_picture == NULL) && 
+                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 */
-            p_free_picture = &p_vout->p_picture[i_picture];            
+            /* Picture is empty and ready for allocation */
+            p_free_picture = &p_vout->p_picture[i_picture];
         }
     }
 
     /* 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 */
-        free( p_destroyed_picture->p_data );        
-        p_free_picture = p_destroyed_picture;        
+    {
+        /* 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;
     }
 
     /*
@@ -556,124 +633,127 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout, int i_type,
         /* Allocate memory */
         switch( i_type )
         {
-        case YUV_420_PICTURE:          /* YUV 420: 1,1/4,1/4 samples per pixel */
-            i_chroma_width = i_width / 2;            
+        case YUV_420_PICTURE:        /* YUV 420: 1,1/4,1/4 samples per pixel */
+            i_chroma_width = i_width / 2;
             p_free_picture->p_data = malloc( i_height * i_chroma_width * 3 * sizeof( yuv_data_t ) );
             p_free_picture->p_y = (yuv_data_t *)p_free_picture->p_data;
             p_free_picture->p_u = (yuv_data_t *)p_free_picture->p_data +i_height*i_chroma_width*4/2;
             p_free_picture->p_v = (yuv_data_t *)p_free_picture->p_data +i_height*i_chroma_width*5/2;
             break;
-        case YUV_422_PICTURE:          /* YUV 422: 1,1/2,1/2 samples per pixel */
-            i_chroma_width = i_width / 2;            
+        case YUV_422_PICTURE:        /* YUV 422: 1,1/2,1/2 samples per pixel */
+            i_chroma_width = i_width / 2;
             p_free_picture->p_data = malloc( i_height * i_chroma_width * 4 * sizeof( yuv_data_t ) );
             p_free_picture->p_y = (yuv_data_t *)p_free_picture->p_data;
             p_free_picture->p_u = (yuv_data_t *)p_free_picture->p_data +i_height*i_chroma_width*2;
             p_free_picture->p_v = (yuv_data_t *)p_free_picture->p_data +i_height*i_chroma_width*3;
             break;
-        case YUV_444_PICTURE:              /* YUV 444: 1,1,1 samples per pixel */
-            i_chroma_width = i_width;            
+        case YUV_444_PICTURE:            /* YUV 444: 1,1,1 samples per pixel */
+            i_chroma_width = i_width;
             p_free_picture->p_data = malloc( i_height * i_chroma_width * 3 * sizeof( yuv_data_t ) );
             p_free_picture->p_y = (yuv_data_t *)p_free_picture->p_data;
             p_free_picture->p_u = (yuv_data_t *)p_free_picture->p_data +i_height*i_chroma_width;
             p_free_picture->p_v = (yuv_data_t *)p_free_picture->p_data +i_height*i_chroma_width*2;
-            break;                
+            break;
 #ifdef DEBUG
         default:
             intf_DbgMsg("error: unknown picture type %d\n", i_type );
-            p_free_picture->p_data   =  NULL;            
-            break;            
-#endif    
+            p_free_picture->p_data   =  NULL;
+            break;
+#endif
         }
 
         if( p_free_picture->p_data != NULL )
-        {        
+        {
             /* Copy picture informations, 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_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_chroma_width              = i_chroma_width;
             p_free_picture->i_display_horizontal_offset = 0;
-            p_free_picture->i_display_vertical_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_free_picture->i_aspect_ratio              = AR_SQUARE_PICTURE;
+            p_free_picture->i_refcount                  = 0;
+            p_vout->i_pictures++;
         }
         else
         {
             /* Memory allocation failed : set picture as empty */
-            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 ) );            
+            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 ) );
         }
-        
+
 #ifdef DEBUG_VIDEO
-        intf_DbgMsg("picture %p (in free picture slot)\n", p_free_picture );        
+        intf_DbgMsg("picture %p (in free picture slot)\n", p_free_picture );
 #endif
         vlc_mutex_unlock( &p_vout->picture_lock );
         return( p_free_picture );
     }
-    
+
     /* No free or destroyed picture could be found */
     intf_DbgMsg( "warning: heap is full\n" );
     vlc_mutex_unlock( &p_vout->picture_lock );
     return( NULL );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_DestroyPicture: remove a permanent or reserved picture from the heap
- ******************************************************************************
+ *****************************************************************************
  * 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) && 
+   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 );       
-   }   
+       intf_DbgMsg("error: picture %p has invalid status %d\n", p_pic, p_pic->i_status );
+   }
 #endif
 
-    p_pic->i_status = DESTROYED_PICTURE;
+   p_pic->i_status = DESTROYED_PICTURE;
+   p_vout->i_pictures--;
 
 #ifdef DEBUG_VIDEO
-    intf_DbgMsg("picture %p\n", p_pic);    
+   intf_DbgMsg("picture %p\n", p_pic);
 #endif
+   vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_LinkPicture: increment reference counter of a picture
- ******************************************************************************
+ *****************************************************************************
  * This function increment the reference counter of a picture in the video
  * heap. It needs a lock since several producer threads can access the picture.
- ******************************************************************************/
+ *****************************************************************************/
 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 );    
+    intf_DbgMsg("picture %p refcount=%d\n", p_pic, p_pic->i_refcount );
 #endif
 
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_UnlinkPicture: decrement reference counter of a picture
- ******************************************************************************
+ *****************************************************************************
  * This function decrement the reference counter of a picture in the video heap.
- ******************************************************************************/
+ *****************************************************************************/
 void vout_UnlinkPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
     vlc_mutex_lock( &p_vout->picture_lock );
@@ -683,28 +763,29 @@ void vout_UnlinkPicture( vout_thread_t *p_vout, picture_t *p_pic )
     if( p_pic->i_refcount < 0 )
     {
         intf_DbgMsg("error: refcount < 0\n");
-        p_pic->i_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 );    
+    intf_DbgMsg("picture %p refcount=%d\n", p_pic, p_pic->i_refcount );
 #endif
 
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * vout_SetBuffers: set buffers adresses
- ******************************************************************************
- * This function is called by system drivers to set buffers video memory 
+ *****************************************************************************
+ * This function is called by system drivers to set buffers video memory
  * adresses.
- ******************************************************************************/
+ *****************************************************************************/
 void vout_SetBuffers( vout_thread_t *p_vout, void *p_buf1, void *p_buf2 )
 {
     /* No picture previously */
@@ -726,8 +807,8 @@ void vout_SetBuffers( vout_thread_t *p_vout, void *p_buf1, void *p_buf2 )
     p_vout->p_buffer[1].pi_area_end[0] =          p_vout->i_height - 1;
 
     /* Set adresses */
-    p_vout->p_buffer[0].p_data = p_buf1;    
-    p_vout->p_buffer[1].p_data = p_buf2;    
+    p_vout->p_buffer[0].p_data = p_buf1;
+    p_vout->p_buffer[1].p_data = p_buf2;
 }
 
 /*****************************************************************************
@@ -748,7 +829,7 @@ void vout_Pixel2RGB( vout_thread_t *p_vout, u32 i_pixel, int *pi_red, int *pi_gr
 /*****************************************************************************
  * BinaryLog: computes the base 2 log of a binary value
  *****************************************************************************
- * This functions is used by MaskToShift, to get a bit index from a binary 
+ * This functions is used by MaskToShift, to get a bit index from a binary
  * value.
  *****************************************************************************/
 static int BinaryLog(u32 i)
@@ -756,30 +837,30 @@ static int BinaryLog(u32 i)
     int i_log;
 
     i_log = 0;
-    if (i & 0xffff0000) 
-    {        
+    if (i & 0xffff0000)
+    {
         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++;
-    }    
+    }
     if (i != ((u32)1 << i_log))
-    {        
-       intf_ErrMsg("internal error: binary log overflow\n");        
-    }    
+    {
+        intf_ErrMsg("internal error: binary log overflow\n");
+    }
 
     return( i_log );
 }
@@ -806,64 +887,68 @@ static void MaskToShift( int *pi_left, int *pi_right, u32 i_mask )
     *pi_right = (8 - i_high + i_low);
 }
 
-/******************************************************************************
+/*****************************************************************************
  * InitThread: initialize video output thread
- ******************************************************************************
+ *****************************************************************************
  * This function is called from RunThread and performs the second step of the
  * initialization. It returns 0 on success. Note that the thread's flag are not
  * modified inside this function.
- ******************************************************************************/
+ *****************************************************************************/
 static int InitThread( vout_thread_t *p_vout )
 {
     /* Update status */
     intf_DbgMsg("\n");
-    *p_vout->pi_status = THREAD_START;    
+    *p_vout->pi_status = THREAD_START;
 
-    /* Initialize output method - this function issues its own error messages */
-    if( vout_SysInit( p_vout ) )
+   /* Initialize output method - this function issues its own error messages */
+    if( p_vout->p_sys_init( p_vout ) )
     {
         return( 1 );
-    } 
+    }
 
     /* Initialize convertion tables and functions */
     if( vout_InitYUV( p_vout ) )
     {
         intf_ErrMsg("error: can't allocate YUV translation tables\n");
-        return( 1 );                
+        return( 1 );
     }
-    
+
     /* Mark thread as running and return */
-    p_vout->b_active =          1;    
-    *p_vout->pi_status =        THREAD_READY;    
-    intf_DbgMsg("thread ready\n");    
-    return( 0 );    
+    p_vout->b_active =          1;
+    *p_vout->pi_status =        THREAD_READY;
+    intf_DbgMsg("thread ready\n");
+    return( 0 );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * RunThread: video output thread
- ******************************************************************************
+ *****************************************************************************
  * Video output thread. This function does only returns when the thread is
  * terminated. It handles the pictures arriving in the video heap and the
  * display device events.
- ******************************************************************************/
+ *****************************************************************************/
 static void RunThread( vout_thread_t *p_vout)
 {
-    int             i_index;                                 /* index in heap */
-    mtime_t         current_date;                             /* current date */
-    mtime_t         display_date;                             /* display date */    
-    boolean_t       b_display;                                /* display flag */    
-    picture_t *     p_pic;                                 /* picture pointer */
-    subpicture_t *  p_subpic;                           /* subpicture pointer */    
-     
-    /* 
+    /* XXX?? welcome to gore land */
+    static int i_trash_count = 0;
+    static mtime_t last_display_date = 0;
+
+    int             i_index;                                /* index in heap */
+    mtime_t         current_date;                            /* current date */
+    mtime_t         display_date;                            /* display date */
+    boolean_t       b_display;                               /* display flag */
+    picture_t *     p_pic;                                /* picture pointer */
+    subpicture_t *  p_subpic;                          /* subpicture pointer */
+
+    /*
      * Initialize thread
      */
     p_vout->b_error = InitThread( p_vout );
     if( p_vout->b_error )
     {
         DestroyThread( p_vout, THREAD_ERROR );
-        return;        
-    }    
+        return;
+    }
     intf_DbgMsg("\n");
 
     /*
@@ -875,157 +960,167 @@ static void RunThread( vout_thread_t *p_vout)
         /* Initialize loop variables */
         p_pic =         NULL;
         p_subpic =      NULL;
-        display_date =  0;        
+        display_date =  0;
         current_date =  mdate();
 
-        /* 
-        * Find the picture to display - this operation does not need lock,
-         * since only READY_PICTUREs are handled 
+        /*
+         * 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;                
-           }
-       }
+                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 */
+#endif
+/* XXX?? */
+i_trash_count++;
+//fprintf( stderr, "gap : %Ld\n", display_date-last_display_date );
+last_display_date = display_date;
+#if 1
+            if( display_date < current_date && i_trash_count > 4 )
+            {
+                /* 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_DbgMsg( "warning: late picture %p skipped refcount=%d\n", p_pic, p_pic->i_refcount );
                 vlc_mutex_unlock( &p_vout->picture_lock );
-                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;                
-           }
-        }
 
+                /* Update synchronization information as if display delay
+                 * was 0 */
+                Synchronize( p_vout, display_date - current_date );
+
+                p_pic =         NULL;
+                display_date =  0;
+                i_trash_count = 0;
+            }
+            else
+#endif
+                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
+            {
+                /* Picture will be displayed, update synchronization
+                 * information */
+                Synchronize( p_vout, display_date - current_date );
+            }
+        }
         /*
-         * 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
          */
-        if( p_pic )                          /* picture and perhaps subpicture */
+        if( p_pic )                        /* picture and perhaps subpicture */
         {
-            b_display = p_vout->b_active;            
+            b_display = p_vout->b_active;
             p_vout->last_display_date = display_date;
-            
+
             if( b_display )
-            {                
+            {
                 /* Set picture dimensions and clear buffer */
                 SetBufferPicture( p_vout, p_pic );
 
                 /* Render picture and informations */
-                RenderPicture( p_vout, p_pic );             
+                RenderPicture( p_vout, p_pic );
                 if( p_vout->b_info )
                 {
                     RenderPictureInfo( p_vout, p_pic );
-                    RenderInfo( p_vout );                
+                    RenderInfo( p_vout );
+                }
+                if( p_subpic )
+                {
+                    RenderSubPicture( p_vout, p_subpic );
                 }
             }
-            
+
             /* 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 )
+            if( p_pic->i_refcount )
             {
-                RenderInterface( p_vout );                
+                p_pic->i_status = DISPLAYED_PICTURE;
             }
-            if( p_subpic )
+            else
             {
-                if( b_display )
-                {                    
-                    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 );                          
+                p_pic->i_status = DESTROYED_PICTURE;
+                p_vout->i_pictures--;
             }
+            vlc_mutex_unlock( &p_vout->picture_lock );
 
-        }
-        else if( p_subpic )                                /* subpicture alone */
-        {
-            b_display = p_vout->b_active;
-            p_vout->last_display_date = display_date;            
-
-            if( b_display )
-            {                
-                /* 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 );            
-            }            
+            /* Render interface and subpicture */
+            if( b_display && p_vout->b_interface )
+            {
+                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 )        /* idle or interface screen alone */
         {
-            if( p_vout->b_interface && 0 /* && ?? intf_change */ )
+            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 */
-                p_vout->last_idle_date = 0;                
+                p_vout->last_idle_date = 0;
             }
 
             /* Render idle screen and update idle date, then render interface if
              * required */
             b_display = RenderIdle( p_vout );
             if( b_display )
-            {                
+            {
                 p_vout->last_idle_date = current_date;
                 if( p_vout->b_interface )
                 {
-                    RenderInterface( p_vout );                
+                    RenderInterface( p_vout );
                 }
             }
-            
-        }       
+
+        }
         else
         {
-            b_display = 0;            
-        }        
+            b_display = 0;
+        }
 
         /*
          * Sleep, wake up and display rendered picture
@@ -1037,7 +1132,7 @@ static void RunThread( vout_thread_t *p_vout)
 #endif
 
         /* Give back change lock */
-        vlc_mutex_unlock( &p_vout->change_lock );        
+        vlc_mutex_unlock( &p_vout->change_lock );
 
         /* Sleep a while or until a given date */
         if( display_date != 0 )
@@ -1046,55 +1141,57 @@ static void RunThread( vout_thread_t *p_vout)
         }
         else
         {
-            msleep( VOUT_IDLE_SLEEP );                
-        }            
+            msleep( VOUT_IDLE_SLEEP );
+        }
 
-        /* On awakening, take back lock and send immediately picture to display, 
+        /* On awakening, take back lock and send immediately picture to display,
          * then swap buffers */
-        vlc_mutex_lock( &p_vout->change_lock );        
+        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) );        
-#endif            
+                     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) )
         {
-            vout_SysDisplay( p_vout );
+            p_vout->p_sys_display( p_vout );
+#ifndef SYS_BEOS
             p_vout->i_buffer_index = ++p_vout->i_buffer_index & 1;
+#endif
         }
 
         /*
          * 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->p_sys_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;
+        }
+    }
 
     /*
      * Error loop - wait until the thread destruction is requested
      */
     if( p_vout->b_error )
     {
-        ErrorThread( p_vout );        
+        ErrorThread( p_vout );
     }
 
     /* End of thread */
     EndThread( p_vout );
-    DestroyThread( p_vout, THREAD_OVER ); 
+    DestroyThread( p_vout, THREAD_OVER );
     intf_DbgMsg( "thread end\n" );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * ErrorThread: RunThread() error loop
- ******************************************************************************
+ *****************************************************************************
  * This function is called when an error occured during thread main's loop. The
  * thread can still receive feed, but must be ready to terminate as soon as
  * possible.
- ******************************************************************************/
+ *****************************************************************************/
 static void ErrorThread( vout_thread_t *p_vout )
 {
     /* Wait until a `die' order */
@@ -1102,109 +1199,117 @@ static void ErrorThread( vout_thread_t *p_vout )
     while( !p_vout->b_die )
     {
         /* Sleep a while */
-        msleep( VOUT_IDLE_SLEEP );                
+        msleep( VOUT_IDLE_SLEEP );
     }
 }
 
-/*******************************************************************************
+/*****************************************************************************
  * EndThread: thread destruction
- *******************************************************************************
- * This function is called when the thread ends after a sucessful
+ *****************************************************************************
+ * 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 )
 {
-    int     i_index;                                          /* index in heap */
-            
+    int     i_index;                                        /* index in heap */
+
     /* Store status */
     intf_DbgMsg("\n");
-    *p_vout->pi_status = THREAD_END;    
+    *p_vout->pi_status = THREAD_END;
 
     /* 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 );            
-        }        
+            free( p_vout->p_subpicture[i_index].p_data );
+        }
     }
 
     /* Destroy translation tables */
-    vout_EndYUV( p_vout );  
-    vout_SysEnd( p_vout );    
+    vout_EndYUV( p_vout );
+    p_vout->p_sys_end( p_vout );
 }
 
-/*******************************************************************************
+/*****************************************************************************
  * DestroyThread: thread destruction
- *******************************************************************************
+ *****************************************************************************
  * This function is called when the thread ends. It frees all ressources
  * allocated by CreateThread. Status is available at this stage.
- *******************************************************************************/
+ *****************************************************************************/
 static void DestroyThread( vout_thread_t *p_vout, int i_status )
-{  
-    int *pi_status;                                           /* status adress */
-    
+{
+    int *pi_status;                                         /* status adress */
+
     /* Store status adress */
     intf_DbgMsg("\n");
-    pi_status = p_vout->pi_status;    
-    
+    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 );
+    vout_UnloadFont( p_vout->p_large_font );
+    p_vout->p_sys_destroy( p_vout );
+
+    /* Close plugin */
+    TrashPlugin( p_vout->vout_plugin );
+
+    /* Free structure */
     free( p_vout );
-    *pi_status = i_status;    
+    *pi_status = 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.
- *******************************************************************************/
+ *****************************************************************************/
 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 )
 {
-    int                 i_text_height;                    /* total text height */
-    int                 i_text_width;                      /* total text width */
+    int                 i_text_height;                  /* total text height */
+    int                 i_text_width;                    /* total text width */
 
     /* Update upper left coordinates according to alignment */
     vout_TextSize( p_vout->p_default_font, 0, psz_text, &i_text_width, &i_text_height );
     if( !Align( p_vout, &i_x, &i_y, i_text_width, i_text_height, i_h_align, i_v_align ) )
     {
         /* Set area and print text */
-        SetBufferArea( p_vout, i_x, i_y, i_text_width, i_text_height );    
-        vout_Print( p_vout->p_default_font, p_vout->p_buffer[ p_vout->i_buffer_index ].p_data + 
+        SetBufferArea( p_vout, i_x, i_y, i_text_width, i_text_height );
+        vout_Print( p_vout->p_default_font, p_vout->p_buffer[ p_vout->i_buffer_index ].p_data +
                     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, 
+                    p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
+                    p_vout->i_white_pixel, 0, 0,
                     0, psz_text );
-    }    
+    }
 }
 
-/*******************************************************************************
+/*****************************************************************************
  * SetBufferArea: activate an area in current buffer
- *******************************************************************************
+ *****************************************************************************
  * This function is called when something is rendered on the current buffer.
  * It set the area as active and prepare it to be cleared on next rendering.
  * Pay attention to the fact that in this functions, i_h is in fact the end y
  * coordinate of the new area.
- *******************************************************************************/
+ *****************************************************************************/
 static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int i_h )
 {
-    vout_buffer_t *     p_buffer;                            /* current buffer */
-    int                 i_area_begin, i_area_end;   /* area vertical extension */
-    int                 i_area, i_area_copy;                     /* area index */    
-    int                 i_area_shift;              /* shift distance for areas */    
-    
+    vout_buffer_t *     p_buffer;                          /* current buffer */
+    int                 i_area_begin, i_area_end; /* area vertical extension */
+    int                 i_area, i_area_copy;                   /* area index */
+    int                 i_area_shift;            /* shift distance for areas */
+
     /* Choose buffer and modify h to end of area position */
-    p_buffer =  &p_vout->p_buffer[ p_vout->i_buffer_index ];    
+    p_buffer =  &p_vout->p_buffer[ p_vout->i_buffer_index ];
     i_h +=      i_y - 1;
-    /* 
+
+    /*
      * Remove part of the area which is inside the picture - this is done
      * by calling again SetBufferArea with the correct areas dimensions.
      */
@@ -1216,74 +1321,74 @@ static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int
         if( ((i_y >= i_area_begin) && (i_y <= i_area_end)) ||
             ((i_h >= i_area_begin) && (i_h <= i_area_end)) ||
             ((i_y <  i_area_begin) && (i_h > i_area_end)) )
-        {                    
+        {
             /* Keep the stripe above the picture, if any */
             if( i_y < i_area_begin )
             {
-                SetBufferArea( p_vout, i_x, i_y, i_w, i_area_begin - i_y );            
+                SetBufferArea( p_vout, i_x, i_y, i_w, i_area_begin - i_y );
             }
             /* Keep the stripe below the picture, if any */
             if( i_h > i_area_end )
             {
                 SetBufferArea( p_vout, i_x, i_area_end, i_w, i_h - i_area_end );
-            }        
+            }
             return;
-        }        
-    }   
+        }
+    }
 
     /* Skip some extensions until interesting areas */
-    for( i_area = 0; 
+    for( i_area = 0;
          (i_area < p_buffer->i_areas) &&
-             (p_buffer->pi_area_end[i_area] + 1 <= i_y); 
+             (p_buffer->pi_area_end[i_area] + 1 <= i_y);
          i_area++ )
     {
-        ;        
+        ;
     }
-    
+
     if( i_area == p_buffer->i_areas )
     {
-        /* New area is below all existing ones: just add it at the end of the 
+        /* 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 */
         if( i_area < VOUT_MAX_AREAS )
         {
             p_buffer->pi_area_begin[i_area] = i_y;
-            p_buffer->pi_area_end[i_area] = i_h;            
-            p_buffer->i_areas++;            
+            p_buffer->pi_area_end[i_area] = i_h;
+            p_buffer->i_areas++;
         }
         else
         {
 #ifdef DEBUG_VIDEO
-            intf_DbgMsg("areas overflow\n");            
+            intf_DbgMsg("areas overflow\n");
 #endif
-            p_buffer->pi_area_end[VOUT_MAX_AREAS - 1] = i_h;  
-        }        
+            p_buffer->pi_area_end[VOUT_MAX_AREAS - 1] = i_h;
+        }
     }
-    else 
+    else
     {
         i_area_begin =  p_buffer->pi_area_begin[i_area];
         i_area_end =    p_buffer->pi_area_end[i_area];
-        
-        if( i_y < i_area_begin ) 
+
+        if( i_y < i_area_begin )
         {
             if( i_h >= i_area_begin - 1 )
-            {                
+            {
                 /* Extend area above */
                 p_buffer->pi_area_begin[i_area] = i_y;
             }
             else
             {
-                /* Create a new area above : merge last area if overflow, then 
+                /* Create a new area above : merge last area if overflow, then
                  * move all old areas down */
                 if( p_buffer->i_areas == VOUT_MAX_AREAS )
-                {                    
+                {
 #ifdef DEBUG_VIDEO
-                    intf_DbgMsg("areas overflow\n");       
+                    intf_DbgMsg("areas overflow\n");
 #endif
-                    p_buffer->pi_area_end[VOUT_MAX_AREAS - 2] = p_buffer->pi_area_end[VOUT_MAX_AREAS - 1];                    
+                    p_buffer->pi_area_end[VOUT_MAX_AREAS - 2] = p_buffer->pi_area_end[VOUT_MAX_AREAS - 1];
                 }
                 else
                 {
-                    p_buffer->i_areas++;                    
+                    p_buffer->i_areas++;
                 }
                 for( i_area_copy = p_buffer->i_areas - 1; i_area_copy > i_area; i_area_copy-- )
                 {
@@ -1293,19 +1398,19 @@ static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int
                 p_buffer->pi_area_begin[i_area] = i_y;
                 p_buffer->pi_area_end[i_area] = i_h;
                 return;
-            }              
+            }
         }
         if( i_h > i_area_end )
         {
             /* Find further areas which can be merged with the new one */
-            for( i_area_copy = i_area + 1; 
+            for( i_area_copy = i_area + 1;
                  (i_area_copy < p_buffer->i_areas) &&
                      (p_buffer->pi_area_begin[i_area] <= i_h);
                  i_area_copy++ )
             {
-                ;                
+                ;
             }
-            i_area_copy--;            
+            i_area_copy--;
 
             if( i_area_copy != i_area )
             {
@@ -1313,7 +1418,7 @@ static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int
                 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;                
+                i_area_shift = i_area_copy - i_area;
                 p_buffer->i_areas -= i_area_shift;
                 for( i_area_copy = i_area + 1; i_area_copy < p_buffer->i_areas; i_area_copy++ )
                 {
@@ -1325,95 +1430,95 @@ static void SetBufferArea( vout_thread_t *p_vout, int i_x, int i_y, int i_w, int
             {
                 /* Extend area below */
                 p_buffer->pi_area_end[i_area] = i_h;
-            }            
+            }
         }
     }
 }
 
-/*******************************************************************************
+/*****************************************************************************
  * SetBufferPicture: clear buffer and set picture area
- *******************************************************************************
- * This function is called before any rendering. It clears the current 
+ *****************************************************************************
+ * This function is called before any rendering. It clears the current
  * rendering buffer and set the new picture area. If the picture pointer is
  * NULL, then no picture area is defined. Floating operations are avoided since
  * some MMX calculations may follow.
- *******************************************************************************/
+ *****************************************************************************/
 static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
-    vout_buffer_t *     p_buffer;                            /* current buffer */
-    int                 i_pic_x, i_pic_y;                  /* picture position */
-    int                 i_pic_width, i_pic_height;       /* picture dimensions */    
-    int                 i_old_pic_y, i_old_pic_height;     /* old picture area */    
-    int                 i_vout_width, i_vout_height;     /* display dimensions */
-    int                 i_area;                                  /* area index */    
-    int                 i_data_index;                       /* area data index */    
-    int                 i_data_size;     /* area data size, in 256 bytes blocs */
-    u64 *               p_data;                     /* area data, for clearing */    
-    byte_t *            p_data8;             /* area data, for clearing (slow) */
-    
+    vout_buffer_t *     p_buffer;                          /* current buffer */
+    int                 i_pic_x, i_pic_y;                /* picture position */
+    int                 i_pic_width, i_pic_height;     /* picture dimensions */
+    int                 i_old_pic_y, i_old_pic_height;   /* old picture area */
+    int                 i_vout_width, i_vout_height;   /* display dimensions */
+    int                 i_area;                                /* area index */
+    int                 i_data_index;                     /* area data index */
+    int                 i_data_size;   /* area data size, in 256 bytes blocs */
+    u64 *               p_data;                   /* area data, for clearing */
+    byte_t *            p_data8;           /* area data, for clearing (slow) */
+
     /* Choose buffer and set display dimensions */
-    p_buffer =          &p_vout->p_buffer[ p_vout->i_buffer_index ];    
+    p_buffer =          &p_vout->p_buffer[ p_vout->i_buffer_index ];
     i_vout_width =      p_vout->i_width;
-    i_vout_height =     p_vout->i_height;    
+    i_vout_height =     p_vout->i_height;
 
     /*
-     * Computes new picture size 
+     * Computes new picture size
      */
     if( p_pic != NULL )
     {
         /* Try horizontal scaling first - width must be a mutiple of 16 */
-        i_pic_width = (( p_vout->b_scale || (p_pic->i_width > i_vout_width)) ? 
+        i_pic_width = (( p_vout->b_scale || (p_pic->i_width > i_vout_width)) ?
                        i_vout_width : p_pic->i_width) & ~0xf;
         switch( p_pic->i_aspect_ratio )
         {
         case AR_3_4_PICTURE:
             i_pic_height = i_pic_width * 3 / 4;
-            break;                
+            break;
         case AR_16_9_PICTURE:
             i_pic_height = i_pic_width * 9 / 16;
             break;
-        case AR_221_1_PICTURE:        
+        case AR_221_1_PICTURE:
             i_pic_height = i_pic_width * 100 / 221;
-            break;               
+            break;
         case AR_SQUARE_PICTURE:
         default:
-            i_pic_height = p_pic->i_height * i_pic_width / p_pic->i_width;            
+            i_pic_height = p_pic->i_height * i_pic_width / p_pic->i_width;
             break;
         }
 
-        /* If picture dimensions using horizontal scaling are too large, use 
+        /* If picture dimensions using horizontal scaling are too large, use
          * vertical scaling. Since width must be a multiple of 16, height is
          * adjusted again after. */
         if( i_pic_height > i_vout_height )
         {
-            i_pic_height = ( p_vout->b_scale || (p_pic->i_height > i_vout_height)) ? 
+            i_pic_height = ( p_vout->b_scale || (p_pic->i_height > i_vout_height)) ?
                 i_vout_height : p_pic->i_height;
             switch( p_pic->i_aspect_ratio )
             {
             case AR_3_4_PICTURE:
                 i_pic_width = (i_pic_height * 4 / 3) & ~0xf;
                 i_pic_height = i_pic_width * 3 / 4;
-                break;                
+                break;
             case AR_16_9_PICTURE:
                 i_pic_width = (i_pic_height * 16 / 9) & ~0xf;
                 i_pic_height = i_pic_width * 9 / 16;
                 break;
-            case AR_221_1_PICTURE:        
+            case AR_221_1_PICTURE:
                 i_pic_width = (i_pic_height * 221 / 100) & ~0xf;
                 i_pic_height = i_pic_width * 100 / 221;
-                break;               
+                break;
             case AR_SQUARE_PICTURE:
             default:
                 i_pic_width = (p_pic->i_width * i_pic_height / p_pic->i_height) & ~0xf;
                 i_pic_height = p_pic->i_height * i_pic_width / p_pic->i_width;
                 break;
-            }        
-        }        
+            }
+        }
 
         /* 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;                
-    }    
+        i_pic_y = (p_vout->i_height - i_pic_height) / 2;
+    }
     else
     {
         /* No picture: size is 0 */
@@ -1424,9 +1529,9 @@ 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) )
     {
         i_old_pic_y =            p_buffer->i_pic_y;
@@ -1434,7 +1539,7 @@ static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
         p_buffer->i_pic_x =      i_pic_x;
         p_buffer->i_pic_y =      i_pic_y;
         p_buffer->i_pic_width =  i_pic_width;
-        p_buffer->i_pic_height = i_pic_height;                        
+        p_buffer->i_pic_height = i_pic_height;
         SetBufferArea( p_vout, 0, i_old_pic_y, p_vout->i_width, i_old_pic_height );
     }
     else
@@ -1442,7 +1547,7 @@ static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
         p_buffer->i_pic_x =      i_pic_x;
         p_buffer->i_pic_y =      i_pic_y;
         p_buffer->i_pic_width =  i_pic_width;
-        p_buffer->i_pic_height = i_pic_height;    
+        p_buffer->i_pic_height = i_pic_height;
     }
 
     /*
@@ -1450,8 +1555,8 @@ 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 DEBUG_VIDEO
+        intf_DbgMsg("clearing picture %p area in buffer %d: %d-%d\n", 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;
@@ -1471,15 +1576,15 @@ static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
         for( i_data_index = (i_data_size % 256) / 16; i_data_index--; )
         {
             /* Clear remaining 16 bytes blocks */
-            *p_data++ = 0;  *p_data++ = 0;            
+            *p_data++ = 0;  *p_data++ = 0;
         }
-        p_data8 = (byte_t *)p_data;        
+        p_data8 = (byte_t *)p_data;
         for( i_data_index = i_data_size % 16; i_data_index--; )
         {
             /* Clear remaining bytes */
-            *p_data8++ = 0;            
+            *p_data8++ = 0;
         }
-    }    
+    }
 
     /*
      * Clear areas array
@@ -1487,104 +1592,104 @@ static void SetBufferPicture( vout_thread_t *p_vout, picture_t *p_pic )
     p_buffer->i_areas = 0;
 }
 
-/******************************************************************************
+/*****************************************************************************
  * 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.
- ******************************************************************************/
+ *****************************************************************************/
 static void RenderPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
 #ifdef DEBUG_VIDEO
-    char                psz_date[MSTRTIME_MAX_SIZE];          /* picture date */    
-    mtime_t             render_time;                /* picture rendering time */
+    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 */
-    
+    vout_buffer_t *     p_buffer;                        /* rendering buffer */
+    byte_t *            p_pic_data;                /* convertion destination */
+
     /* Get and set rendering informations */
-    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;
+    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
-    render_time = mdate();    
+    render_time = mdate();
 #endif
 
     /*
-     * Choose appropriate rendering function and render picture 
+     * 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_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_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;        
+        break;
     case YUV_422_PICTURE:
-        p_vout->yuv.p_Convert422( p_vout, p_pic_data, 
+        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_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;        
+        break;
     case YUV_444_PICTURE:
-        p_vout->yuv.p_Convert444( p_vout, p_pic_data, 
+        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_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;        
+        break;
 #ifdef DEBUG
-    default:        
+    default:
         intf_DbgMsg("error: unknown picture type %d\n", p_pic->i_type );
-        break;        
+        break;
 #endif
     }
 
 #ifdef DEBUG_VIDEO
     /* Print picture date and rendering time */
     intf_DbgMsg("picture %p rendered in buffer %d (%ld us), display date: %s\n", p_pic,
-                p_vout->i_buffer_index, (long) (mdate() - render_time), 
+                p_vout->i_buffer_index, (long) (mdate() - render_time),
                 mstrtime( psz_date, p_pic->date ));
 #endif
 }
 
-/******************************************************************************
+/*****************************************************************************
  * RenderPictureInfo: print additionnal informations on a picture
- ******************************************************************************
+ *****************************************************************************
  * This function will print informations such as fps and other picture
  * dependant informations.
- ******************************************************************************/
+ *****************************************************************************/
 static void RenderPictureInfo( vout_thread_t *p_vout, picture_t *p_pic )
 {
 #if defined(STATS) || defined(DEBUG)
-    char        psz_buffer[256];                             /* string buffer */
+    char        psz_buffer[256];                            /* string buffer */
 #endif
 
 #ifdef STATS
-    /* 
-     * Print FPS rate in upper right corner 
+    /*
+     * 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 ] ) );        
+                   p_vout->p_fps_sample[ p_vout->c_fps_samples % VOUT_FPS_SAMPLES ] ) );
         Print( p_vout, 0, 0, RIGHT_RALIGN, TOP_RALIGN, psz_buffer );
     }
 
-    /* 
-     * Print frames count and loop time in upper left corner 
+    /*
+     * Print frames count and loop time in upper left corner
      */
-    sprintf( psz_buffer, "%ld frames   rendering: %ld us", 
+    sprintf( psz_buffer, "%ld frames   rendering: %ld us",
              (long) p_vout->c_fps_samples, (long) p_vout->render_time );
     Print( p_vout, 0, 0, LEFT_RALIGN, TOP_RALIGN, psz_buffer );
 #endif
@@ -1607,69 +1712,69 @@ static void RenderPictureInfo( vout_thread_t *p_vout, picture_t *p_pic )
              p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_width,
              p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_height,
              p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_x,
-             p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_y );    
+             p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_y );
     Print( p_vout, 0, 0, RIGHT_RALIGN, BOTTOM_RALIGN, psz_buffer );
 #endif
 }
 
-/******************************************************************************
+/*****************************************************************************
  * RenderIdle: render idle picture
- ******************************************************************************
- * This function will print something on the screen. It will return 0 if 
+ *****************************************************************************
+ * 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 
+ * Unlike other rendering functions, this one calls the SetBufferPicture
  * function when needed.
- ******************************************************************************/
+ *****************************************************************************/
 static int RenderIdle( vout_thread_t *p_vout )
 {
-    int         i_x = 0, i_y = 0;                            /* text position */    
-    int         i_width, i_height;                               /* text size */    
-    mtime_t     current_date;                                 /* current date */
-    const char *psz_text = "no stream";                    /* text to display */
-    
-    
-    current_date = mdate();    
+    int         i_x = 0, i_y = 0;                           /* text position */
+    int         i_width, i_height;                              /* text size */
+    mtime_t     current_date;                                /* current date */
+    const char *psz_text = "waiting for stream ...";      /* text to display */
+
+
+    current_date = mdate();
     if( (current_date - p_vout->last_display_date) > VOUT_IDLE_DELAY &&
         (current_date - p_vout->last_idle_date) > VOUT_IDLE_DELAY )
     {
-        SetBufferPicture( p_vout, NULL );            
+        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, 
+        {
+            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 );        
+                        WIDE_TEXT | OUTLINED_TEXT, psz_text );
             SetBufferArea( p_vout, i_x, i_y, i_width, i_height );
-        }        
-        return( 1 );        
+        }
+        return( 1 );
     }
-    return( 0 );    
+    return( 0 );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * RenderInfo: render additionnal informations
- ******************************************************************************
+ *****************************************************************************
  * This function render informations which do not depend of the current picture
  * rendered.
- ******************************************************************************/
+ *****************************************************************************/
 static void RenderInfo( vout_thread_t *p_vout )
 {
 #ifdef DEBUG
-    char        psz_buffer[256];                             /* string buffer */
-    int         i_ready_pic = 0;                            /* ready pictures */
-    int         i_reserved_pic = 0;                      /* reserved pictures */
-    int         i_picture;                                   /* picture index */
+    char        psz_buffer[256];                            /* string buffer */
+    int         i_ready_pic = 0;                           /* ready pictures */
+    int         i_reserved_pic = 0;                     /* reserved pictures */
+    int         i_picture;                                  /* picture index */
 #endif
 
 #ifdef DEBUG
-    /* 
-     * Print thread state in lower left corner  
+    /*
+     * Print thread state in lower left corner
      */
     for( i_picture = 0; i_picture < VOUT_MAX_PICTURES; i_picture++ )
     {
@@ -1678,74 +1783,103 @@ static void RenderInfo( vout_thread_t *p_vout )
         case RESERVED_PICTURE:
         case RESERVED_DATED_PICTURE:
         case RESERVED_DISP_PICTURE:
-            i_reserved_pic++;            
-            break;            
+            i_reserved_pic++;
+            break;
         case READY_PICTURE:
-            i_ready_pic++;            
-            break;            
-        }        
+            i_ready_pic++;
+            break;
+        }
     }
-    sprintf( psz_buffer, "pic: %d/%d/%d", 
-             i_reserved_pic, i_ready_pic, VOUT_MAX_PICTURES );
-    Print( p_vout, 0, 0, LEFT_RALIGN, BOTTOM_RALIGN, psz_buffer );    
+    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
 }
 
-/*******************************************************************************
+/*****************************************************************************
  * RenderSubPicture: render a subpicture
- *******************************************************************************
+ *****************************************************************************
  * This function render 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 )
-    {
-    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;            
-        }
+    p_vout_font_t       p_font;                                 /* text font */
+    int                 i_width, i_height;          /* subpicture dimensions */
 
-        /* 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 ) )
+    while( p_subpic != NULL )
+    {
+        switch( p_subpic->i_type )
         {
-            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;        
-        
+        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;
+            }
+
+            /* 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 );
+                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_DbgMsg( "error: unknown subpicture %p type %d\n",
+                         p_subpic, p_subpic->i_type );
 #endif
+        }
+
+        p_subpic = p_subpic->p_next;
     }
 }
 
-/*******************************************************************************
+/*****************************************************************************
  * RenderInterface: render the interface
- *******************************************************************************
+ *****************************************************************************
  * This function render the interface, if any.
- *******************************************************************************/
+ *****************************************************************************/
 static void RenderInterface( vout_thread_t *p_vout )
 {
-    int         i_height, i_text_height;              /* total and text height */
-    int         i_width_1, i_width_2;                            /* text width */
-    int         i_byte;                                          /* byte index */
+    int         i_height, i_text_height;            /* total and text height */
+    int         i_width_1, i_width_2;                          /* text width */
+    int         i_byte;                                        /* byte index */
     const char *psz_text_1 = "[1-9] Channel   [i]nfo   [c]olor     [g/G]amma";
-    const char *psz_text_2 = "[+/-] Volume    [m]ute   [s]caling   [Q]uit";    
+    const char *psz_text_2 = "[+/-] Volume    [m]ute   [s]caling   [Q]uit";
 
     /* Get text size */
     vout_TextSize( p_vout->p_large_font, OUTLINED_TEXT, psz_text_1, &i_width_1, &i_height );
@@ -1757,13 +1891,13 @@ 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;
-    }    
+    }
 
     /* Render text, if not larger than screen */
     if( i_width_1 < p_vout->i_width )
-    {        
+    {
         vout_Print( p_vout->p_large_font, p_vout->p_buffer[ p_vout->i_buffer_index ].p_data +
                     (p_vout->i_height - i_height) * p_vout->i_bytes_per_line,
                     p_vout->i_bytes_per_pixel, p_vout->i_bytes_per_line,
@@ -1771,112 +1905,199 @@ static void RenderInterface( vout_thread_t *p_vout )
                     OUTLINED_TEXT, psz_text_1 );
     }
     if( i_width_2 < p_vout->i_width )
-    {        
+    {
         vout_Print( p_vout->p_large_font, p_vout->p_buffer[ p_vout->i_buffer_index ].p_data +
                     (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 );
-    }    
+    }
 
     /* Activate modified area */
     SetBufferArea( p_vout, 0, p_vout->i_height - i_height, p_vout->i_width, i_height );
 }
 
-/******************************************************************************
+/*****************************************************************************
+ * Synchronize: update synchro level depending of heap state
+ *****************************************************************************
+ * This function is called during the main vout loop.
+ *****************************************************************************/
+static void Synchronize( vout_thread_t *p_vout, s64 i_delay )
+{
+    int i_synchro_inc = 0;
+    /* XXX?? gore following */
+    static int i_panic_count = 0;
+    static int i_last_synchro_inc = 0;
+    static float r_synchro_level = VOUT_SYNCHRO_LEVEL_START;
+    static int i_truc = 10;
+
+    if( i_delay < 0 )
+    {
+        //fprintf( stderr, "PANIC %d\n", i_panic_count );
+        i_panic_count++;
+    }
+
+    i_truc *= 2;
+
+    if( p_vout->i_pictures > VOUT_SYNCHRO_HEAP_IDEAL_SIZE+1 )
+    {
+        i_truc = 40;
+        i_synchro_inc += p_vout->i_pictures - VOUT_SYNCHRO_HEAP_IDEAL_SIZE - 1;
+
+    }
+    else
+    {
+        if( p_vout->i_pictures < VOUT_SYNCHRO_HEAP_IDEAL_SIZE )
+        {
+            i_truc = 32;
+            i_synchro_inc += p_vout->i_pictures - VOUT_SYNCHRO_HEAP_IDEAL_SIZE;
+        }
+    }
+
+    if( i_truc > VOUT_SYNCHRO_LEVEL_MAX*2*2*2*2*2 ||
+        i_synchro_inc*i_last_synchro_inc < 0 )
+    {
+        i_truc = 32;
+    }
+
+    if( i_delay < 6000 )
+    {
+        i_truc = 16;
+        i_synchro_inc -= 2;
+    }
+    else if( i_delay < 70000 )
+    {
+        i_truc = 24+(24*i_delay)/70000;
+        if( i_truc < 16 )
+            i_truc = 16;
+        i_synchro_inc -= 1+(5*(70000-i_delay))/70000;
+    }
+    else if( i_delay > 100000 )
+    {
+        r_synchro_level += 1;
+        if( i_delay > 130000 )
+            r_synchro_level += 1;
+    }
+
+    r_synchro_level += (float)i_synchro_inc / i_truc;
+    p_vout->i_synchro_level = (int)(r_synchro_level+0.5);
+
+    if( r_synchro_level > VOUT_SYNCHRO_LEVEL_MAX )
+    {
+        r_synchro_level = VOUT_SYNCHRO_LEVEL_MAX;
+    }
+
+    //fprintf( stderr, "synchro level : %d, heap : %d (%d, %d) (%d, %f) - %Ld\n", p_vout->i_synchro_level,
+    //        p_vout->i_pictures, i_last_synchro_inc, i_synchro_inc, i_truc, r_synchro_level, i_delay );
+    i_last_synchro_inc = i_synchro_inc;
+}
+
+/*****************************************************************************
  * Manage: manage thread
- ******************************************************************************
+ *****************************************************************************
  * This function will handle changes in thread configuration.
- ******************************************************************************/
+ *****************************************************************************/
 static int Manage( vout_thread_t *p_vout )
 {
 #ifdef DEBUG_VIDEO
     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)\n", p_vout->i_changes,
+                    p_vout->i_changes & VOUT_NODISPLAY_CHANGE );
+    }
 #endif
 
     /* On gamma or grayscale change, rebuild tables */
-    if( p_vout->i_changes & (VOUT_GAMMA_CHANGE | VOUT_GRAYSCALE_CHANGE | 
+    if( p_vout->i_changes & (VOUT_GAMMA_CHANGE | VOUT_GRAYSCALE_CHANGE |
                              VOUT_YUV_CHANGE) )
     {
         if( vout_ResetYUV( p_vout ) )
         {
-            intf_ErrMsg("error: can't rebuild convertion tables\n");            
-            return( 1 );            
-        }        
+            intf_ErrMsg("error: can't rebuild convertion tables\n");
+            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 | 
+    p_vout->i_changes &= ~(VOUT_GAMMA_CHANGE | VOUT_GRAYSCALE_CHANGE |
+                           VOUT_YUV_CHANGE   | VOUT_INFO_CHANGE |
                            VOUT_INTF_CHANGE  | VOUT_SCALE_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" );        
-        return( 1 );        
+        /* Some changes were not acknowledged by p_vout->p_sys_manage or this
+         * function, it means they should not be authorized */
+        intf_ErrMsg( "error: unauthorized changes in the video output thread\n" );
+        return( 1 );
     }
-    
-    return( 0 );    
+
+    return( 0 );
 }
 
-/******************************************************************************
+/*****************************************************************************
  * Align: align a subpicture in the screen
- ******************************************************************************
+ *****************************************************************************
  * This function is used for rendering text or subpictures. It returns non 0
  * it the final aera is not fully included in display area. Return coordinates
  * are absolute.
- ******************************************************************************/
-static int Align( vout_thread_t *p_vout, int *pi_x, int *pi_y, 
+ *****************************************************************************/
+static int Align( vout_thread_t *p_vout, int *pi_x, int *pi_y,
                    int i_width, int i_height, int i_h_align, int i_v_align )
 {
     /* Align horizontally */
     switch( i_h_align )
     {
     case CENTER_ALIGN:
-        *pi_x -= i_width / 2;        
-        break;        
+        *pi_x -= i_width / 2;
+        break;
     case CENTER_RALIGN:
-        *pi_x += (p_vout->i_width - i_width) / 2;        
-        break;        
-    case RIGHT_ALIGN:   
-        *pi_x -= i_width;        
-        break;        
+        *pi_x += (p_vout->i_width - i_width) / 2;
+        break;
+    case RIGHT_ALIGN:
+        *pi_x -= i_width;
+        break;
     case RIGHT_RALIGN:
-        *pi_x += p_vout->i_width - i_width;        
-        break;        
+        *pi_x += p_vout->i_width - i_width;
+        break;
     }
 
     /* Align vertically */
     switch( i_v_align )
     {
     case CENTER_ALIGN:
-        *pi_y -= i_height / 2;        
+        *pi_y -= i_height / 2;
         break;
     case CENTER_RALIGN:
-        *pi_y += (p_vout->i_height - i_height) / 2;        
-        break;        
+        *pi_y += (p_vout->i_height - i_height) / 2;
+        break;
     case BOTTOM_ALIGN:
-        *pi_y -= i_height;        
-        break;        
+        *pi_y -= i_height;
+        break;
     case BOTTOM_RALIGN:
-        *pi_y += p_vout->i_height - i_height;        
-        break;        
+        *pi_y += p_vout->i_height - i_height;
+        break;
     case SUBTITLE_RALIGN:
-        *pi_y += (p_vout->i_height + p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_y + 
-                  p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_height - i_height) / 2;        
-        break;        
+        *pi_y += (p_vout->i_height + p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_y +
+                  p_vout->p_buffer[ p_vout->i_buffer_index ].i_pic_height - i_height) / 2;
+        break;
     }
 
     /* Return non 0 if clipping failed */
-    return( (*pi_x < 0) || (*pi_y < 0) || 
-            (*pi_x + i_width > p_vout->i_width) || (*pi_y + i_height > p_vout->i_height) );    
+    return( (*pi_x < 0) || (*pi_y < 0) ||
+            (*pi_x + i_width > p_vout->i_width) || (*pi_y + i_height > p_vout->i_height) );
+}
+
+/*****************************************************************************
+ * SetPalette: sets an 8 bpp palette
+ *****************************************************************************
+ * This function is just a prototype that does nothing. Architectures that
+ * support palette allocation should override it.
+ *****************************************************************************/
+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" );
 }