video_output.c 68.9 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

Clément Stenac's avatar
Clément Stenac committed
41
#include <vlc_vout.h>
42

Clément Stenac's avatar
Clément Stenac committed
43 44
#include <vlc_filter.h>
#include <vlc_osd.h>
Laurent Aimar's avatar
Laurent Aimar committed
45
#include <assert.h>
46

47
#if defined( __APPLE__ )
Clément Stenac's avatar
Clément Stenac committed
48
/* Include darwin_specific.h here if needed */
49 50
#endif

Clément Stenac's avatar
Clément Stenac committed
51 52
/** FIXME This is quite ugly but needed while we don't have counters
 * helpers */
53
//#include "input/input_internal.h"
Clément Stenac's avatar
Clément Stenac committed
54

55 56
#include <libvlc.h>
#include <vlc_input.h>
57
#include "vout_pictures.h"
Laurent Aimar's avatar
Laurent Aimar committed
58
#include "vout_internal.h"
59

60
/*****************************************************************************
Michel Kaempf's avatar
Michel Kaempf committed
61
 * Local prototypes
62
 *****************************************************************************/
63
static int      InitThread        ( vout_thread_t * );
64
static void*    RunThread         ( void *  );
65
static void     ErrorThread       ( vout_thread_t * );
Laurent Aimar's avatar
Laurent Aimar committed
66
static void     CleanThread       ( vout_thread_t * );
67
static void     EndThread         ( vout_thread_t * );
Sam Hocevar's avatar
 
Sam Hocevar committed
68

Laurent Aimar's avatar
Laurent Aimar committed
69
static void VideoFormatImportRgb( video_format_t *, const picture_heap_t * );
70 71
static void PictureHeapFixRgb( picture_heap_t * );

72 73
static void     vout_Destructor   ( vlc_object_t * p_this );

Gildas Bazin's avatar
 
Gildas Bazin committed
74
/* Object variables callbacks */
75 76
static int FilterCallback( vlc_object_t *, char const *,
                           vlc_value_t, vlc_value_t, void * );
77 78
static int VideoFilter2Callback( vlc_object_t *, char const *,
                                 vlc_value_t, vlc_value_t, void * );
79 80

/* */
81 82 83 84 85
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 * );
86 87
/* */
static void DeinterlaceEnable( vout_thread_t * );
88
static void DeinterlaceNeeded( vout_thread_t *, bool );
Gildas Bazin's avatar
 
Gildas Bazin committed
89

90 91 92
/* From vout_intf.c */
int vout_Snapshot( vout_thread_t *, picture_t * );

93 94 95
/* Display media title in OSD */
static void DisplayTitleOnOSD( vout_thread_t *p_vout );

96 97
/* Time during which the thread will sleep if it has nothing to
 * display (in micro-seconds) */
98
#define VOUT_IDLE_SLEEP                 ((int)(0.020*CLOCK_FREQ))
99 100 101 102 103 104 105 106 107 108 109

/* 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))

110 111 112 113 114 115 116 117
/* 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

118 119 120 121 122
/*****************************************************************************
 * Video Filter2 functions
 *****************************************************************************/
static picture_t *video_new_buffer_filter( filter_t *p_filter )
{
123
    vout_thread_t *p_vout = (vout_thread_t*)p_filter->p_owner;
124
    picture_t *p_picture = vout_CreatePicture( p_vout, 0, 0, 0 );
125

126
    p_picture->i_status = READY_PICTURE;
127 128 129 130 131 132

    return p_picture;
}

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

135 136 137
    vlc_mutex_lock( &p_vout->picture_lock );
    vout_UsePictureLocked( p_vout, p_pic );
    vlc_mutex_unlock( &p_vout->picture_lock );
138 139 140 141
}

static int video_filter_buffer_allocation_init( filter_t *p_filter, void *p_data )
{
142 143
    p_filter->pf_video_buffer_new = video_new_buffer_filter;
    p_filter->pf_video_buffer_del = video_del_buffer_filter;
144 145
    p_filter->p_owner = p_data; /* p_vout */
    return VLC_SUCCESS;
146 147
}

148
/*****************************************************************************
149 150 151 152 153
 * 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.
 *****************************************************************************/
154 155
vout_thread_t *__vout_Request( vlc_object_t *p_this, vout_thread_t *p_vout,
                               video_format_t *p_fmt )
156
{
157
    if( !p_fmt )
158
    {
159 160 161
        /* Video output is no longer used.
         * TODO: support for reusing video outputs with proper _thread-safe_
         * reference handling. */
162
        if( p_vout )
163
            vout_CloseAndRelease( p_vout );
164 165 166 167 168 169
        return NULL;
    }

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

173
    /* TODO: find a suitable unused video output */
174 175 176 177

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

180
        /* We don't directly check for the "vout-filter" variable for obvious
Gildas Bazin's avatar
 
Gildas Bazin committed
181
         * performance reasons. */
182
        if( p_vout->p->b_filter_change )
Gildas Bazin's avatar
 
Gildas Bazin committed
183
        {
184
            char *psz_filter_chain = var_GetString( p_vout, "vout-filter" );
Gildas Bazin's avatar
 
Gildas Bazin committed
185

186
            if( psz_filter_chain && !*psz_filter_chain )
Gildas Bazin's avatar
 
Gildas Bazin committed
187 188 189 190
            {
                free( psz_filter_chain );
                psz_filter_chain = NULL;
            }
191
            if( p_vout->p->psz_filter_chain && !*p_vout->p->psz_filter_chain )
Gildas Bazin's avatar
 
Gildas Bazin committed
192
            {
193 194
                free( p_vout->p->psz_filter_chain );
                p_vout->p->psz_filter_chain = NULL;
Gildas Bazin's avatar
 
Gildas Bazin committed
195 196
            }

197
            if( !psz_filter_chain && !p_vout->p->psz_filter_chain )
Gildas Bazin's avatar
 
Gildas Bazin committed
198
            {
199
                p_vout->p->b_filter_change = false;
Gildas Bazin's avatar
 
Gildas Bazin committed
200 201
            }

202
            free( psz_filter_chain );
Gildas Bazin's avatar
 
Gildas Bazin committed
203 204
        }

205
        if( p_vout->fmt_render.i_chroma != vlc_fourcc_GetCodec( VIDEO_ES, p_fmt->i_chroma ) ||
206 207
            p_vout->fmt_render.i_width != p_fmt->i_width ||
            p_vout->fmt_render.i_height != p_fmt->i_height ||
208
            p_vout->p->b_filter_change )
209
        {
210 211
            vlc_mutex_unlock( &p_vout->change_lock );

212
            /* We are not interested in this format, close this vout */
213
            vout_CloseAndRelease( p_vout );
214
            vlc_object_release( p_vout );
215 216 217 218 219
            p_vout = NULL;
        }
        else
        {
            /* This video output is cool! Hijack it. */
220 221 222 223 224 225
            /* 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 );
226
#if 0
227 228 229 230 231 232
            /* 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;
            }
233 234
#endif

235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250
            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;
251 252 253 254 255 256 257 258 259 260
            }
            vlc_mutex_unlock( &p_vout->change_lock );

            vlc_object_release( p_vout );
        }

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

261
            spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), false );
262
            vlc_object_detach( p_vout );
263

264
            vlc_object_attach( p_vout, p_this );
265
            spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), true );
266 267 268 269 270 271 272
        }
    }

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

273
        p_vout = vout_Create( p_this, p_fmt );
274 275 276 277 278 279 280
    }

    return p_vout;
}

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

291 292
    unsigned int i_width = p_fmt->i_width;
    unsigned int i_height = p_fmt->i_height;
293
    vlc_fourcc_t i_chroma = vlc_fourcc_GetCodec( VIDEO_ES, p_fmt->i_chroma );
294

295 296 297 298
    config_chain_t *p_cfg;
    char *psz_parser;
    char *psz_name;

299
    if( i_width <= 0 || i_height <= 0 )
300 301 302 303 304 305
        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;
306 307 308 309 310
    unsigned int i_aspect = (int64_t)p_fmt->i_sar_num *
                                     i_width *
                                     VOUT_ASPECT_FACTOR /
                                     p_fmt->i_sar_den /
                                     i_height;
311

312
    /* Allocate descriptor */
313 314 315
    static const char typename[] = "video output";
    p_vout = vlc_custom_create( p_parent, sizeof( *p_vout ), VLC_OBJECT_VOUT,
                                typename );
316
    if( p_vout == NULL )
317
        return NULL;
318

319
    /* */
Laurent Aimar's avatar
Laurent Aimar committed
320
    p_vout->p = calloc( 1, sizeof(*p_vout->p) );
321 322 323 324 325 326
    if( !p_vout->p )
    {
        vlc_object_release( p_vout );
        return NULL;
    }

327
    /* Initialize pictures - translation tables and functions
Gildas Bazin's avatar
 
Gildas Bazin committed
328
     * will be initialized later in InitThread */
329
    for( i_index = 0; i_index < 2 * VOUT_MAX_PICTURES + 1; i_index++)
Gildas Bazin's avatar
 
Gildas Bazin committed
330 331 332 333 334
    {
        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;
335
        p_vout->p_picture[i_index].b_slow   = 0;
Gildas Bazin's avatar
 
Gildas Bazin committed
336 337 338 339 340 341 342
    }

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

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

344 345
    p_vout->fmt_render        = *p_fmt;   /* FIXME palette */
    p_vout->fmt_in            = *p_fmt;   /* FIXME palette */
346

Gildas Bazin's avatar
 
Gildas Bazin committed
347 348 349 350 351
    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;

352 353 354
    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
355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371

    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;
372 373
    p_vout->b_autoscale  = 1;
    p_vout->i_zoom      = ZOOM_FP_FACTOR;
Gildas Bazin's avatar
 
Gildas Bazin committed
374
    p_vout->b_fullscreen = 0;
Gildas Bazin's avatar
 
Gildas Bazin committed
375
    p_vout->i_alignment  = 0;
376 377
    p_vout->p->render_time  = 10;
    p_vout->p->c_fps_samples = 0;
378
    vout_statistic_Init( &p_vout->p->statistic );
379
    p_vout->p->b_filter_change = 0;
Laurent Aimar's avatar
Laurent Aimar committed
380 381
    p_vout->p->b_paused = false;
    p_vout->p->i_pause_date = 0;
382
    p_vout->pf_control = NULL;
383 384
    p_vout->p->i_par_num =
    p_vout->p->i_par_den = 1;
385
    p_vout->p->p_picture_displayed = NULL;
386 387 388
    p_vout->p->i_picture_displayed_date = 0;
    p_vout->p->b_picture_displayed = false;
    p_vout->p->b_picture_empty = false;
389
    p_vout->p->i_picture_qtype = QTYPE_NONE;
390
    p_vout->p->b_picture_interlaced = false;
Gildas Bazin's avatar
 
Gildas Bazin committed
391

392 393
    vlc_mouse_Init( &p_vout->p->mouse );

394
    vout_snapshot_Init( &p_vout->p->snapshot );
395

Gildas Bazin's avatar
 
Gildas Bazin committed
396
    /* Initialize locks */
397 398
    vlc_mutex_init( &p_vout->picture_lock );
    vlc_cond_init( &p_vout->p->picture_wait );
399
    vlc_mutex_init( &p_vout->change_lock );
400
    vlc_mutex_init( &p_vout->p->vfilter_lock );
Gildas Bazin's avatar
 
Gildas Bazin committed
401

402 403 404 405 406
    /* 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 );
407
    var_Create( p_vout, "mouse-clicked", VLC_VAR_BOOL );
408

409
    /* Initialize subpicture unit */
410
    p_vout->p_spu = spu_Create( p_vout );
411

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

415
    /* */
416 417
    spu_Init( p_vout->p_spu );

418 419
    spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), true );

420 421
    /* Take care of some "interface/control" related initialisations */
    vout_IntfInit( p_vout );
Gildas Bazin's avatar
 
Gildas Bazin committed
422

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

        /* Apply video filter2 objects on the first vout */
432
        p_vout->p->psz_vf2 =
433
            var_CreateGetStringCommand( p_vout, "video-filter" );
434 435

        p_vout->p->b_first_vout = true;
Gildas Bazin's avatar
 
Gildas Bazin committed
436 437 438 439
    }
    else
    {
        /* continue the parent's filter chain */
440
        char *psz_tmp;
Gildas Bazin's avatar
 
Gildas Bazin committed
441

442
        /* Ugly hack to jump to our configuration chain */
443 444 445 446
        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 );
447 448
        config_ChainDestroy( p_cfg );
        free( psz_tmp );
449 450

        /* Create a video filter2 var ... but don't inherit values */
451 452
        var_Create( p_vout, "video-filter",
                    VLC_VAR_STRING | VLC_VAR_ISCOMMAND );
453
        p_vout->p->psz_vf2 = var_GetString( p_vout, "video-filter" );
454 455 456

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

459
    var_AddCallback( p_vout, "video-filter", VideoFilter2Callback, NULL );
460
    p_vout->p->p_vf2_chain = filter_chain_New( p_vout, "video filter2",
461
        false, video_filter_buffer_allocation_init, NULL, p_vout );
462

Gildas Bazin's avatar
 
Gildas Bazin committed
463
    /* Choose the video output module */
464
    if( !p_vout->p->psz_filter_chain || !*p_vout->p->psz_filter_chain )
Gildas Bazin's avatar
 
Gildas Bazin committed
465
    {
466
        psz_parser = NULL;
Gildas Bazin's avatar
 
Gildas Bazin committed
467 468 469
    }
    else
    {
470 471
        psz_parser = strdup( p_vout->p->psz_filter_chain );
        p_vout->p->b_title_show = false;
Sam Hocevar's avatar
 
Sam Hocevar committed
472 473
    }

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

    /* 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 );

489 490 491 492 493 494
    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
495 496

    /* */
497
    vlc_object_set_destructor( p_vout, vout_Destructor );
Gildas Bazin's avatar
 
Gildas Bazin committed
498

499
    /* */
500 501 502
    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
503
    {
504
        spu_Attach( p_vout->p_spu, VLC_OBJECT(p_vout), false );
505 506
        spu_Destroy( p_vout->p_spu );
        p_vout->p_spu = NULL;
507
        vlc_object_release( p_vout );
508 509 510
        return NULL;
    }

511 512 513 514 515 516 517 518 519
    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 );

520 521 522
    if( p_vout->b_error )
    {
        msg_Err( p_vout, "video output creation failed" );
523
        vout_CloseAndRelease( p_vout );
524
        return NULL;
525
    }
Michel Kaempf's avatar
Michel Kaempf committed
526

527
    return p_vout;
Michel Kaempf's avatar
Michel Kaempf committed
528 529
}

530
/*****************************************************************************
531
 * vout_Close: Close a vout created by vout_Create.
532
 *****************************************************************************
533
 * 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
534
 * You should NEVER call it on vout not obtained through vout_Create
535
 * (like with vout_Request or vlc_object_find.)
536
 * You can use vout_CloseAndRelease() as a convenience method.
537
 *****************************************************************************/
538
void vout_Close( vout_thread_t *p_vout )
539 540 541
{
    assert( p_vout );

542 543 544 545
    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 );
546

547
    vout_snapshot_End( &p_vout->p->snapshot );
548

549
    vlc_join( p_vout->p->thread, NULL );
550 551 552
}

/* */
553 554
static void vout_Destructor( vlc_object_t * p_this )
{
555
    vout_thread_t *p_vout = (vout_thread_t *)p_this;
556

557 558 559
    /* Make sure the vout was stopped first */
    assert( !p_vout->p_module );

560 561
    free( p_vout->p->psz_module_name );

Laurent Aimar's avatar
Laurent Aimar committed
562
    /* */
563 564
    if( p_vout->p_spu )
        spu_Destroy( p_vout->p_spu );
Laurent Aimar's avatar
Laurent Aimar committed
565

566
    /* Destroy the locks */
567
    vlc_cond_destroy( &p_vout->p->change_wait );
568
    vlc_cond_destroy( &p_vout->p->picture_wait );
569 570
    vlc_mutex_destroy( &p_vout->picture_lock );
    vlc_mutex_destroy( &p_vout->change_lock );
571
    vlc_mutex_destroy( &p_vout->p->vfilter_lock );
572

573 574 575
    /* */
    vout_statistic_Clean( &p_vout->p->statistic );

576
    /* */
577
    vout_snapshot_Clean( &p_vout->p->snapshot );
578 579

    /* */
580
    free( p_vout->p->psz_filter_chain );
581
    free( p_vout->p->psz_title );
Gildas Bazin's avatar
 
Gildas Bazin committed
582

583 584
    config_ChainDestroy( p_vout->p_cfg );

585 586
    free( p_vout->p );

Michel Kaempf's avatar
Michel Kaempf committed
587 588
}

Laurent Aimar's avatar
Laurent Aimar committed
589 590 591
/* */
void vout_ChangePause( vout_thread_t *p_vout, bool b_paused, mtime_t i_date )
{
592
    vlc_mutex_lock( &p_vout->change_lock );
Laurent Aimar's avatar
Laurent Aimar committed
593

594
    assert( !p_vout->p->b_paused || !b_paused );
595 596 597 598 599

    vlc_mutex_lock( &p_vout->picture_lock );

    p_vout->p->i_picture_displayed_date = 0;

Laurent Aimar's avatar
Laurent Aimar committed
600 601 602 603 604 605 606 607 608 609 610
    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;
        }
611 612 613
        vlc_cond_signal( &p_vout->p->picture_wait );
        vlc_mutex_unlock( &p_vout->picture_lock );

Laurent Aimar's avatar
Laurent Aimar committed
614
        spu_OffsetSubtitleDate( p_vout->p_spu, i_duration );
Laurent Aimar's avatar
Laurent Aimar committed
615
    }
616 617 618 619
    else
    {
        vlc_mutex_unlock( &p_vout->picture_lock );
    }
Laurent Aimar's avatar
Laurent Aimar committed
620 621 622
    p_vout->p->b_paused = b_paused;
    p_vout->p->i_pause_date = i_date;

623
    vlc_mutex_unlock( &p_vout->change_lock );
Laurent Aimar's avatar
Laurent Aimar committed
624
}
625

626 627
void vout_GetResetStatistic( vout_thread_t *p_vout, int *pi_displayed, int *pi_lost )
{
628 629
    vout_statistic_GetReset( &p_vout->p->statistic,
                             pi_displayed, pi_lost );
630
}
631

Laurent Aimar's avatar
Laurent Aimar committed
632 633 634
void vout_Flush( vout_thread_t *p_vout, mtime_t i_date )
{
    vlc_mutex_lock( &p_vout->picture_lock );
635
    p_vout->p->i_picture_displayed_date = 0;
Laurent Aimar's avatar
Laurent Aimar committed
636 637 638
    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
639

Laurent Aimar's avatar
Laurent Aimar committed
640 641 642 643 644 645 646 647 648
        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;
        }
    }
649
    vlc_cond_signal( &p_vout->p->picture_wait );
Laurent Aimar's avatar
Laurent Aimar committed
650 651
    vlc_mutex_unlock( &p_vout->picture_lock );
}
652

653
void vout_FixLeaks( vout_thread_t *p_vout, bool b_forced )
654 655 656 657 658
{
    int i_pic, i_ready_pic;

    vlc_mutex_lock( &p_vout->picture_lock );

659
    for( i_pic = 0, i_ready_pic = 0; i_pic < p_vout->render.i_pictures && !b_forced; i_pic++ )
660 661 662 663 664 665 666 667 668 669 670 671 672 673 674 675 676 677 678 679 680 681
    {
        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;
        }
    }
682
    if( i_pic < p_vout->render.i_pictures && !b_forced )
683 684 685 686 687 688 689
    {
        vlc_mutex_unlock( &p_vout->picture_lock );
        return;
    }

    /* Too many pictures are still referenced, there is probably a bug
     * with the decoder */
690 691
    if( !b_forced )
        msg_Err( p_vout, "pictures leaked, resetting the heap" );
692 693 694 695 696

    /* 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];
697 698

        msg_Dbg( p_vout, "[%d] %d %d", i_pic, p_pic->i_status, p_pic->i_refcount );
699
        p_pic->i_refcount = 0;
Laurent Aimar's avatar
Laurent Aimar committed
700 701 702 703 704 705 706 707 708 709

        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;
        }
710
    }
711
    vlc_cond_signal( &p_vout->p->picture_wait );
712 713
    vlc_mutex_unlock( &p_vout->picture_lock );
}
714 715 716 717 718 719 720 721 722 723 724 725 726 727 728 729 730 731 732 733 734 735 736
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 );
}
737

738 739 740 741
void vout_DisplayTitle( vout_thread_t *p_vout, const char *psz_title )
{
    assert( psz_title );

742
    if( !var_InheritBool( p_vout, "osd" ) )
743 744
        return;

745
    vlc_mutex_lock( &p_vout->change_lock );
746 747
    free( p_vout->p->psz_title );
    p_vout->p->psz_title = strdup( psz_title );
748
    vlc_mutex_unlock( &p_vout->change_lock );
749
}
750

751 752 753 754 755
spu_t *vout_GetSpu( vout_thread_t *p_vout )
{
    return p_vout->p_spu;
}

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

767 768 769 770 771
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;

772 773 774 775
     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 )
776 777 778 779 780 781 782
         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
783
static int InitThread( vout_thread_t *p_vout )
Vincent Seguin's avatar
Vincent Seguin committed
784
{
785
    int i;
Sam Hocevar's avatar
 
Sam Hocevar committed
786

Sam Hocevar's avatar
 
Sam Hocevar committed
787
    /* Initialize output method, it allocates direct buffers for us */
Sam Hocevar's avatar
 
Sam Hocevar committed
788
    if( p_vout->pf_init( p_vout ) )
789
        return VLC_EGENERIC;
Sam Hocevar's avatar
 
Sam Hocevar committed
790

791 792
    p_vout->p->p_picture_displayed = NULL;

Sam Hocevar's avatar
 
Sam Hocevar committed
793
    if( !I_OUTPUTPICTURES )
Sam Hocevar's avatar
 
Sam Hocevar committed
794
    {
795 796
        msg_Err( p_vout, "plugin was unable to allocate at least "
                         "one direct buffer" );
Sam Hocevar's avatar
 
Sam Hocevar committed
797
        p_vout->pf_end( p_vout );
798
        return VLC_EGENERIC;
799
    }
Vincent Seguin's avatar
Vincent Seguin committed
800

Gildas Bazin's avatar
 
Gildas Bazin committed
801 802 803 804 805 806 807 808
    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;
    }

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

811 812 813 814 815 816 817 818 819 820 821 822
    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 )
    {
823
        p_vout->fmt_out.i_sar_num = p_vout->output.i_aspect *
824 825 826 827 828
            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
829 830
    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 );
831

832
    /* FIXME removed the need of both fmt_* and heap infos */
Sam Hocevar's avatar
 
Sam Hocevar committed
833
    /* Calculate shifts from system-updated masks */
834 835 836 837 838
    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
839

basos g's avatar
basos g committed
840 841
    /* print some usefull debug info about different vout formats
     */
842
    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
843 844 845 846 847 848 849 850
             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 );

851
    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
852 853 854 855 856 857 858 859
             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 );

860
    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
861 862 863 864 865 866 867 868
             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
869
    /* Check whether we managed to create direct buffers similar to
870
     * the render buffers, ie same size and chroma */
Sam Hocevar's avatar
 
Sam Hocevar committed
871 872
    if( ( p_vout->output.i_width == p_vout->render.i_width )
     && ( p_vout->output.i_height == p_vout->render.i_height )
873
     && ( ChromaIsEqual( &p_vout->output, &p_vout->render ) ) )
Sam Hocevar's avatar
 
Sam Hocevar committed
874
    {
Sam Hocevar's avatar
 
Sam Hocevar committed
875 876 877 878
        /* 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 */
879
        p_vout->p->b_direct = true;
Sam Hocevar's avatar
 
Sam Hocevar committed
880

Sam Hocevar's avatar
 
Sam Hocevar committed
881 882
        for( i = 1; i < VOUT_MAX_PICTURES; i++ )
        {
Gildas Bazin's avatar
 
Gildas Bazin committed
883 884 885 886 887 888 889 890
            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
891 892 893
            PP_RENDERPICTURE[ I_RENDERPICTURES ] = &p_vout->p_picture[ i ];
            I_RENDERPICTURES++;
        }
Gildas Bazin's avatar
 
Gildas Bazin committed
894 895 896 897

        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
898 899 900
    }
    else
    {
Sam Hocevar's avatar
 
Sam Hocevar committed
901 902 903
        /* 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 */
904
        p_vout->p->b_direct = false;
Sam Hocevar's avatar
 
Sam Hocevar committed
905

906
        if( ChromaCreate( p_vout ) )
Sam Hocevar's avatar
 
Sam Hocevar committed
907 908
        {
            p_vout->pf_end( p_vout );
909
            return VLC_EGENERIC;
Sam Hocevar's avatar
 
Sam Hocevar committed
910
        }
Sam Hocevar's avatar
 
Sam Hocevar committed
911

Gildas Bazin's avatar
 
Gildas Bazin committed
912 913 914 915
        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
916 917

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

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

929
    return VLC_SUCCESS;
Vincent Seguin's avatar
Vincent Seguin committed
930 931
}

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

    /*
     * Initialize thread
     */
950 951 952 953
    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") );
954 955 956

    vlc_mutex_lock( &p_vout->change_lock );

957 958 959 960
    if( p_vout->p_module )
        p_vout->b_error = InitThread( p_vout );
    else
        p_vout->b_error = true;
961

Gildas Bazin's avatar
 
Gildas Bazin committed
962
    /* signal the creation of the vout */
963 964
    p_vout->p->b_ready = true;
    vlc_cond_signal( &p_vout->p->change_wait );
Gildas Bazin's avatar
 
Gildas Bazin committed
965

Sam Hocevar's avatar
 
Sam Hocevar committed
966
    if( p_vout->b_error )
967 968 969 970
        goto exit_thread;

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

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

987 988 989
        if( p_vout->p->b_title_show && p_vout->p->psz_title )
            DisplayTitleOnOSD( p_vout );

990 991
        vlc_mutex_lock( &p_vout->picture_lock );

992 993 994 995
        /* Look for the earliest picture but after the last displayed one */
        picture_t *p_last = p_vout->p->p_picture_displayed;;

        p_picture = NULL;
996
        for( i_index = 0; i_index < I_RENDERPICTURES; i_index++ )
Sam Hocevar's avatar
 
Sam Hocevar committed
997
        {
Laurent Aimar's avatar
Laurent Aimar committed
998 999
            picture_t *p_pic = PP_RENDERPICTURE[i_index];

1000 1001 1002 1003 1004 1005 1006
            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 )
1007
            {
1008
                /* Drop old picture */
1009 1010 1011 1012 1013 1014 1015 1016 1017
                vout_UsePictureLocked( p_vout, p_pic );
            }
            else if( !p_vout->p->b_paused && !p_pic->b_force && p_pic != p_last &&
                     p_pic->date < current_date + p_vout->p->render_time &&
                     b_drop_late )