video_output.c 69.2 KB
Newer Older
1
/*****************************************************************************
Michel Kaempf's avatar
Michel Kaempf committed
2
 * video_output.c : video output thread
3
 *
Michel Kaempf's avatar
Michel Kaempf committed
4 5
 * This module describes the programming interface for video output threads.
 * It includes functions allowing to open a new thread, send pictures to a
6 7
 * thread, and destroy a previously oppened video output thread.
 *****************************************************************************
8
 * Copyright (C) 2000-2007 the VideoLAN team
Gildas Bazin's avatar
Gildas Bazin committed
9
 * $Id$
10
 *
Sam Hocevar's avatar
 
Sam Hocevar committed
11
 * Authors: Vincent Seguin <seguin@via.ecp.fr>
12
 *          Gildas Bazin <gbazin@videolan.org>
13 14 15 16 17
 *
 * 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.
18
 *
19 20
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
21 22
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
23
 *
24 25
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
Antoine Cellerier's avatar
Antoine Cellerier committed
26
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
27
 *****************************************************************************/
Michel Kaempf's avatar
Michel Kaempf committed
28

29
/*****************************************************************************
Michel Kaempf's avatar
Michel Kaempf committed
30
 * Preamble
31
 *****************************************************************************/
32 33 34 35
#ifdef HAVE_CONFIG_H
# include "config.h"
#endif

36
#include <vlc_common.h>
37

38
#include <stdlib.h>                                                /* free() */
39
#include <string.h>
Vincent Seguin's avatar
Vincent Seguin committed
40

Sam Hocevar's avatar
 
Sam Hocevar committed
41

42 43
#ifdef HAVE_SYS_TIMES_H
#   include <sys/times.h>
44
#endif
45

Clément Stenac's avatar
Clément Stenac committed
46
#include <vlc_vout.h>
47

Clément Stenac's avatar
Clément Stenac committed
48 49
#include <vlc_filter.h>
#include <vlc_osd.h>
Laurent Aimar's avatar
Laurent Aimar committed
50
#include <assert.h>
51

52
#if defined( __APPLE__ )
Clément Stenac's avatar
Clément Stenac committed
53
/* Include darwin_specific.h here if needed */
54 55
#endif

Clément Stenac's avatar
Clément Stenac committed
56 57
/** FIXME This is quite ugly but needed while we don't have counters
 * helpers */
58
//#include "input/input_internal.h"
Clément Stenac's avatar
Clément Stenac committed
59

60 61
#include <libvlc.h>
#include <vlc_input.h>
62
#include "vout_pictures.h"
Laurent Aimar's avatar
Laurent Aimar committed
63
#include "vout_internal.h"
64

65
/*****************************************************************************
Michel Kaempf's avatar
Michel Kaempf committed
66
 * Local prototypes
67
 *****************************************************************************/
68
static int      InitThread        ( vout_thread_t * );
69
static void*    RunThread         ( void *  );
70
static void     ErrorThread       ( vout_thread_t * );
Laurent Aimar's avatar
Laurent Aimar committed
71
static void     CleanThread       ( vout_thread_t * );
72
static void     EndThread         ( vout_thread_t * );
Sam Hocevar's avatar
 
Sam Hocevar committed
73

Laurent Aimar's avatar
Laurent Aimar committed
74
static void VideoFormatImportRgb( video_format_t *, const picture_heap_t * );
75 76
static void PictureHeapFixRgb( picture_heap_t * );

77 78
static void     vout_Destructor   ( vlc_object_t * p_this );

Gildas Bazin's avatar
 
Gildas Bazin committed
79
/* Object variables callbacks */
80 81
static int FilterCallback( vlc_object_t *, char const *,
                           vlc_value_t, vlc_value_t, void * );
82 83
static int VideoFilter2Callback( vlc_object_t *, char const *,
                                 vlc_value_t, vlc_value_t, void * );
84 85

/* */
86 87 88 89 90
static void PostProcessEnable( vout_thread_t * );
static void PostProcessDisable( vout_thread_t * );
static void PostProcessSetFilterQuality( vout_thread_t *p_vout );
static int  PostProcessCallback( vlc_object_t *, char const *,
                                 vlc_value_t, vlc_value_t, void * );
91 92
/* */
static void DeinterlaceEnable( vout_thread_t * );
93
static void DeinterlaceNeeded( vout_thread_t *, bool );
Gildas Bazin's avatar
 
Gildas Bazin committed
94

95 96 97
/* From vout_intf.c */
int vout_Snapshot( vout_thread_t *, picture_t * );

98 99 100
/* Display media title in OSD */
static void DisplayTitleOnOSD( vout_thread_t *p_vout );

101 102
/* Time during which the thread will sleep if it has nothing to
 * display (in micro-seconds) */
103
#define VOUT_IDLE_SLEEP                 ((int)(0.020*CLOCK_FREQ))
104 105 106 107 108 109 110 111 112 113 114

/* Maximum lap of time allowed between the beginning of rendering and
 * display. If, compared to the current date, the next image is too
 * late, the thread will perform an idle loop. This time should be
 * at least VOUT_IDLE_SLEEP plus the time required to render a few
 * images, to avoid trashing of decoded images */
#define VOUT_DISPLAY_DELAY              ((int)(0.200*CLOCK_FREQ))

/* Better be in advance when awakening than late... */
#define VOUT_MWAIT_TOLERANCE            ((mtime_t)(0.020*CLOCK_FREQ))

115 116 117 118 119 120 121 122
/* Minimum number of direct pictures the video output will accept without
 * creating additional pictures in system memory */
#ifdef OPTIMIZE_MEMORY
#   define VOUT_MIN_DIRECT_PICTURES        (VOUT_MAX_PICTURES/2)
#else
#   define VOUT_MIN_DIRECT_PICTURES        (3*VOUT_MAX_PICTURES/4)
#endif

123 124 125 126 127
/*****************************************************************************
 * Video Filter2 functions
 *****************************************************************************/
static picture_t *video_new_buffer_filter( filter_t *p_filter )
{
128
    vout_thread_t *p_vout = (vout_thread_t*)p_filter->p_owner;
129
    picture_t *p_picture = vout_CreatePicture( p_vout, 0, 0, 0 );
130

131
    p_picture->i_status = READY_PICTURE;
132 133 134 135 136 137

    return p_picture;
}

static void video_del_buffer_filter( filter_t *p_filter, picture_t *p_pic )
{
138 139
    vout_thread_t *p_vout = (vout_thread_t*)p_filter->p_owner;

140 141 142
    vlc_mutex_lock( &p_vout->picture_lock );
    vout_UsePictureLocked( p_vout, p_pic );
    vlc_mutex_unlock( &p_vout->picture_lock );
143 144 145 146 147 148 149 150
}

static int video_filter_buffer_allocation_init( filter_t *p_filter, void *p_data )
{
    p_filter->pf_vout_buffer_new = video_new_buffer_filter;
    p_filter->pf_vout_buffer_del = video_del_buffer_filter;
    p_filter->p_owner = p_data; /* p_vout */
    return VLC_SUCCESS;
151 152
}

153
/*****************************************************************************
154 155 156 157 158
 * vout_Request: find a video output thread, create one, or destroy one.
 *****************************************************************************
 * This function looks for a video output thread matching the current
 * properties. If not found, it spawns a new one.
 *****************************************************************************/
159 160
vout_thread_t *__vout_Request( vlc_object_t *p_this, vout_thread_t *p_vout,
                               video_format_t *p_fmt )
161
{
162
    if( !p_fmt )
163
    {
164 165 166
        /* Video output is no longer used.
         * TODO: support for reusing video outputs with proper _thread-safe_
         * reference handling. */
167
        if( p_vout )
168
            vout_CloseAndRelease( p_vout );
169 170 171 172 173 174
        return NULL;
    }

    /* If a video output was provided, lock it, otherwise look for one. */
    if( p_vout )
    {
175
        vlc_object_hold( p_vout );
176 177
    }

178
    /* TODO: find a suitable unused video output */
179 180 181 182

    /* If we now have a video output, check it has the right properties */
    if( p_vout )
    {
183 184
        vlc_mutex_lock( &p_vout->change_lock );

185
        /* We don't directly check for the "vout-filter" variable for obvious
Gildas Bazin's avatar
 
Gildas Bazin committed
186
         * performance reasons. */
187
        if( p_vout->p->b_filter_change )
Gildas Bazin's avatar
 
Gildas Bazin committed
188
        {
189
            char *psz_filter_chain = var_GetString( p_vout, "vout-filter" );
Gildas Bazin's avatar
 
Gildas Bazin committed
190

191
            if( psz_filter_chain && !*psz_filter_chain )
Gildas Bazin's avatar
 
Gildas Bazin committed
192 193 194 195
            {
                free( psz_filter_chain );
                psz_filter_chain = NULL;
            }
196
            if( p_vout->p->psz_filter_chain && !*p_vout->p->psz_filter_chain )
Gildas Bazin's avatar
 
Gildas Bazin committed
197
            {
198 199
                free( p_vout->p->psz_filter_chain );
                p_vout->p->psz_filter_chain = NULL;
Gildas Bazin's avatar
 
Gildas Bazin committed
200 201
            }

202
            if( !psz_filter_chain && !p_vout->p->psz_filter_chain )
Gildas Bazin's avatar
 
Gildas Bazin committed
203
            {
204
                p_vout->p->b_filter_change = false;
Gildas Bazin's avatar
 
Gildas Bazin committed
205 206
            }

207
            free( psz_filter_chain );
Gildas Bazin's avatar
 
Gildas Bazin committed
208 209
        }

210
        if( p_vout->fmt_render.i_chroma != vlc_fourcc_GetCodec( VIDEO_ES, p_fmt->i_chroma ) ||
211 212
            p_vout->fmt_render.i_width != p_fmt->i_width ||
            p_vout->fmt_render.i_height != p_fmt->i_height ||
213
            p_vout->p->b_filter_change )
214
        {
215 216
            vlc_mutex_unlock( &p_vout->change_lock );

217
            /* We are not interested in this format, close this vout */
218
            vout_CloseAndRelease( p_vout );
219
            vlc_object_release( p_vout );
220 221 222 223 224
            p_vout = NULL;
        }
        else
        {
            /* This video output is cool! Hijack it. */
225 226 227 228 229 230
            /* Correct aspect ratio on change
             * FIXME factorize this code with other aspect ration related code */
            unsigned int i_sar_num;
            unsigned int i_sar_den;
            vlc_ureduce( &i_sar_num, &i_sar_den,
                         p_fmt->i_sar_num, p_fmt->i_sar_den, 50000 );
231
#if 0
232 233 234 235 236 237
            /* What's that, it does not seems to be used correcly everywhere */
            if( p_vout->i_par_num > 0 && p_vout->i_par_den > 0 )
            {
                i_sar_num *= p_vout->i_par_den;
                i_sar_den *= p_vout->i_par_num;
            }
238 239
#endif

240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255
            if( i_sar_num > 0 && i_sar_den > 0 &&
                ( i_sar_num != p_vout->fmt_render.i_sar_num ||
                  i_sar_den != p_vout->fmt_render.i_sar_den ) )
            {
                p_vout->fmt_in.i_sar_num = i_sar_num;
                p_vout->fmt_in.i_sar_den = i_sar_den;

                p_vout->fmt_render.i_sar_num = i_sar_num;
                p_vout->fmt_render.i_sar_den = i_sar_den;

                p_vout->render.i_aspect = (int64_t)i_sar_num *
                                                   p_vout->fmt_render.i_width *
                                                   VOUT_ASPECT_FACTOR /
                                                   i_sar_den /
                                                   p_vout->fmt_render.i_height;
                p_vout->i_changes |= VOUT_ASPECT_CHANGE;
256 257 258 259 260 261 262 263 264 265
            }
            vlc_mutex_unlock( &p_vout->change_lock );

            vlc_object_release( p_vout );
        }

        if( p_vout )
        {
            msg_Dbg( p_this, "reusing provided vout" );

266
            spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), false );
267
            vlc_object_detach( p_vout );
268

269
            vlc_object_attach( p_vout, p_this );
270
            spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), true );
271 272 273 274 275 276 277
        }
    }

    if( !p_vout )
    {
        msg_Dbg( p_this, "no usable vout present, spawning one" );

278
        p_vout = vout_Create( p_this, p_fmt );
279 280 281 282 283 284 285
    }

    return p_vout;
}

/*****************************************************************************
 * vout_Create: creates a new video output thread
286
 *****************************************************************************
Michel Kaempf's avatar
Michel Kaempf committed
287 288
 * This function creates a new video output thread, and returns a pointer
 * to its description. On error, it returns NULL.
289
 *****************************************************************************/
290
vout_thread_t * __vout_Create( vlc_object_t *p_parent, video_format_t *p_fmt )
Michel Kaempf's avatar
Michel Kaempf committed
291
{
Gildas Bazin's avatar
 
Gildas Bazin committed
292 293
    vout_thread_t  * p_vout;                            /* thread descriptor */
    int              i_index;                               /* loop variable */
294
    vlc_value_t      text;
Michel Kaempf's avatar
Michel Kaempf committed
295

296 297
    unsigned int i_width = p_fmt->i_width;
    unsigned int i_height = p_fmt->i_height;
298
    vlc_fourcc_t i_chroma = vlc_fourcc_GetCodec( VIDEO_ES, p_fmt->i_chroma );
299

300 301 302 303
    config_chain_t *p_cfg;
    char *psz_parser;
    char *psz_name;

304
    if( i_width <= 0 || i_height <= 0 )
305 306 307 308 309 310
        return NULL;

    vlc_ureduce( &p_fmt->i_sar_num, &p_fmt->i_sar_den,
                 p_fmt->i_sar_num, p_fmt->i_sar_den, 50000 );
    if( p_fmt->i_sar_num <= 0 || p_fmt->i_sar_den <= 0 )
        return NULL;
311 312 313 314 315
    unsigned int i_aspect = (int64_t)p_fmt->i_sar_num *
                                     i_width *
                                     VOUT_ASPECT_FACTOR /
                                     p_fmt->i_sar_den /
                                     i_height;
316

317
    /* Allocate descriptor */
318 319 320
    static const char typename[] = "video output";
    p_vout = vlc_custom_create( p_parent, sizeof( *p_vout ), VLC_OBJECT_VOUT,
                                typename );
321
    if( p_vout == NULL )
322
        return NULL;
323

324
    /* */
Laurent Aimar's avatar
Laurent Aimar committed
325
    p_vout->p = calloc( 1, sizeof(*p_vout->p) );
326 327 328 329 330 331
    if( !p_vout->p )
    {
        vlc_object_release( p_vout );
        return NULL;
    }

332
    /* Initialize pictures - translation tables and functions
Gildas Bazin's avatar
 
Gildas Bazin committed
333
     * will be initialized later in InitThread */
334
    for( i_index = 0; i_index < 2 * VOUT_MAX_PICTURES + 1; i_index++)
Gildas Bazin's avatar
 
Gildas Bazin committed
335 336 337 338 339
    {
        p_vout->p_picture[i_index].pf_lock = NULL;
        p_vout->p_picture[i_index].pf_unlock = NULL;
        p_vout->p_picture[i_index].i_status = FREE_PICTURE;
        p_vout->p_picture[i_index].i_type   = EMPTY_PICTURE;
340
        p_vout->p_picture[i_index].b_slow   = 0;
Gildas Bazin's avatar
 
Gildas Bazin committed
341 342 343 344 345 346 347
    }

    /* No images in the heap */
    p_vout->i_heap_size = 0;

    /* Initialize the rendering heap */
    I_RENDERPICTURES = 0;
348

349 350
    p_vout->fmt_render        = *p_fmt;   /* FIXME palette */
    p_vout->fmt_in            = *p_fmt;   /* FIXME palette */
351

Gildas Bazin's avatar
 
Gildas Bazin committed
352 353 354 355 356
    p_vout->render.i_width    = i_width;
    p_vout->render.i_height   = i_height;
    p_vout->render.i_chroma   = i_chroma;
    p_vout->render.i_aspect   = i_aspect;

357 358 359
    p_vout->render.i_rmask    = p_fmt->i_rmask;
    p_vout->render.i_gmask    = p_fmt->i_gmask;
    p_vout->render.i_bmask    = p_fmt->i_bmask;
Gildas Bazin's avatar
 
Gildas Bazin committed
360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376

    p_vout->render.i_last_used_pic = -1;
    p_vout->render.b_allow_modify_pics = 1;

    /* Zero the output heap */
    I_OUTPUTPICTURES = 0;
    p_vout->output.i_width    = 0;
    p_vout->output.i_height   = 0;
    p_vout->output.i_chroma   = 0;
    p_vout->output.i_aspect   = 0;

    p_vout->output.i_rmask    = 0;
    p_vout->output.i_gmask    = 0;
    p_vout->output.i_bmask    = 0;

    /* Initialize misc stuff */
    p_vout->i_changes    = 0;
377 378
    p_vout->b_autoscale  = 1;
    p_vout->i_zoom      = ZOOM_FP_FACTOR;
Gildas Bazin's avatar
 
Gildas Bazin committed
379
    p_vout->b_fullscreen = 0;
Gildas Bazin's avatar
 
Gildas Bazin committed
380
    p_vout->i_alignment  = 0;
381 382
    p_vout->p->render_time  = 10;
    p_vout->p->c_fps_samples = 0;
383
    vout_statistic_Init( &p_vout->p->statistic );
384
    p_vout->p->b_filter_change = 0;
Laurent Aimar's avatar
Laurent Aimar committed
385 386
    p_vout->p->b_paused = false;
    p_vout->p->i_pause_date = 0;
387
    p_vout->pf_control = NULL;
388 389
    p_vout->p->i_par_num =
    p_vout->p->i_par_den = 1;
390
    p_vout->p->p_picture_displayed = NULL;
391 392 393
    p_vout->p->i_picture_displayed_date = 0;
    p_vout->p->b_picture_displayed = false;
    p_vout->p->b_picture_empty = false;
394
    p_vout->p->i_picture_qtype = QTYPE_NONE;
395
    p_vout->p->b_picture_interlaced = false;
Gildas Bazin's avatar
 
Gildas Bazin committed
396

397 398
    vlc_mouse_Init( &p_vout->p->mouse );

399
    vout_snapshot_Init( &p_vout->p->snapshot );
400

Gildas Bazin's avatar
 
Gildas Bazin committed
401
    /* Initialize locks */
402 403
    vlc_mutex_init( &p_vout->picture_lock );
    vlc_cond_init( &p_vout->p->picture_wait );
404
    vlc_mutex_init( &p_vout->change_lock );
405
    vlc_mutex_init( &p_vout->p->vfilter_lock );
Gildas Bazin's avatar
 
Gildas Bazin committed
406

407 408 409 410 411
    /* Mouse coordinates */
    var_Create( p_vout, "mouse-x", VLC_VAR_INTEGER );
    var_Create( p_vout, "mouse-y", VLC_VAR_INTEGER );
    var_Create( p_vout, "mouse-button-down", VLC_VAR_INTEGER );
    var_Create( p_vout, "mouse-moved", VLC_VAR_BOOL );
412
    var_Create( p_vout, "mouse-clicked", VLC_VAR_BOOL );
413

414
    /* Initialize subpicture unit */
415
    p_vout->p_spu = spu_Create( p_vout );
416

Gildas Bazin's avatar
 
Gildas Bazin committed
417 418 419
    /* Attach the new object now so we can use var inheritance below */
    vlc_object_attach( p_vout, p_parent );

420
    /* */
421 422
    spu_Init( p_vout->p_spu );

423 424
    spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), true );

425 426
    /* Take care of some "interface/control" related initialisations */
    vout_IntfInit( p_vout );
Gildas Bazin's avatar
 
Gildas Bazin committed
427

Gildas Bazin's avatar
 
Gildas Bazin committed
428 429
    /* If the parent is not a VOUT object, that means we are at the start of
     * the video output pipe */
430
    if( vlc_internals( p_parent )->i_object_type != VLC_OBJECT_VOUT )
Sam Hocevar's avatar
 
Sam Hocevar committed
431
    {
432
        /* Look for the default filter configuration */
433
        p_vout->p->psz_filter_chain =
434
            var_CreateGetStringCommand( p_vout, "vout-filter" );
435 436

        /* Apply video filter2 objects on the first vout */
437
        p_vout->p->psz_vf2 =
438
            var_CreateGetStringCommand( p_vout, "video-filter" );
439 440

        p_vout->p->b_first_vout = true;
Gildas Bazin's avatar
 
Gildas Bazin committed
441 442 443 444
    }
    else
    {
        /* continue the parent's filter chain */
445
        char *psz_tmp;
Gildas Bazin's avatar
 
Gildas Bazin committed
446

447
        /* Ugly hack to jump to our configuration chain */
448 449 450 451
        p_vout->p->psz_filter_chain
            = ((vout_thread_t *)p_parent)->p->psz_filter_chain;
        p_vout->p->psz_filter_chain
            = config_ChainCreate( &psz_tmp, &p_cfg, p_vout->p->psz_filter_chain );
452 453
        config_ChainDestroy( p_cfg );
        free( psz_tmp );
454 455

        /* Create a video filter2 var ... but don't inherit values */
456 457
        var_Create( p_vout, "video-filter",
                    VLC_VAR_STRING | VLC_VAR_ISCOMMAND );
458
        p_vout->p->psz_vf2 = var_GetString( p_vout, "video-filter" );
459 460 461

        /* */
        p_vout->p->b_first_vout = false;
Gildas Bazin's avatar
 
Gildas Bazin committed
462 463
    }

464
    var_AddCallback( p_vout, "video-filter", VideoFilter2Callback, NULL );
465
    p_vout->p->p_vf2_chain = filter_chain_New( p_vout, "video filter2",
466
        false, video_filter_buffer_allocation_init, NULL, p_vout );
467

Gildas Bazin's avatar
 
Gildas Bazin committed
468
    /* Choose the video output module */
469
    if( !p_vout->p->psz_filter_chain || !*p_vout->p->psz_filter_chain )
Gildas Bazin's avatar
 
Gildas Bazin committed
470
    {
471
        psz_parser = var_CreateGetString( p_vout, "vout" );
Gildas Bazin's avatar
 
Gildas Bazin committed
472 473 474
    }
    else
    {
475 476
        psz_parser = strdup( p_vout->p->psz_filter_chain );
        p_vout->p->b_title_show = false;
Sam Hocevar's avatar
 
Sam Hocevar committed
477 478
    }

Gildas Bazin's avatar
 
Gildas Bazin committed
479
    /* Create the vout thread */
480
    char* psz_tmp = config_ChainCreate( &psz_name, &p_cfg, psz_parser );
Antoine Cellerier's avatar
Antoine Cellerier committed
481
    free( psz_parser );
482
    free( psz_tmp );
483
    p_vout->p_cfg = p_cfg;
Laurent Aimar's avatar
Laurent Aimar committed
484 485 486 487 488 489 490 491 492 493

    /* Create a few object variables for interface interaction */
    var_Create( p_vout, "vout-filter", VLC_VAR_STRING | VLC_VAR_DOINHERIT );
    text.psz_string = _("Filters");
    var_Change( p_vout, "vout-filter", VLC_VAR_SETTEXT, &text, NULL );
    var_AddCallback( p_vout, "vout-filter", FilterCallback, NULL );

    /* */
    DeinterlaceEnable( p_vout );

494 495 496 497 498 499
    if( p_vout->p->psz_filter_chain && *p_vout->p->psz_filter_chain )
        p_vout->p->psz_module_type = "video filter";
    else
        p_vout->p->psz_module_type = "video output";
    p_vout->p->psz_module_name = psz_name;
    p_vout->p_module = NULL;
Laurent Aimar's avatar
Laurent Aimar committed
500 501

    /* */
502
    vlc_object_set_destructor( p_vout, vout_Destructor );
Gildas Bazin's avatar
 
Gildas Bazin committed
503

504
    /* */
505 506 507
    vlc_cond_init( &p_vout->p->change_wait );
    if( vlc_clone( &p_vout->p->thread, RunThread, p_vout,
                   VLC_THREAD_PRIORITY_OUTPUT ) )
Michel Kaempf's avatar
Michel Kaempf committed
508
    {
509
        spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), false );
510 511
        spu_Destroy( p_vout->p_spu );
        p_vout->p_spu = NULL;
512
        vlc_object_release( p_vout );
513 514 515
        return NULL;
    }

516 517 518 519 520 521 522 523 524
    vlc_mutex_lock( &p_vout->change_lock );
    while( !p_vout->p->b_ready )
    {   /* We are (ab)using the same condition in opposite directions for
         * b_ready and b_done. This works because of the strict ordering. */
        assert( !p_vout->p->b_done );
        vlc_cond_wait( &p_vout->p->change_wait, &p_vout->change_lock );
    }
    vlc_mutex_unlock( &p_vout->change_lock );

525 526 527
    if( p_vout->b_error )
    {
        msg_Err( p_vout, "video output creation failed" );
528
        vout_CloseAndRelease( p_vout );
529
        return NULL;
530
    }
Michel Kaempf's avatar
Michel Kaempf committed
531

532
    return p_vout;
Michel Kaempf's avatar
Michel Kaempf committed
533 534
}

535
/*****************************************************************************
536
 * vout_Close: Close a vout created by vout_Create.
537
 *****************************************************************************
538
 * You HAVE to call it on vout created by vout_Create before vlc_object_release.
Rémi Denis-Courmont's avatar
Typo  
Rémi Denis-Courmont committed
539
 * You should NEVER call it on vout not obtained through vout_Create
540
 * (like with vout_Request or vlc_object_find.)
541
 * You can use vout_CloseAndRelease() as a convenience method.
542
 *****************************************************************************/
543
void vout_Close( vout_thread_t *p_vout )
544 545 546
{
    assert( p_vout );

547 548 549 550
    vlc_mutex_lock( &p_vout->change_lock );
    p_vout->p->b_done = true;
    vlc_cond_signal( &p_vout->p->change_wait );
    vlc_mutex_unlock( &p_vout->change_lock );
551

552
    vout_snapshot_End( &p_vout->p->snapshot );
553

554
    vlc_join( p_vout->p->thread, NULL );
555 556 557
}

/* */
558 559
static void vout_Destructor( vlc_object_t * p_this )
{
560
    vout_thread_t *p_vout = (vout_thread_t *)p_this;
561

562 563 564
    /* Make sure the vout was stopped first */
    assert( !p_vout->p_module );

565 566
    free( p_vout->p->psz_module_name );

Laurent Aimar's avatar
Laurent Aimar committed
567
    /* */
568 569
    if( p_vout->p_spu )
        spu_Destroy( p_vout->p_spu );
Laurent Aimar's avatar
Laurent Aimar committed
570

571
    /* Destroy the locks */
572
    vlc_cond_destroy( &p_vout->p->change_wait );
573
    vlc_cond_destroy( &p_vout->p->picture_wait );
574 575
    vlc_mutex_destroy( &p_vout->picture_lock );
    vlc_mutex_destroy( &p_vout->change_lock );
576
    vlc_mutex_destroy( &p_vout->p->vfilter_lock );
577

578 579 580
    /* */
    vout_statistic_Clean( &p_vout->p->statistic );

581
    /* */
582
    vout_snapshot_Clean( &p_vout->p->snapshot );
583 584

    /* */
585
    free( p_vout->p->psz_filter_chain );
586
    free( p_vout->p->psz_title );
Gildas Bazin's avatar
 
Gildas Bazin committed
587

588 589
    config_ChainDestroy( p_vout->p_cfg );

590 591
    free( p_vout->p );

Michel Kaempf's avatar
Michel Kaempf committed
592 593
}

Laurent Aimar's avatar
Laurent Aimar committed
594 595 596
/* */
void vout_ChangePause( vout_thread_t *p_vout, bool b_paused, mtime_t i_date )
{
597
    vlc_mutex_lock( &p_vout->change_lock );
Laurent Aimar's avatar
Laurent Aimar committed
598

599
    assert( !p_vout->p->b_paused || !b_paused );
600 601 602 603 604

    vlc_mutex_lock( &p_vout->picture_lock );

    p_vout->p->i_picture_displayed_date = 0;

Laurent Aimar's avatar
Laurent Aimar committed
605 606 607 608 609 610 611 612 613 614 615
    if( p_vout->p->b_paused )
    {
        const mtime_t i_duration = i_date - p_vout->p->i_pause_date;

        for( int i_index = 0; i_index < I_RENDERPICTURES; i_index++ )
        {
            picture_t *p_pic = PP_RENDERPICTURE[i_index];

            if( p_pic->i_status == READY_PICTURE )
                p_pic->date += i_duration;
        }
616 617 618
        vlc_cond_signal( &p_vout->p->picture_wait );
        vlc_mutex_unlock( &p_vout->picture_lock );

Laurent Aimar's avatar
Laurent Aimar committed
619
        spu_OffsetSubtitleDate( p_vout->p_spu, i_duration );
Laurent Aimar's avatar
Laurent Aimar committed
620
    }
621 622 623 624
    else
    {
        vlc_mutex_unlock( &p_vout->picture_lock );
    }
Laurent Aimar's avatar
Laurent Aimar committed
625 626 627
    p_vout->p->b_paused = b_paused;
    p_vout->p->i_pause_date = i_date;

628
    vlc_mutex_unlock( &p_vout->change_lock );
Laurent Aimar's avatar
Laurent Aimar committed
629
}
630

631 632
void vout_GetResetStatistic( vout_thread_t *p_vout, int *pi_displayed, int *pi_lost )
{
633 634
    vout_statistic_GetReset( &p_vout->p->statistic,
                             pi_displayed, pi_lost );
635
}
636

Laurent Aimar's avatar
Laurent Aimar committed
637 638 639
void vout_Flush( vout_thread_t *p_vout, mtime_t i_date )
{
    vlc_mutex_lock( &p_vout->picture_lock );
640
    p_vout->p->i_picture_displayed_date = 0;
Laurent Aimar's avatar
Laurent Aimar committed
641 642 643
    for( int i = 0; i < p_vout->render.i_pictures; i++ )
    {
        picture_t *p_pic = p_vout->render.pp_picture[i];
Laurent Aimar's avatar
Laurent Aimar committed
644

Laurent Aimar's avatar
Laurent Aimar committed
645 646 647 648 649 650 651 652 653
        if( p_pic->i_status == READY_PICTURE ||
            p_pic->i_status == DISPLAYED_PICTURE )
        {
            /* We cannot change picture status if it is in READY_PICTURE state,
             * Just make sure they won't be displayed */
            if( p_pic->date > i_date )
                p_pic->date = i_date;
        }
    }
654
    vlc_cond_signal( &p_vout->p->picture_wait );
Laurent Aimar's avatar
Laurent Aimar committed
655 656
    vlc_mutex_unlock( &p_vout->picture_lock );
}
657

658
void vout_FixLeaks( vout_thread_t *p_vout, bool b_forced )
659 660 661 662 663
{
    int i_pic, i_ready_pic;

    vlc_mutex_lock( &p_vout->picture_lock );

664
    for( i_pic = 0, i_ready_pic = 0; i_pic < p_vout->render.i_pictures && !b_forced; i_pic++ )
665 666 667 668 669 670 671 672 673 674 675 676 677 678 679 680 681 682 683 684 685 686
    {
        const picture_t *p_pic = p_vout->render.pp_picture[i_pic];

        if( p_pic->i_status == READY_PICTURE )
        {
            i_ready_pic++;
            /* If we have at least 2 ready pictures, wait for the vout thread to
             * process one */
            if( i_ready_pic >= 2 )
                break;

            continue;
        }

        if( p_pic->i_status == DISPLAYED_PICTURE )
        {
            /* If at least one displayed picture is not referenced
             * let vout free it */
            if( p_pic->i_refcount == 0 )
                break;
        }
    }
687
    if( i_pic < p_vout->render.i_pictures && !b_forced )
688 689 690 691 692 693 694
    {
        vlc_mutex_unlock( &p_vout->picture_lock );
        return;
    }

    /* Too many pictures are still referenced, there is probably a bug
     * with the decoder */
695 696
    if( !b_forced )
        msg_Err( p_vout, "pictures leaked, resetting the heap" );
697 698 699 700 701

    /* Just free all the pictures */
    for( i_pic = 0; i_pic < p_vout->render.i_pictures; i_pic++ )
    {
        picture_t *p_pic = p_vout->render.pp_picture[i_pic];
702 703

        msg_Dbg( p_vout, "[%d] %d %d", i_pic, p_pic->i_status, p_pic->i_refcount );
704
        p_pic->i_refcount = 0;
Laurent Aimar's avatar
Laurent Aimar committed
705 706 707 708 709 710 711 712 713 714

        switch( p_pic->i_status )
        {
        case READY_PICTURE:
        case DISPLAYED_PICTURE:
        case RESERVED_PICTURE:
            if( p_pic != p_vout->p->p_picture_displayed )
                vout_UsePictureLocked( p_vout, p_pic );
            break;
        }
715
    }
716
    vlc_cond_signal( &p_vout->p->picture_wait );
717 718
    vlc_mutex_unlock( &p_vout->picture_lock );
}
719 720 721 722 723 724 725 726 727 728 729 730 731 732 733 734 735 736 737 738 739 740 741
void vout_NextPicture( vout_thread_t *p_vout, mtime_t *pi_duration )
{
    vlc_mutex_lock( &p_vout->picture_lock );

    const mtime_t i_displayed_date = p_vout->p->i_picture_displayed_date;

    p_vout->p->b_picture_displayed = false;
    p_vout->p->b_picture_empty = false;
    if( p_vout->p->p_picture_displayed )
    {
        p_vout->p->p_picture_displayed->date = 1;
        vlc_cond_signal( &p_vout->p->picture_wait );
    }

    while( !p_vout->p->b_picture_displayed && !p_vout->p->b_picture_empty )
        vlc_cond_wait( &p_vout->p->picture_wait, &p_vout->picture_lock );

    *pi_duration = __MAX( p_vout->p->i_picture_displayed_date - i_displayed_date, 0 );

    /* TODO advance subpicture by the duration ... */

    vlc_mutex_unlock( &p_vout->picture_lock );
}
742

743 744 745 746 747 748 749
void vout_DisplayTitle( vout_thread_t *p_vout, const char *psz_title )
{
    assert( psz_title );

    if( !config_GetInt( p_vout, "osd" ) )
        return;

750
    vlc_mutex_lock( &p_vout->change_lock );
751 752
    free( p_vout->p->psz_title );
    p_vout->p->psz_title = strdup( psz_title );
753
    vlc_mutex_unlock( &p_vout->change_lock );
754
}
755

756 757 758 759 760
spu_t *vout_GetSpu( vout_thread_t *p_vout )
{
    return p_vout->p_spu;
}

761
/*****************************************************************************
Sam Hocevar's avatar
 
Sam Hocevar committed
762
 * InitThread: initialize video output thread
763
 *****************************************************************************
Sam Hocevar's avatar
 
Sam Hocevar committed
764 765 766
 * 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.
767
 * XXX You have to enter it with change_lock taken.
768
 *****************************************************************************/
769 770
static int ChromaCreate( vout_thread_t *p_vout );
static void ChromaDestroy( vout_thread_t *p_vout );
771

772 773 774 775 776
static bool ChromaIsEqual( const picture_heap_t *p_output, const picture_heap_t *p_render )
{
     if( !vout_ChromaCmp( p_output->i_chroma, p_render->i_chroma ) )
         return false;

777 778 779 780
     if( p_output->i_chroma != VLC_CODEC_RGB15 &&
         p_output->i_chroma != VLC_CODEC_RGB16 &&
         p_output->i_chroma != VLC_CODEC_RGB24 &&
         p_output->i_chroma != VLC_CODEC_RGB32 )
781 782 783 784 785 786 787
         return true;

     return p_output->i_rmask == p_render->i_rmask &&
            p_output->i_gmask == p_render->i_gmask &&
            p_output->i_bmask == p_render->i_bmask;
}

Sam Hocevar's avatar
 
Sam Hocevar committed
788
static int InitThread( vout_thread_t *p_vout )
Vincent Seguin's avatar
Vincent Seguin committed
789
{
790
    int i;
Sam Hocevar's avatar
 
Sam Hocevar committed
791

Sam Hocevar's avatar
 
Sam Hocevar committed
792
    /* Initialize output method, it allocates direct buffers for us */
Sam Hocevar's avatar
 
Sam Hocevar committed
793
    if( p_vout->pf_init( p_vout ) )
794
        return VLC_EGENERIC;
Sam Hocevar's avatar
 
Sam Hocevar committed
795

796 797
    p_vout->p->p_picture_displayed = NULL;

Sam Hocevar's avatar
 
Sam Hocevar committed
798
    if( !I_OUTPUTPICTURES )
Sam Hocevar's avatar
 
Sam Hocevar committed
799
    {
800 801
        msg_Err( p_vout, "plugin was unable to allocate at least "
                         "one direct buffer" );
Sam Hocevar's avatar
 
Sam Hocevar committed
802
        p_vout->pf_end( p_vout );
803
        return VLC_EGENERIC;
804
    }
Vincent Seguin's avatar
Vincent Seguin committed
805

Gildas Bazin's avatar
 
Gildas Bazin committed
806 807 808 809 810 811 812 813
    if( I_OUTPUTPICTURES > VOUT_MAX_PICTURES )
    {
        msg_Err( p_vout, "plugin allocated too many direct buffers, "
                         "our internal buffers must have overflown." );
        p_vout->pf_end( p_vout );
        return VLC_EGENERIC;
    }

814
    msg_Dbg( p_vout, "got %i direct buffer(s)", I_OUTPUTPICTURES );
Sam Hocevar's avatar
 
Sam Hocevar committed
815

816 817 818 819 820 821 822 823 824 825 826 827
    if( !p_vout->fmt_out.i_width || !p_vout->fmt_out.i_height )
    {
        p_vout->fmt_out.i_width = p_vout->fmt_out.i_visible_width =
            p_vout->output.i_width;
        p_vout->fmt_out.i_height = p_vout->fmt_out.i_visible_height =
            p_vout->output.i_height;
        p_vout->fmt_out.i_x_offset =  p_vout->fmt_out.i_y_offset = 0;

        p_vout->fmt_out.i_chroma = p_vout->output.i_chroma;
    }
    if( !p_vout->fmt_out.i_sar_num || !p_vout->fmt_out.i_sar_num )
    {
828
        p_vout->fmt_out.i_sar_num = p_vout->output.i_aspect *
829 830 831 832 833
            p_vout->fmt_out.i_height;
        p_vout->fmt_out.i_sar_den = VOUT_ASPECT_FACTOR *
            p_vout->fmt_out.i_width;
    }

Rémi Denis-Courmont's avatar
Rémi Denis-Courmont committed
834 835
    vlc_ureduce( &p_vout->fmt_out.i_sar_num, &p_vout->fmt_out.i_sar_den,
                 p_vout->fmt_out.i_sar_num, p_vout->fmt_out.i_sar_den, 0 );
836

837
    /* FIXME removed the need of both fmt_* and heap infos */
Sam Hocevar's avatar
 
Sam Hocevar committed
838
    /* Calculate shifts from system-updated masks */
839 840 841 842 843
    PictureHeapFixRgb( &p_vout->render );
    VideoFormatImportRgb( &p_vout->fmt_render, &p_vout->render );

    PictureHeapFixRgb( &p_vout->output );
    VideoFormatImportRgb( &p_vout->fmt_out, &p_vout->output );
Sam Hocevar's avatar
 
Sam Hocevar committed
844

basos g's avatar
basos g committed
845 846
    /* print some usefull debug info about different vout formats
     */
847
    msg_Dbg( p_vout, "pic render sz %ix%i, of (%i,%i), vsz %ix%i, 4cc %4.4s, sar %i:%i, msk r0x%x g0x%x b0x%x",
basos g's avatar
basos g committed
848 849 850 851 852 853 854 855
             p_vout->fmt_render.i_width, p_vout->fmt_render.i_height,
             p_vout->fmt_render.i_x_offset, p_vout->fmt_render.i_y_offset,
             p_vout->fmt_render.i_visible_width,
             p_vout->fmt_render.i_visible_height,
             (char*)&p_vout->fmt_render.i_chroma,
             p_vout->fmt_render.i_sar_num, p_vout->fmt_render.i_sar_den,
             p_vout->fmt_render.i_rmask, p_vout->fmt_render.i_gmask, p_vout->fmt_render.i_bmask );

856
    msg_Dbg( p_vout, "pic in sz %ix%i, of (%i,%i), vsz %ix%i, 4cc %4.4s, sar %i:%i, msk r0x%x g0x%x b0x%x",
basos g's avatar
basos g committed
857 858 859 860 861 862 863 864
             p_vout->fmt_in.i_width, p_vout->fmt_in.i_height,
             p_vout->fmt_in.i_x_offset, p_vout->fmt_in.i_y_offset,
             p_vout->fmt_in.i_visible_width,
             p_vout->fmt_in.i_visible_height,
             (char*)&p_vout->fmt_in.i_chroma,
             p_vout->fmt_in.i_sar_num, p_vout->fmt_in.i_sar_den,
             p_vout->fmt_in.i_rmask, p_vout->fmt_in.i_gmask, p_vout->fmt_in.i_bmask );

865
    msg_Dbg( p_vout, "pic out sz %ix%i, of (%i,%i), vsz %ix%i, 4cc %4.4s, sar %i:%i, msk r0x%x g0x%x b0x%x",
basos g's avatar
basos g committed
866 867 868 869 870 871 872 873
             p_vout->fmt_out.i_width, p_vout->fmt_out.i_height,
             p_vout->fmt_out.i_x_offset, p_vout->fmt_out.i_y_offset,
             p_vout->fmt_out.i_visible_width,
             p_vout->fmt_out.i_visible_height,
             (char*)&p_vout->fmt_out.i_chroma,
             p_vout->fmt_out.i_sar_num, p_vout->fmt_out.i_sar_den,
             p_vout->fmt_out.i_rmask, p_vout->fmt_out.i_gmask, p_vout->fmt_out.i_bmask );

Sam Hocevar's avatar
 
Sam Hocevar committed
874
    /* Check whether we managed to create direct buffers similar to
875
     * the render buffers, ie same size and chroma */
Sam Hocevar's avatar
 
Sam Hocevar committed
876 877
    if( ( p_vout->output.i_width == p_vout->render.i_width )
     && ( p_vout->output.i_height == p_vout->render.i_height )
878
     && ( ChromaIsEqual( &p_vout->output, &p_vout->render ) ) )
Sam Hocevar's avatar
 
Sam Hocevar committed
879
    {
Sam Hocevar's avatar
 
Sam Hocevar committed
880 881 882 883
        /* Cool ! We have direct buffers, we can ask the decoder to
         * directly decode into them ! Map the first render buffers to
         * the first direct buffers, but keep the first direct buffer
         * for memcpy operations */
884
        p_vout->p->b_direct = true;
Sam Hocevar's avatar
 
Sam Hocevar committed
885

Sam Hocevar's avatar
 
Sam Hocevar committed
886 887
        for( i = 1; i < VOUT_MAX_PICTURES; i++ )
        {
Gildas Bazin's avatar
 
Gildas Bazin committed
888 889 890 891 892 893 894 895
            if( p_vout->p_picture[ i ].i_type != DIRECT_PICTURE &&
                I_RENDERPICTURES >= VOUT_MIN_DIRECT_PICTURES - 1 &&
                p_vout->p_picture[ i - 1 ].i_type == DIRECT_PICTURE )
            {
                /* We have enough direct buffers so there's no need to
                 * try to use system memory buffers. */
                break;
            }
Sam Hocevar's avatar
 
Sam Hocevar committed
896 897 898
            PP_RENDERPICTURE[ I_RENDERPICTURES ] = &p_vout->p_picture[ i ];
            I_RENDERPICTURES++;
        }
Gildas Bazin's avatar
 
Gildas Bazin committed
899 900 901 902

        msg_Dbg( p_vout, "direct render, mapping "
                 "render pictures 0-%i to system pictures 1-%i",
                 VOUT_MAX_PICTURES - 2, VOUT_MAX_PICTURES - 1 );
Sam Hocevar's avatar
 
Sam Hocevar committed
903 904 905
    }
    else
    {
Sam Hocevar's avatar
 
Sam Hocevar committed
906 907 908
        /* Rats... Something is wrong here, we could not find an output
         * plugin able to directly render what we decode. See if we can
         * find a chroma plugin to do the conversion */
909
        p_vout->p->b_direct = false;
Sam Hocevar's avatar
 
Sam Hocevar committed
910

911
        if( ChromaCreate( p_vout ) )
Sam Hocevar's avatar
 
Sam Hocevar committed
912 913
        {
            p_vout->pf_end( p_vout );
914
            return VLC_EGENERIC;
Sam Hocevar's avatar
 
Sam Hocevar committed
915
        }
Sam Hocevar's avatar
 
Sam Hocevar committed
916

Gildas Bazin's avatar
 
Gildas Bazin committed
917 918 919 920
        msg_Dbg( p_vout, "indirect render, mapping "
                 "render pictures 0-%i to system pictures %i-%i",
                 VOUT_MAX_PICTURES - 1, I_OUTPUTPICTURES,
                 I_OUTPUTPICTURES + VOUT_MAX_PICTURES - 1 );
Sam Hocevar's avatar
 
Sam Hocevar committed
921 922

        /* Append render buffers after the direct buffers */
Sam Hocevar's avatar
 
Sam Hocevar committed
923
        for( i = I_OUTPUTPICTURES; i < 2 * VOUT_MAX_PICTURES; i++ )
Sam Hocevar's avatar
 
Sam Hocevar committed
924 925 926
        {
            PP_RENDERPICTURE[ I_RENDERPICTURES ] = &p_vout->p_picture[ i ];
            I_RENDERPICTURES++;
Gildas Bazin's avatar
 
Gildas Bazin committed
927 928 929 930

            /* Check if we have enough render pictures */
            if( I_RENDERPICTURES == VOUT_MAX_PICTURES )
                break;
Sam Hocevar's avatar
 
Sam Hocevar committed
931 932
        }
    }
Vincent Seguin's avatar
Vincent Seguin committed
933

934
    return VLC_SUCCESS;
Vincent Seguin's avatar
Vincent Seguin committed
935 936
}

937
/*****************************************************************************
Sam Hocevar's avatar
 
Sam Hocevar committed
938
 * RunThread: video output thread
939
 *****************************************************************************
Sam Hocevar's avatar
 
Sam Hocevar committed
940 941 942
 * 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.
943
 *****************************************************************************/
944
static void* RunThread( void *p_this )
Vincent Seguin's avatar
Vincent Seguin committed
945
{
946
    vout_thread_t *p_vout = p_this;
947
    int             i_idle_loops = 0;  /* loops without displaying a picture */
948
    int             i_picture_qtype_last = QTYPE_NONE;
949 950
    bool            b_picture_interlaced_last = false;
    mtime_t         i_picture_interlaced_last_date;
Sam Hocevar's avatar
 
Sam Hocevar committed
951 952 953 954

    /*
     * Initialize thread
     */
955 956 957 958
    p_vout->p_module = module_need( p_vout,
                                    p_vout->p->psz_module_type,
                                    p_vout->p->psz_module_name,
                                    !strcmp(p_vout->p->psz_module_type, "video filter") );
959 960 961

    vlc_mutex_lock( &p_vout->change_lock );

962 963 964 965
    if( p_vout->p_module )
        p_vout->b_error = InitThread( p_vout );
    else
        p_vout->b_error = true;
966

Gildas Bazin's avatar
 
Gildas Bazin committed
967
    /* signal the creation of the vout */
968 969
    p_vout->p->b_ready = true;
    vlc_cond_signal( &p_vout->p->change_wait );
Gildas Bazin's avatar
 
Gildas Bazin committed
970

Sam Hocevar's avatar
 
Sam Hocevar committed
971
    if( p_vout->b_error )
972 973 974 975
        goto exit_thread;

    /* */
    const bool b_drop_late = var_CreateGetBool( p_vout, "drop-late-frames" );
976
    i_picture_interlaced_last_date = mdate();
977

978
    /*
Sam Hocevar's avatar
Sam Hocevar committed
979
     * Main loop - it is not executed if an error occurred during
Sam Hocevar's avatar
 
Sam Hocevar committed
980
     * initialization
981
     */
982
    while( !p_vout->p->b_done && !p_vout->b_error )
983
    {
Sam Hocevar's avatar
 
Sam Hocevar committed
984
        /* Initialize loop variables */
Laurent Aimar's avatar
Laurent Aimar committed
985
        const mtime_t current_date = mdate();
986
        picture_t *p_picture;
987
        picture_t *p_filtered_picture;
988
        mtime_t display_date;
Laurent Aimar's avatar
Laurent Aimar committed
989 990
        picture_t *p_directbuffer;
        int i_index;
Sam Hocevar's avatar
 
Sam Hocevar committed
991

992 993 994
        if( p_vout->p->b_title_show && p_vout->p->psz_title )
            DisplayTitleOnOSD( p_vout );

995 996
        vlc_mutex_lock( &p_vout->picture_lock );

997 998 999 1000
        /* Look for the earliest picture but after the last displayed one */
        picture_t *p_last = p_vout->p->p_picture_displayed;;

        p_picture = NULL;
1001
        for( i_index = 0; i_index < I_RENDERPICTURES; i_index++ )
Sam Hocevar's avatar
 
Sam Hocevar committed
1002
        {
Laurent Aimar's avatar
Laurent Aimar committed
1003 1004
            picture_t *p_pic = PP_RENDERPICTURE[i_index];

1005 1006 1007 1008 1009 1010 1011
            if( p_pic->i_status != READY_PICTURE )
                continue;

            if( p_vout->p->b_paused && p_last && p_last->date > 1 )
                continue;

            if( p_last && p_pic != p_last && p_pic->date <= p_last->date )
1012
            {
1013
                /* Drop old picture */
1014 1015 1016 1017 1018 1019 1020 1021 1022
                vout_UsePictureLocked( p_vout, p_pic );