]> git.sesse.net Git - vlc/blobdiff - src/video_output/vout_pictures.c
Add vout_AspectRatio routine to get retrieve a vout's aspect ratio.
[vlc] / src / video_output / vout_pictures.c
index 30c1e2c6145315439721fa2e14d405d09679e9b9..285b094947cc3d5f059729a49192581a1c62f70b 100644 (file)
@@ -1,8 +1,8 @@
 /*****************************************************************************
  * vout_pictures.c : picture management functions
  *****************************************************************************
- * Copyright (C) 2000 VideoLAN
- * $Id: vout_pictures.c,v 1.3 2001/12/13 20:47:46 sam Exp $
+ * Copyright (C) 2000-2004 VideoLAN
+ * $Id: vout_pictures.c,v 1.44 2004/01/06 12:02:06 zorglub Exp $
  *
  * Authors: Vincent Seguin <seguin@via.ecp.fr>
  *          Samuel Hocevar <sam@zoy.org>
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include "defs.h"
-
-#include <errno.h>                                                 /* ENOMEM */
 #include <stdlib.h>                                                /* free() */
 #include <stdio.h>                                              /* sprintf() */
 #include <string.h>                                            /* strerror() */
 
-#include "common.h"
-#include "intf_msg.h"
-#include "threads.h"
-#include "mtime.h"
+#include <vlc/vlc.h>
 
-#include "video.h"
+#include "vlc_video.h"
 #include "video_output.h"
 
+#include "vout_pictures.h"
+
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
-static void     NewPicture        ( vout_thread_t *, picture_t * );
+static void CopyPicture( vout_thread_t *, picture_t *, picture_t * );
 
 /*****************************************************************************
  * 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
+ * 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
  * called.
  *****************************************************************************/
-void vout_DisplayPicture( vout_thread_t *p_vout, picture_t *p_picture )
+void vout_DisplayPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
     vlc_mutex_lock( &p_vout->picture_lock );
-    switch( p_picture->i_status )
+    switch( p_pic->i_status )
     {
     case RESERVED_PICTURE:
-        p_picture->i_status = RESERVED_DISP_PICTURE;
+        p_pic->i_status = RESERVED_DISP_PICTURE;
         break;
     case RESERVED_DATED_PICTURE:
-        p_picture->i_status = READY_PICTURE;
+        p_pic->i_status = READY_PICTURE;
         break;
-#ifdef DEBUG
     default:
-        intf_ErrMsg("error: picture %p has invalid status %d", p_picture, p_picture->i_status );
+        msg_Err( p_vout, "picture to display %p has invalid status %d",
+                         p_pic, p_pic->i_status );
         break;
-#endif
     }
 
-#ifdef TRACE_VOUT
-    intf_DbgMsg("picture %p", p_picture);
-#endif
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
@@ -84,32 +76,24 @@ void vout_DisplayPicture( vout_thread_t *p_vout, picture_t *p_picture )
  * been called.
  *****************************************************************************/
 void vout_DatePicture( vout_thread_t *p_vout,
-                       picture_t *p_picture, mtime_t date )
+                       picture_t *p_pic, mtime_t date )
 {
-#ifdef TRACE_VOUT
-    char        psz_date[ MSTRTIME_MAX_SIZE ];                       /* date */
-#endif
-
     vlc_mutex_lock( &p_vout->picture_lock );
-    p_picture->date = date;
-    switch( p_picture->i_status )
+    p_pic->date = date;
+    switch( p_pic->i_status )
     {
     case RESERVED_PICTURE:
-        p_picture->i_status = RESERVED_DATED_PICTURE;
+        p_pic->i_status = RESERVED_DATED_PICTURE;
         break;
     case RESERVED_DISP_PICTURE:
-        p_picture->i_status = READY_PICTURE;
+        p_pic->i_status = READY_PICTURE;
         break;
-#ifdef DEBUG
     default:
-        intf_ErrMsg("error: picture %p has invalid status %d", p_picture, p_picture->i_status );
+        msg_Err( p_vout, "picture to date %p has invalid status %d",
+                         p_pic, p_pic->i_status );
         break;
-#endif
     }
 
-#ifdef TRACE_VOUT
-    intf_DbgMsg("picture %p, display date: %s", p_picture, mstrtime( psz_date, p_picture->date) );
-#endif
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
@@ -118,52 +102,57 @@ void vout_DatePicture( vout_thread_t *p_vout,
  *****************************************************************************
  * This function creates 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.
+ * 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 )
+picture_t *vout_CreatePicture( vout_thread_t *p_vout,
+                               vlc_bool_t b_progressive,
+                               vlc_bool_t b_top_field_first,
+                               unsigned int i_nb_fields )
 {
-    int         i_picture;                                  /* picture index */
-    picture_t * p_picture;
-    picture_t * p_free_picture = NULL;                 /* first free picture */
+    int         i_pic;                                      /* picture index */
+    picture_t * p_pic;
+    picture_t * p_freepic = NULL;                      /* first free picture */
 
     /* Get lock */
     vlc_mutex_lock( &p_vout->picture_lock );
 
     /*
-     * Look for an empty place. XXX: we start at 1 because the first
-     * directbuffer is reserved for memcpy()ed pictures.
+     * Look for an empty place in the picture heap.
      */
-    for( i_picture = 0;
-         i_picture < I_RENDERPICTURES && p_free_picture == NULL;
-         i_picture++ )
+    for( i_pic = 0; i_pic < I_RENDERPICTURES; i_pic++ )
     {
-        p_picture = PP_RENDERPICTURE[ i_picture ];
+        p_pic = PP_RENDERPICTURE[(p_vout->render.i_last_used_pic + i_pic + 1)
+                                 % I_RENDERPICTURES];
 
-        /* If the picture we found is a memory buffer, and we might have
-         * enough room later for a direct buffer, skip it. If no other
-         * pictures are found, the video decoder will try again later. */
-        if( p_vout->b_direct && ( p_vout->output.i_pictures > 3 )
-             && ( p_picture->i_type != DIRECT_PICTURE ) )
-        {
-            break;
-        }
-
-        switch( p_picture->i_status )
+        switch( p_pic->i_status )
         {
             case DESTROYED_PICTURE:
                 /* 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_picture->i_status = RESERVED_PICTURE;
-                p_vout->i_heap_size++;
+                p_pic->i_status   = RESERVED_PICTURE;
+                p_pic->i_refcount = 0;
+                p_pic->b_force    = 0;
+
+                p_pic->b_progressive        = b_progressive;
+                p_pic->i_nb_fields          = i_nb_fields;
+                p_pic->b_top_field_first    = b_top_field_first;
 
+                p_vout->i_heap_size++;
+                p_vout->render.i_last_used_pic =
+                    ( p_vout->render.i_last_used_pic + i_pic + 1 )
+                    % I_RENDERPICTURES;
                 vlc_mutex_unlock( &p_vout->picture_lock );
-                return( p_picture );
+                return( p_pic );
 
             case FREE_PICTURE:
                 /* Picture is empty and ready for allocation */
-                p_free_picture = p_picture;
+                p_vout->render.i_last_used_pic =
+                    ( p_vout->render.i_last_used_pic + i_pic + 1 )
+                    % I_RENDERPICTURES;
+                p_freepic = p_pic;
                 break;
 
             default:
@@ -174,33 +163,41 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout )
     /*
      * Prepare picture
      */
-    if( p_free_picture != NULL )
+    if( p_freepic != NULL )
     {
-        NewPicture( p_vout, p_free_picture );
+        vout_AllocatePicture( p_vout, p_freepic, p_vout->render.i_chroma,
+                              p_vout->render.i_width, p_vout->render.i_height,
+                              p_vout->render.i_aspect );
 
-        if( p_free_picture->i_planes )
+        if( p_freepic->i_planes )
         {
             /* Copy picture information, set some default values */
-            p_free_picture->i_status = RESERVED_PICTURE;
-            p_free_picture->i_matrix_coefficients = 1;
-            p_free_picture->i_refcount = 0;
+            p_freepic->i_status   = RESERVED_PICTURE;
+            p_freepic->i_type     = MEMORY_PICTURE;
+
+            p_freepic->i_refcount = 0;
+            p_freepic->b_force = 0;
+
+            p_freepic->b_progressive        = b_progressive;
+            p_freepic->i_nb_fields          = i_nb_fields;
+            p_freepic->b_top_field_first    = b_top_field_first;
+
+            p_freepic->i_matrix_coefficients = 1;
+
             p_vout->i_heap_size++;
         }
         else
         {
             /* Memory allocation failed : set picture as empty */
-            p_free_picture->i_status = FREE_PICTURE;
-            p_free_picture = NULL;
+            p_freepic->i_status = FREE_PICTURE;
+            p_freepic = NULL;
 
-            intf_ErrMsg( "vout error: picture allocation failed" );
+            msg_Err( p_vout, "picture allocation failed" );
         }
 
         vlc_mutex_unlock( &p_vout->picture_lock );
 
-        /* Initialize mutex */
-        vlc_mutex_init( &(p_free_picture->lock_deccount) );
-
-        return( p_free_picture );
+        return( p_freepic );
     }
 
     /* No free or destroyed picture could be found, but the decoder
@@ -217,27 +214,24 @@ picture_t *vout_CreatePicture( vout_thread_t *p_vout )
  * 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 !
  *****************************************************************************/
-void vout_DestroyPicture( vout_thread_t *p_vout, picture_t *p_picture )
+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_picture->i_status != RESERVED_PICTURE) &&
-        (p_picture->i_status != RESERVED_DATED_PICTURE) &&
-        (p_picture->i_status != RESERVED_DISP_PICTURE) )
+    if( (p_pic->i_status != RESERVED_PICTURE) &&
+        (p_pic->i_status != RESERVED_DATED_PICTURE) &&
+        (p_pic->i_status != RESERVED_DISP_PICTURE) )
     {
-        intf_ErrMsg( "error: picture %p has invalid status %d",
-                     p_picture, p_picture->i_status );
+        msg_Err( p_vout, "picture to destroy %p has invalid status %d",
+                         p_pic, p_pic->i_status );
     }
 #endif
 
-    p_picture->i_status = DESTROYED_PICTURE;
+    p_pic->i_status = DESTROYED_PICTURE;
     p_vout->i_heap_size--;
 
-    /* destroy the lock that had been initialized in CreatePicture */
-    vlc_mutex_destroy( &(p_picture->lock_deccount) );
-
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
@@ -247,47 +241,37 @@ void vout_DestroyPicture( vout_thread_t *p_vout, picture_t *p_picture )
  * This function increments 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_picture )
+void vout_LinkPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
     vlc_mutex_lock( &p_vout->picture_lock );
-    p_picture->i_refcount++;
-
-#ifdef TRACE_VOUT
-    intf_DbgMsg( "picture %p refcount=%d", p_picture, p_picture->i_refcount );
-#endif
-
+    p_pic->i_refcount++;
     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.
+ * This function decrement the reference counter of a picture in the video heap
  *****************************************************************************/
-void vout_UnlinkPicture( vout_thread_t *p_vout, picture_t *p_picture )
+void vout_UnlinkPicture( vout_thread_t *p_vout, picture_t *p_pic )
 {
     vlc_mutex_lock( &p_vout->picture_lock );
-    p_picture->i_refcount--;
+    p_pic->i_refcount--;
 
-#ifdef TRACE_VOUT
-    if( p_picture->i_refcount < 0 )
+    if( p_pic->i_refcount < 0 )
     {
-        intf_DbgMsg( "error: refcount < 0" );
-        p_picture->i_refcount = 0;
+        msg_Err( p_vout, "picture %p refcount is %i", 
+                 p_pic, p_pic->i_refcount );
+        p_pic->i_refcount = 0;
     }
-#endif
 
-    if( ( p_picture->i_refcount == 0 ) &&
-        ( p_picture->i_status == DISPLAYED_PICTURE ) )
+    if( ( p_pic->i_refcount == 0 ) &&
+        ( p_pic->i_status == DISPLAYED_PICTURE ) )
     {
-        p_picture->i_status = DESTROYED_PICTURE;
+        p_pic->i_status = DESTROYED_PICTURE;
         p_vout->i_heap_size--;
     }
 
-#ifdef TRACE_VOUT
-    intf_DbgMsg( "picture %p refcount=%d", p_picture, p_picture->i_refcount );
-#endif
-
     vlc_mutex_unlock( &p_vout->picture_lock );
 }
 
@@ -298,21 +282,19 @@ void vout_UnlinkPicture( vout_thread_t *p_vout, picture_t *p_picture )
  * before rendering, does the subpicture magic, and tells the video output
  * thread which direct buffer needs to be displayed.
  *****************************************************************************/
-picture_t * vout_RenderPicture( vout_thread_t *p_vout, picture_t *p_picture,
+picture_t * vout_RenderPicture( vout_thread_t *p_vout, picture_t *p_pic,
                                                        subpicture_t *p_subpic )
 {
-    int i_index;
-
-    if( p_picture == NULL )
+    if( p_pic == NULL )
     {
         /* XXX: subtitles */
 
         return NULL;
     }
 
-    if( p_picture->i_type == DIRECT_PICTURE )
+    if( p_pic->i_type == DIRECT_PICTURE )
     {
-        if( p_picture->i_refcount )
+        if( !p_vout->render.b_allow_modify_pics || p_pic->i_refcount )
         {
             /* Picture is in a direct buffer and is still in use,
              * we need to copy it to another direct buffer before
@@ -322,13 +304,7 @@ picture_t * vout_RenderPicture( vout_thread_t *p_vout, picture_t *p_picture,
                 /* We have subtitles. First copy the picture to
                  * the spare direct buffer, then render the
                  * subtitles. */
-                for( i_index = 0 ; i_index < p_picture->i_planes ; i_index++ )
-                {
-                    p_main->fast_memcpy(
-                        PP_OUTPUTPICTURE[0]->planes[ i_index ].p_data,
-                        p_picture->planes[ i_index ].p_data,
-                        p_picture->planes[ i_index ].i_bytes );
-                }
+                CopyPicture( p_vout, p_pic, PP_OUTPUTPICTURE[0] );
 
                 vout_RenderSubPictures( p_vout, PP_OUTPUTPICTURE[0], p_subpic );
 
@@ -338,15 +314,15 @@ picture_t * vout_RenderPicture( vout_thread_t *p_vout, picture_t *p_picture,
             /* No subtitles, picture is in a directbuffer so
              * we can display it directly even if it is still
              * in use. */
-            return p_picture;
+            return p_pic;
         }
 
         /* Picture is in a direct buffer but isn't used by the
          * decoder. We can safely render subtitles on it and
          * display it. */
-        vout_RenderSubPictures( p_vout, p_picture, p_subpic );
+        vout_RenderSubPictures( p_vout, p_pic, p_subpic );
 
-        return p_picture;
+        return p_pic;
     }
 
     /* Not a direct buffer. We either need to copy it to a direct buffer,
@@ -356,28 +332,43 @@ picture_t * vout_RenderPicture( vout_thread_t *p_vout, picture_t *p_picture,
         /* Picture is not in a direct buffer, but is exactly the
          * same size as the direct buffers. A memcpy() is enough,
          * then render the subtitles. */
-        for( i_index = 0; i_index < p_picture->i_planes; i_index++ )
-        {
-            p_main->fast_memcpy( PP_OUTPUTPICTURE[0]->planes[ i_index ].p_data,
-                                 p_picture->planes[ i_index ].p_data,
-                                 p_picture->planes[ i_index ].i_bytes );
-        }
+
+        if( PP_OUTPUTPICTURE[0]->pf_lock )
+            if( PP_OUTPUTPICTURE[0]->pf_lock( p_vout, PP_OUTPUTPICTURE[0] ) )
+            {
+                if( PP_OUTPUTPICTURE[0]->pf_unlock )
+                PP_OUTPUTPICTURE[0]->pf_unlock( p_vout, PP_OUTPUTPICTURE[0] );
+
+                return NULL;
+            }
+
+        CopyPicture( p_vout, p_pic, PP_OUTPUTPICTURE[0] );
 
         vout_RenderSubPictures( p_vout, PP_OUTPUTPICTURE[0], p_subpic );
 
+        if( PP_OUTPUTPICTURE[0]->pf_unlock )
+            PP_OUTPUTPICTURE[0]->pf_unlock( p_vout, PP_OUTPUTPICTURE[0] );
+
         return PP_OUTPUTPICTURE[0];
     }
 
     /* Picture is not in a direct buffer, and needs to be converted to
      * another size/chroma. Then the subtitles need to be rendered as
-     * well. */
-
-    /* This usually means software YUV, or hardware YUV with a
+     * well. This usually means software YUV, or hardware YUV with a
      * different chroma. */
 
-    /* XXX: render to direct buffer */
+    if( p_vout->p_picture[0].pf_lock )
+        if( p_vout->p_picture[0].pf_lock( p_vout, &p_vout->p_picture[0] ) )
+            return NULL;
+
+    /* Convert image to the first direct buffer */
+    p_vout->chroma.pf_convert( p_vout, p_pic, &p_vout->p_picture[0] );
+
+    /* Render subpictures on the first direct buffer */
+    vout_RenderSubPictures( p_vout, &p_vout->p_picture[0], p_subpic );
 
-    vout_RenderSubPictures( p_vout, p_picture, p_subpic );
+    if( p_vout->p_picture[0].pf_unlock )
+        p_vout->p_picture[0].pf_unlock( p_vout, &p_vout->p_picture[0] );
 
     return &p_vout->p_picture[0];
 }
@@ -388,10 +379,17 @@ picture_t * vout_RenderPicture( vout_thread_t *p_vout, picture_t *p_picture,
  * This function will be accessed by plugins. It calculates the relative
  * position of the output window and the image window.
  *****************************************************************************/
-void vout_PlacePicture( vout_thread_t *p_vout, int i_width, int i_height,
-                        int *pi_x, int *pi_y, int *pi_width, int *pi_height )
+void vout_PlacePicture( vout_thread_t *p_vout,
+                        unsigned int i_width, unsigned int i_height,
+                        unsigned int *pi_x, unsigned int *pi_y,
+                        unsigned int *pi_width, unsigned int *pi_height )
 {
-    int i_ratio;
+    if( (i_width <= 0) || (i_height <=0) )
+    {
+        *pi_width = *pi_height = *pi_x = *pi_y = 0;
+
+        return;
+    }
 
     if( p_vout->b_scale )
     {
@@ -400,132 +398,444 @@ void vout_PlacePicture( vout_thread_t *p_vout, int i_width, int i_height,
     }
     else
     {
-        *pi_width = MIN( i_width, p_vout->render.i_width );
-        *pi_height = MIN( i_height, p_vout->render.i_height );
+        *pi_width = __MIN( i_width, p_vout->render.i_width );
+        *pi_height = __MIN( i_height, p_vout->render.i_height );
     }
 
-    switch( p_vout->render.i_aspect )
+    if( VOUT_ASPECT_FACTOR * *pi_width / *pi_height < p_vout->render.i_aspect )
     {
-        case AR_3_4_PICTURE:
-            i_ratio = 900 * 4 / 3;
-            break;
-
-        case AR_16_9_PICTURE:
-            i_ratio = 900 * 16 / 9;
-            break;
-
-        case AR_221_1_PICTURE:
-            i_ratio = 900 * 221 / 100;
-            break;
-
-        case AR_SQUARE_PICTURE:
-        default:
-            i_ratio = 900 * p_vout->render.i_width
-                          / p_vout->render.i_height;
-            break;
-    }
-
-    if( 900 * *pi_width / *pi_height < i_ratio )
-    {
-        *pi_width = *pi_height * i_ratio / 900;
+        *pi_width = *pi_height * p_vout->render.i_aspect / VOUT_ASPECT_FACTOR;
     }
     else
     {
-        *pi_height = *pi_width * 900 / i_ratio;
+        *pi_height = *pi_width * VOUT_ASPECT_FACTOR / p_vout->render.i_aspect;
     }
 
     if( *pi_width > i_width )
     {
         *pi_width = i_width;
-        *pi_height = 900 * *pi_width / i_ratio;
+        *pi_height = VOUT_ASPECT_FACTOR * *pi_width / p_vout->render.i_aspect;
     }
 
     if( *pi_height > i_height )
     {
         *pi_height = i_height;
-        *pi_width = *pi_height * i_ratio / 900;
+        *pi_width = *pi_height * p_vout->render.i_aspect / VOUT_ASPECT_FACTOR;
     }
 
-    *pi_x = ( i_width - *pi_width ) / 2;
-    *pi_y = ( i_height - *pi_height ) / 2;
-}
+    switch( p_vout->i_alignment & VOUT_ALIGN_HMASK )
+    {
+    case VOUT_ALIGN_LEFT:
+        *pi_x = 0;
+        break;
+    case VOUT_ALIGN_RIGHT:
+        *pi_x = i_width - *pi_width;
+        break;
+    default:
+        *pi_x = ( i_width - *pi_width ) / 2;
+    }
 
-/* Following functions are local */
+    switch( p_vout->i_alignment & VOUT_ALIGN_VMASK )
+    {
+    case VOUT_ALIGN_TOP:
+        *pi_y = 0;
+        break;
+    case VOUT_ALIGN_BOTTOM:
+        *pi_y = i_height - *pi_height;
+        break;
+    default:
+        *pi_y = ( i_height - *pi_height ) / 2;
+    }
+}
 
 /*****************************************************************************
- * NewPicture: allocate a new picture in the heap.
+ * vout_AllocatePicture: allocate a new picture in the heap.
  *****************************************************************************
  * This function allocates a fake direct buffer in memory, which can be
  * used exactly like a video buffer. The video output thread then manages
  * how it gets displayed.
  *****************************************************************************/
-static void NewPicture( vout_thread_t *p_vout, picture_t *p_picture )
+void vout_AllocatePicture( vout_thread_t *p_vout, picture_t *p_pic,
+                           vlc_fourcc_t i_chroma,
+                           int i_width, int i_height, int i_aspect )
+{
+    int i_bytes, i_index;
+
+    vout_InitPicture( VLC_OBJECT(p_vout), p_pic, i_chroma,
+                      i_width, i_height, i_aspect );
+
+    /* Calculate how big the new image should be */
+    i_bytes = p_pic->format.i_bits_per_pixel *
+        p_pic->format.i_width * p_pic->format.i_height / 8;
+
+    p_pic->p_data = vlc_memalign( &p_pic->p_data_orig, 16, i_bytes );
+
+    if( p_pic->p_data == NULL )
+    {
+        p_pic->i_planes = 0;
+        return;
+    }
+
+    /* Fill the p_pixels field for each plane */
+    p_pic->p[ 0 ].p_pixels = p_pic->p_data;
+
+    for( i_index = 1; i_index < p_pic->i_planes; i_index++ )
+    {
+        p_pic->p[i_index].p_pixels = p_pic->p[i_index-1].p_pixels
+                                          + p_pic->p[i_index-1].i_lines
+                                             * p_pic->p[i_index-1].i_pitch;
+    }
+}
+
+/*****************************************************************************
+ * vout_InitFormat: initialise the video format fields given chroma/size.
+ *****************************************************************************
+ * This function initializes all the video_frame_format_t fields given the
+ * static properties of a picture (chroma and size).
+ *****************************************************************************/
+void vout_InitFormat( video_frame_format_t *p_format, vlc_fourcc_t i_chroma,
+                      int i_width, int i_height, int i_aspect )
 {
-    int i_data_size = 0;
+    p_format->i_chroma   = i_chroma;
+    p_format->i_width    = p_format->i_visible_width  = i_width;
+    p_format->i_height   = p_format->i_visible_height = i_height;
+    p_format->i_x_offset = p_format->i_y_offset = 0;
+    p_format->i_aspect   = i_aspect;
+
+#if 0
+    /* Assume we have square pixels */
+    if( i_width && i_height )
+        p_format->i_aspect = i_width * VOUT_ASPECT_FACTOR / i_height;
+    else
+        p_format->i_aspect = 0;
+#endif
 
-    p_picture->i_size = p_vout->render.i_width
-                         * p_vout->render.i_height;
+    switch( i_chroma )
+    {
+        case FOURCC_I444:
+            p_format->i_bits_per_pixel = 24;
+            break;
+        case FOURCC_I422:
+        case FOURCC_YUY2:
+            p_format->i_bits_per_pixel = 16;
+            p_format->i_bits_per_pixel = 16;
+            break;
+        case FOURCC_I411:
+        case FOURCC_YV12:
+        case FOURCC_I420:
+        case FOURCC_IYUV:
+            p_format->i_bits_per_pixel = 12;
+            break;
+        case FOURCC_I410:
+            p_format->i_bits_per_pixel = 9;
+            break;
+        case FOURCC_Y211:
+            p_format->i_bits_per_pixel = 8;
+            break;
+        case FOURCC_RV32:
+            p_format->i_bits_per_pixel = 32;
+            break;
+        case FOURCC_RV24:
+            /* FIXME: Should be 24 here but x11 and our chroma conversion
+             * routines assume 32. */
+#ifdef WIN32
+            p_format->i_bits_per_pixel = 24;
+#else
+            p_format->i_bits_per_pixel = 32;
+#endif
+            break;
+        case FOURCC_RV15:
+        case FOURCC_RV16:
+            p_format->i_bits_per_pixel = 16;
+            break;
+        case FOURCC_RGB2:
+            p_format->i_bits_per_pixel = 8;
+            break;
+        default:
+            p_format->i_bits_per_pixel = 0;
+            break;
+    }
+}
+
+/*****************************************************************************
+ * vout_InitPicture: initialise the picture_t fields given chroma/size.
+ *****************************************************************************
+ * This function initializes most of the picture_t fields given a chroma and
+ * size. It makes the assumption that stride == width.
+ *****************************************************************************/
+void vout_InitPicture( vlc_object_t *p_this, picture_t *p_pic,
+                       vlc_fourcc_t i_chroma,
+                       int i_width, int i_height, int i_aspect )
+{
+    int i_index;
+
+    /* Store default values */
+    for( i_index = 0; i_index < VOUT_MAX_PLANES; i_index++ )
+    {
+        p_pic->p[i_index].p_pixels = NULL;
+        p_pic->p[i_index].i_pixel_pitch = 1;
+    }
+
+    vout_InitFormat( &p_pic->format, i_chroma, i_width, i_height, i_aspect );
 
     /* Calculate coordinates */
-    switch( p_vout->render.i_chroma )
+    switch( i_chroma )
     {
-        case YUV_420_PICTURE:        /* YUV 420: 1,1/4,1/4 samples per pixel */
-            p_picture->i_chroma_size = p_picture->i_size / 4;
-            p_picture->i_chroma_width = p_vout->render.i_width / 2;
+        case FOURCC_I411:
+            p_pic->p[ Y_PLANE ].i_lines = i_height;
+            p_pic->p[ Y_PLANE ].i_pitch = i_width;
+            p_pic->p[ Y_PLANE ].i_visible_pitch = p_pic->p[ Y_PLANE ].i_pitch;
+            p_pic->p[ U_PLANE ].i_lines = i_height;
+            p_pic->p[ U_PLANE ].i_pitch = i_width / 4;
+            p_pic->p[ U_PLANE ].i_visible_pitch = p_pic->p[ U_PLANE ].i_pitch;
+            p_pic->p[ V_PLANE ].i_lines = i_height;
+            p_pic->p[ V_PLANE ].i_pitch = i_width / 4;
+            p_pic->p[ V_PLANE ].i_visible_pitch = p_pic->p[ V_PLANE ].i_pitch;
+            p_pic->i_planes = 3;
             break;
 
-        case YUV_422_PICTURE:        /* YUV 422: 1,1/2,1/2 samples per pixel */
-            p_picture->i_chroma_size = p_picture->i_size / 2;
-            p_picture->i_chroma_width = p_vout->render.i_width / 2;
+        case FOURCC_I410:
+            p_pic->p[ Y_PLANE ].i_lines = i_height;
+            p_pic->p[ Y_PLANE ].i_pitch = i_width;
+            p_pic->p[ Y_PLANE ].i_visible_pitch = p_pic->p[ Y_PLANE ].i_pitch;
+            p_pic->p[ U_PLANE ].i_lines = i_height / 4;
+            p_pic->p[ U_PLANE ].i_pitch = i_width / 4;
+            p_pic->p[ U_PLANE ].i_visible_pitch = p_pic->p[ U_PLANE ].i_pitch;
+            p_pic->p[ V_PLANE ].i_lines = i_height / 4;
+            p_pic->p[ V_PLANE ].i_pitch = i_width / 4;
+            p_pic->p[ V_PLANE ].i_visible_pitch = p_pic->p[ V_PLANE ].i_pitch;
+            p_pic->i_planes = 3;
             break;
 
-        case YUV_444_PICTURE:            /* YUV 444: 1,1,1 samples per pixel */
-            p_picture->i_chroma_size = p_picture->i_size;
-            p_picture->i_chroma_width = p_vout->render.i_width;
+        case FOURCC_YV12:
+        case FOURCC_I420:
+        case FOURCC_IYUV:
+            p_pic->p[ Y_PLANE ].i_lines = i_height;
+            p_pic->p[ Y_PLANE ].i_pitch = i_width;
+            p_pic->p[ Y_PLANE ].i_visible_pitch = p_pic->p[ Y_PLANE ].i_pitch;
+            p_pic->p[ U_PLANE ].i_lines = i_height / 2;
+            p_pic->p[ U_PLANE ].i_pitch = i_width / 2;
+            p_pic->p[ U_PLANE ].i_visible_pitch = p_pic->p[ U_PLANE ].i_pitch;
+            p_pic->p[ V_PLANE ].i_lines = i_height / 2;
+            p_pic->p[ V_PLANE ].i_pitch = i_width / 2;
+            p_pic->p[ V_PLANE ].i_visible_pitch = p_pic->p[ V_PLANE ].i_pitch;
+            p_pic->i_planes = 3;
+            break;
+
+        case FOURCC_I422:
+            p_pic->p[ Y_PLANE ].i_lines = i_height;
+            p_pic->p[ Y_PLANE ].i_pitch = i_width;
+            p_pic->p[ Y_PLANE ].i_visible_pitch = p_pic->p[ Y_PLANE ].i_pitch;
+            p_pic->p[ U_PLANE ].i_lines = i_height;
+            p_pic->p[ U_PLANE ].i_pitch = i_width / 2;
+            p_pic->p[ U_PLANE ].i_visible_pitch = p_pic->p[ U_PLANE ].i_pitch;
+            p_pic->p[ V_PLANE ].i_lines = i_height;
+            p_pic->p[ V_PLANE ].i_pitch = i_width / 2;
+            p_pic->p[ V_PLANE ].i_visible_pitch = p_pic->p[ V_PLANE ].i_pitch;
+            p_pic->i_planes = 3;
+            break;
+
+        case FOURCC_I444:
+            p_pic->p[ Y_PLANE ].i_lines = i_height;
+            p_pic->p[ Y_PLANE ].i_pitch = i_width;
+            p_pic->p[ Y_PLANE ].i_visible_pitch = p_pic->p[ Y_PLANE ].i_pitch;
+            p_pic->p[ U_PLANE ].i_lines = i_height;
+            p_pic->p[ U_PLANE ].i_pitch = i_width;
+            p_pic->p[ U_PLANE ].i_visible_pitch = p_pic->p[ U_PLANE ].i_pitch;
+            p_pic->p[ V_PLANE ].i_lines = i_height;
+            p_pic->p[ V_PLANE ].i_pitch = i_width;
+            p_pic->p[ V_PLANE ].i_visible_pitch = p_pic->p[ V_PLANE ].i_pitch;
+            p_pic->i_planes = 3;
+            break;
+
+        case FOURCC_Y211:
+            p_pic->p->i_lines = i_height;
+            p_pic->p->i_pitch = i_width;
+            p_pic->p->i_visible_pitch = p_pic->p->i_pitch;
+            p_pic->p->i_pixel_pitch = 4;
+            p_pic->i_planes = 1;
+            break;
+
+        case FOURCC_YUY2:
+            p_pic->p->i_lines = i_height;
+            p_pic->p->i_pitch = i_width * 2;
+            p_pic->p->i_visible_pitch = p_pic->p->i_pitch;
+            p_pic->p->i_pixel_pitch = 4;
+            p_pic->i_planes = 1;
+            break;
+
+        case FOURCC_RGB2:
+            p_pic->p->i_lines = i_height;
+            p_pic->p->i_pitch = i_width;
+            p_pic->p->i_visible_pitch = p_pic->p->i_pitch;
+            p_pic->p->i_pixel_pitch = 1;
+            p_pic->i_planes = 1;
+            break;
+
+        case FOURCC_RV15:
+            p_pic->p->i_lines = i_height;
+            p_pic->p->i_pitch = i_width * 2;
+            p_pic->p->i_visible_pitch = p_pic->p->i_pitch;
+            p_pic->p->i_pixel_pitch = 2;
+/* FIXME: p_heap isn't always reachable
+            p_pic->p_heap->i_rmask = 0x001f;
+            p_pic->p_heap->i_gmask = 0x03e0;
+            p_pic->p_heap->i_bmask = 0x7c00; */
+            p_pic->i_planes = 1;
+            break;
+
+        case FOURCC_RV16:
+            p_pic->p->i_lines = i_height;
+            p_pic->p->i_pitch = i_width * 2;
+            p_pic->p->i_visible_pitch = p_pic->p->i_pitch;
+            p_pic->p->i_pixel_pitch = 2;
+/* FIXME: p_heap isn't always reachable
+            p_pic->p_heap->i_rmask = 0x001f;
+            p_pic->p_heap->i_gmask = 0x07e0;
+            p_pic->p_heap->i_bmask = 0xf800; */
+            p_pic->i_planes = 1;
+            break;
+
+        case FOURCC_RV24:
+            p_pic->p->i_lines = i_height;
+
+            /* FIXME: Should be 3 here but x11 and our chroma conversion
+             * routines assume 4. */
+#ifdef WIN32
+            p_pic->p->i_pitch = i_width * 3;
+            p_pic->p->i_pixel_pitch = 3;
+#else
+            p_pic->p->i_pitch = i_width * 4;
+            p_pic->p->i_pixel_pitch = 4;
+#endif
+            p_pic->p->i_visible_pitch = p_pic->p->i_pitch;
+/* FIXME: p_heap isn't always reachable
+            p_pic->p_heap->i_rmask = 0xff0000;
+            p_pic->p_heap->i_gmask = 0x00ff00;
+            p_pic->p_heap->i_bmask = 0x0000ff; */
+            p_pic->i_planes = 1;
+            break;
+
+        case FOURCC_RV32:
+            p_pic->p->i_lines = i_height;
+            p_pic->p->i_pitch = i_width * 4;
+            p_pic->p->i_visible_pitch = p_pic->p->i_pitch;
+            p_pic->p->i_pixel_pitch = 4;
+/* FIXME: p_heap isn't always reachable
+            p_pic->p_heap->i_rmask = 0xff0000;
+            p_pic->p_heap->i_gmask = 0x00ff00;
+            p_pic->p_heap->i_bmask = 0x0000ff; */
+            p_pic->i_planes = 1;
             break;
 
         default:
-            intf_ErrMsg( "vout error: unknown chroma type %d",
-                         p_vout->render.i_chroma );
-            p_picture->i_planes = 0;
+            msg_Err( p_this, "unknown chroma type 0x%.8x (%4.4s)",
+                             i_chroma, (char*)&i_chroma );
+            p_pic->i_planes = 0;
             return;
     }
+}
 
-    p_picture->i_type = MEMORY_PICTURE;
+/*****************************************************************************
+ * vout_ChromaCmp: compare two chroma values
+ *****************************************************************************
+ * This function returns 1 if the two fourcc values given as argument are
+ * the same format (eg. UYVY/UYNV) or almost the same format (eg. I420/YV12)
+ *****************************************************************************/
+int vout_ChromaCmp( vlc_fourcc_t i_chroma, vlc_fourcc_t i_amorhc )
+{
+    /* If they are the same, they are the same ! */
+    if( i_chroma == i_amorhc )
+    {
+        return 1;
+    }
 
-    /* Allocate memory */
-    switch( p_vout->render.i_chroma )
+    /* Check for equivalence classes */
+    switch( i_chroma )
     {
-        case YUV_420_PICTURE:        /* YUV 420: 1,1/4,1/4 samples per pixel */
-        case YUV_422_PICTURE:        /* YUV 422: 1,1/2,1/2 samples per pixel */
-        case YUV_444_PICTURE:            /* YUV 444: 1,1,1 samples per pixel */
-
-            i_data_size = p_picture->i_size + 2 * p_picture->i_chroma_size;
-
-            /* The Y plane */
-            p_picture->planes[ Y_PLANE ].i_bytes =
-                 p_picture->i_size * sizeof(pixel_data_t);
-            p_picture->planes[ Y_PLANE ].p_data =
-                 memalign( 16, i_data_size * sizeof(pixel_data_t) * 4 );
-            /* The U plane */
-            p_picture->planes[ U_PLANE ].i_bytes =
-                 p_picture->i_chroma_size * sizeof(pixel_data_t);
-            p_picture->planes[ U_PLANE ].p_data =
-                 p_picture->planes[ Y_PLANE ].p_data + p_picture->i_size;
-            /* The V plane */
-            p_picture->planes[ V_PLANE ].i_bytes =
-                 p_picture->i_chroma_size * sizeof(pixel_data_t);
-            p_picture->planes[ V_PLANE ].p_data =
-                 p_picture->planes[ U_PLANE ].p_data + p_picture->i_chroma_size;
-
-            p_picture->i_planes = 3;
+        case FOURCC_I420:
+        case FOURCC_IYUV:
+        case FOURCC_YV12:
+            switch( i_amorhc )
+            {
+                case FOURCC_I420:
+                case FOURCC_IYUV:
+                case FOURCC_YV12:
+                    return 1;
 
-            break;
+                default:
+                    return 0;
+            }
 
-        default:
-            p_picture->i_planes = 0;
+        case FOURCC_UYVY:
+        case FOURCC_UYNV:
+        case FOURCC_Y422:
+            switch( i_amorhc )
+            {
+                case FOURCC_UYVY:
+                case FOURCC_UYNV:
+                case FOURCC_Y422:
+                    return 1;
 
-            break;
+                default:
+                    return 0;
+            }
+
+        case FOURCC_YUY2:
+        case FOURCC_YUNV:
+            switch( i_amorhc )
+            {
+                case FOURCC_YUY2:
+                case FOURCC_YUNV:
+                    return 1;
+
+                default:
+                    return 0;
+            }
+
+        default:
+            return 0;
     }
 }
 
+/* Following functions are local */
+
+/*****************************************************************************
+ * CopyPicture: copy a picture to another one
+ *****************************************************************************
+ * This function takes advantage of the image format, and reduces the
+ * number of calls to memcpy() to the minimum. Source and destination
+ * images must have same width (hence i_visible_pitch), height, and chroma.
+ *****************************************************************************/
+static void CopyPicture( vout_thread_t * p_vout,
+                         picture_t *p_src, picture_t *p_dest )
+{
+    int i;
+
+    for( i = 0; i < p_src->i_planes ; i++ )
+    {
+        if( p_src->p[i].i_pitch == p_dest->p[i].i_pitch )
+        {
+            /* There are margins, but with the same width : perfect ! */
+            p_vout->p_vlc->pf_memcpy(
+                         p_dest->p[i].p_pixels, p_src->p[i].p_pixels,
+                         p_src->p[i].i_pitch * p_src->p[i].i_lines );
+        }
+        else
+        {
+            /* We need to proceed line by line */
+            uint8_t *p_in = p_src->p[i].p_pixels;
+            uint8_t *p_out = p_dest->p[i].p_pixels;
+            int i_line;
+
+            for( i_line = p_src->p[i].i_lines; i_line--; )
+            {
+                p_vout->p_vlc->pf_memcpy( p_out, p_in,
+                                          p_src->p[i].i_visible_pitch );
+                p_in += p_src->p[i].i_pitch;
+                p_out += p_dest->p[i].i_pitch;
+            }
+        }
+    }
+    p_dest->date = p_src->date;
+}