]> git.sesse.net Git - vlc/blobdiff - modules/packetizer/mpegvideo.c
Removes trailing spaces. Removes tabs.
[vlc] / modules / packetizer / mpegvideo.c
index 21f6da9da4e854a2e521b2aba47e7ca9e8a63996..704f4d08d7cad2a28c39575e7dd7c6c957eeafab 100644 (file)
@@ -1,11 +1,13 @@
 /*****************************************************************************
- * mpegvideo.c
+ * mpegvideo.c: parse and packetize an MPEG1/2 video stream
  *****************************************************************************
- * Copyright (C) 2001, 2002 VideoLAN
- * $Id: mpegvideo.c,v 1.22 2003/11/22 16:53:02 fenrir Exp $
+ * Copyright (C) 2001-2006 the VideoLAN team
+ * $Id$
  *
  * Authors: Laurent Aimar <fenrir@via.ecp.fr>
  *          Eric Petit <titer@videolan.org>
+ *          Gildas Bazin <gbazin@videolan.org>
+ *          Jean-Paul Saman <jpsaman #_at_# m2x dot nl>
  *
  * 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
@@ -19,7 +21,7 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 /*****************************************************************************
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <stdlib.h>                                      /* malloc(), free() */
 
 #include <vlc/vlc.h>
-#include <vlc/decoder.h>
-#include <vlc/input.h>
+#include <vlc_block.h>
+#include <vlc_codec.h>
+#include <vlc_block_helper.h>
+
+#define SYNC_INTRAFRAME_TEXT N_("Sync on Intra Frame")
+#define SYNC_INTRAFRAME_LONGTEXT N_("Normally the packetizer would " \
+    "sync on the next full frame. This flags instructs the packetizer " \
+    "to sync on the first Intra Frame found.")
 
 /*****************************************************************************
  * Module descriptor
@@ -52,48 +59,53 @@ static int  Open ( vlc_object_t * );
 static void Close( vlc_object_t * );
 
 vlc_module_begin();
+    set_category( CAT_SOUT );
+    set_subcategory( SUBCAT_SOUT_PACKETIZER );
     set_description( _("MPEG-I/II video packetizer") );
     set_capability( "packetizer", 50 );
     set_callbacks( Open, Close );
-vlc_module_end();
 
+    add_bool( "packetizer-mpegvideo-sync-iframe", 0, NULL, SYNC_INTRAFRAME_TEXT,
+              SYNC_INTRAFRAME_LONGTEXT, VLC_TRUE );
+vlc_module_end();
 
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
 static block_t *Packetize( decoder_t *, block_t ** );
-
-static int mpgv_FindStartCode( uint8_t **pp_start, uint8_t *p_end );
+static block_t *ParseMPEGBlock( decoder_t *, block_t * );
 
 struct decoder_sys_t
 {
-    /* sequence header and extention */
+    /*
+     * Input properties
+     */
+    block_bytestream_t bytestream;
+    int i_state;
+    int i_offset;
+    uint8_t p_startcode[3];
+
+    /* Sequence header and extension */
     block_t *p_seq;
     block_t *p_ext;
 
-    /* current frame being building */
+    /* Current frame being built */
     block_t    *p_frame;
-    vlc_bool_t b_frame_slice;
-    vlc_bool_t b_frame_corrupted;
-    vlc_bool_t b_gop;
+    block_t    **pp_last;
 
-    /* pts of current picture */
+    vlc_bool_t b_frame_slice;
     mtime_t i_pts;
     mtime_t i_dts;
 
-    /* gathering buffer */
-    int         i_buffer;
-    int         i_buffer_size;
-    uint8_t     *p_buffer;
-    uint8_t     *p_start, *p_old;
-
-    /* */
+    /* Sequence properties */
     int         i_frame_rate;
     int         i_frame_rate_base;
     vlc_bool_t  b_seq_progressive;
     vlc_bool_t  b_low_delay;
+    int         i_aspect_ratio_info;
+    vlc_bool_t  b_inited;
 
-    /* */
+    /* Picture properties */
     int i_temporal_ref;
     int i_picture_type;
     int i_picture_structure;
@@ -101,14 +113,21 @@ struct decoder_sys_t
     int i_repeat_first_field;
     int i_progressive_frame;
 
-    /* */
-    int     i_seq_old;  /* How many picture from last seq */
-
-    /* */
-
     mtime_t i_interpolated_dts;
-    mtime_t i_old_duration;
     mtime_t i_last_ref_pts;
+    vlc_bool_t b_second_field;
+
+    /* Number of pictures since last sequence header */
+    int i_seq_old;
+
+    /* Sync behaviour */
+    vlc_bool_t  b_sync_on_intra_frame;
+    vlc_bool_t  b_discontinuity;
+};
+
+enum {
+    STATE_NOSYNC,
+    STATE_NEXT_SYNC
 };
 
 /*****************************************************************************
@@ -126,26 +145,26 @@ static int Open( vlc_object_t *p_this )
         return VLC_EGENERIC;
     }
 
-    es_format_Init( &p_dec->fmt_out, VIDEO_ES, VLC_FOURCC( 'm', 'p', 'g', 'v' ) );
+    es_format_Init( &p_dec->fmt_out, VIDEO_ES, VLC_FOURCC('m','p','g','v') );
     p_dec->pf_packetize = Packetize;
 
     p_dec->p_sys = p_sys = malloc( sizeof( decoder_sys_t ) );
 
+    /* Misc init */
+    p_sys->i_state = STATE_NOSYNC;
+    p_sys->bytestream = block_BytestreamInit( p_dec );
+    p_sys->p_startcode[0] = 0;
+    p_sys->p_startcode[1] = 0;
+    p_sys->p_startcode[2] = 1;
+    p_sys->i_offset = 0;
+
     p_sys->p_seq = NULL;
     p_sys->p_ext = NULL;
     p_sys->p_frame = NULL;
+    p_sys->pp_last = &p_sys->p_frame;
     p_sys->b_frame_slice = VLC_FALSE;
-    p_sys->b_frame_corrupted = VLC_FALSE;
-    p_sys->b_gop = VLC_FALSE;
 
-    p_sys->i_buffer = 0;
-    p_sys->i_buffer_size = 10000;
-    p_sys->p_buffer = malloc( p_sys->i_buffer_size );
-    p_sys->p_start = p_sys->p_buffer;
-    p_sys->p_old = NULL;
-
-    p_sys->i_dts = 0;
-    p_sys->i_pts = 0;
+    p_sys->i_dts = p_sys->i_pts = 0;
 
     p_sys->i_frame_rate = 1;
     p_sys->i_frame_rate_base = 1;
@@ -159,10 +178,16 @@ static int Open( vlc_object_t *p_this )
     p_sys->i_top_field_first = 0;
     p_sys->i_repeat_first_field = 0;
     p_sys->i_progressive_frame = 0;
+    p_sys->b_inited = 0;
 
     p_sys->i_interpolated_dts = 0;
-    p_sys->i_old_duration = 0;
     p_sys->i_last_ref_pts = 0;
+    p_sys->b_second_field = 0;
+
+    p_sys->b_discontinuity = VLC_FALSE;
+    p_sys->b_sync_on_intra_frame = var_CreateGetBool( p_dec, "packetizer-mpegvideo-sync-iframe" );
+    if( p_sys->b_sync_on_intra_frame )
+        msg_Dbg( p_dec, "syncing on intra frame now" );
 
     return VLC_SUCCESS;
 }
@@ -175,6 +200,8 @@ static void Close( vlc_object_t *p_this )
     decoder_t     *p_dec = (decoder_t*)p_this;
     decoder_sys_t *p_sys = p_dec->p_sys;
 
+    block_BytestreamRelease( &p_sys->bytestream );
+
     if( p_sys->p_seq )
     {
         block_Release( p_sys->p_seq );
@@ -185,9 +212,11 @@ static void Close( vlc_object_t *p_this )
     }
     if( p_sys->p_frame )
     {
-        block_Release( p_sys->p_frame );
+        block_ChainRelease( p_sys->p_frame );
     }
-    free( p_sys->p_buffer );
+
+    var_Destroy( p_dec, "packetizer-mpegvideo-sync-iframe" );
+
     free( p_sys );
 }
 
@@ -197,321 +226,421 @@ static void Close( vlc_object_t *p_this )
 static block_t *Packetize( decoder_t *p_dec, block_t **pp_block )
 {
     decoder_sys_t *p_sys = p_dec->p_sys;
-    block_t       *p_chain_out = NULL;
-    block_t       *p_block;
+    block_t       *p_pic;
 
     if( pp_block == NULL || *pp_block == NULL )
     {
         return NULL;
     }
-    p_block = *pp_block;
-    *pp_block = NULL;
 
-    if( p_block->b_discontinuity )
+    if( (*pp_block)->i_flags&(BLOCK_FLAG_DISCONTINUITY|BLOCK_FLAG_CORRUPTED) )
     {
-        p_sys->b_frame_corrupted = VLC_TRUE;
-    }
-
-    /* Append data */
-    if( p_sys->i_buffer + p_block->i_buffer > p_sys->i_buffer_size )
-    {
-        uint8_t *p_buffer = p_sys->p_buffer;
-
-        p_sys->i_buffer_size += p_block->i_buffer + 1024;
-        p_sys->p_buffer = realloc( p_sys->p_buffer, p_sys->i_buffer_size );
-
-        if( p_sys->p_start )
-        {
-            p_sys->p_start = p_sys->p_start - p_buffer + p_sys->p_buffer;
-        }
-        if( p_sys->p_old )
+        if( (*pp_block)->i_flags&BLOCK_FLAG_CORRUPTED )
         {
-            p_sys->p_old = p_sys->p_old - p_buffer + p_sys->p_buffer;
+            p_sys->i_state = STATE_NOSYNC;
+            block_BytestreamFlush( &p_sys->bytestream );
+
+            p_sys->b_discontinuity = VLC_TRUE;
+            if( p_sys->p_frame )
+                block_ChainRelease( p_sys->p_frame );
+            p_sys->p_frame = NULL;
+            p_sys->pp_last = &p_sys->p_frame;
+            p_sys->b_frame_slice = VLC_FALSE;
         }
+//        p_sys->i_interpolated_dts =
+//        p_sys->i_last_ref_pts = 0;
 
+        block_Release( *pp_block );
+        return NULL;
     }
-    memcpy( &p_sys->p_buffer[p_sys->i_buffer], p_block->p_buffer,
-            p_block->i_buffer );
-    p_sys->i_buffer += p_block->i_buffer;
 
 
-    if( p_sys->i_buffer > 10*1000000 )
-    {
-        msg_Err( p_dec, "mmh reseting context" );
-        p_sys->i_buffer = 0;
-    }
+    block_BytestreamPush( &p_sys->bytestream, *pp_block );
 
-    /* Split data in block */
-    for( ;; )
+    while( 1 )
     {
-        if( mpgv_FindStartCode( &p_sys->p_start, &p_sys->p_buffer[p_sys->i_buffer] ) )
+        switch( p_sys->i_state )
         {
-            block_Release( p_block );
 
-            if( p_sys->p_seq == NULL )
+        case STATE_NOSYNC:
+            if( block_FindStartcodeFromOffset( &p_sys->bytestream,
+                    &p_sys->i_offset, p_sys->p_startcode, 3 ) == VLC_SUCCESS )
             {
-                block_ChainRelease( p_chain_out );
-                return NULL;
+                p_sys->i_state = STATE_NEXT_SYNC;
             }
-            return p_chain_out;
-        }
 
-        if( p_sys->p_old )
-        {
-            /* Extract the data */
-            int i_frag = p_sys->p_start - p_sys->p_old;
-            block_t *p_frag = block_New( p_dec, i_frag );
+            if( p_sys->i_offset )
+            {
+                block_SkipBytes( &p_sys->bytestream, p_sys->i_offset );
+                p_sys->i_offset = 0;
+                block_BytestreamFlush( &p_sys->bytestream );
+            }
 
-            memcpy( p_frag->p_buffer, p_sys->p_old, i_frag );
-            if( i_frag < p_sys->i_buffer )
+            if( p_sys->i_state != STATE_NEXT_SYNC )
             {
-                memmove( p_sys->p_buffer, &p_sys->p_buffer[i_frag],
-                         p_sys->i_buffer - i_frag );
+                /* Need more data */
+                return NULL;
             }
-            p_sys->i_buffer -= i_frag;
-            p_sys->p_start -= i_frag;
-            p_sys->p_old   -= i_frag;
 
-            if( p_sys->b_frame_slice && ( p_frag->p_buffer[3] == 0x00 || p_frag->p_buffer[3] > 0xaf ) )
+            p_sys->i_offset = 1; /* To find next startcode */
+
+        case STATE_NEXT_SYNC:
+            /* TODO: If p_block == NULL, flush the buffer without checking the
+             * next sync word */
+
+            /* Find the next startcode */
+            if( block_FindStartcodeFromOffset( &p_sys->bytestream,
+                    &p_sys->i_offset, p_sys->p_startcode, 3 ) != VLC_SUCCESS )
             {
-                /* We have a complete picture output it */
-                if( p_sys->p_seq == NULL )
-                {
-                    msg_Dbg( p_dec, "waiting sequence start" );
-                    block_ChainRelease( p_sys->p_frame );
-                }
-                else if( p_sys->i_dts <= 0 && p_sys->i_pts <= 0 && p_sys->i_interpolated_dts <= 0 )
-                {
-                    msg_Dbg( p_dec, "need a starting pts/dts" );
-                    block_ChainRelease( p_sys->p_frame );
-                }
-                else if( p_sys->b_frame_corrupted )
-                {
-                    msg_Warn( p_dec, "trashing a corrupted picture" );
-                    block_ChainRelease( p_sys->p_frame );
-                    p_sys->b_frame_corrupted = VLC_FALSE;
-                }
-                else
-                {
-                    block_t *p_pic = block_ChainGather( p_sys->p_frame );
-                    mtime_t i_duration = (mtime_t)( 1000000 * p_sys->i_frame_rate_base / p_sys->i_frame_rate);
-
-                    if( !p_sys->b_seq_progressive && p_sys->i_picture_structure != 0x03 )
-                    {
-                        i_duration /= 2;
-                    }
-
-                    if( p_sys->b_seq_progressive )
-                    {
-                        if( p_sys->i_top_field_first == 0 && p_sys->i_repeat_first_field == 1 )
-                        {
-                            i_duration *= 2;
-                        }
-                        else if( p_sys->i_top_field_first == 1 && p_sys->i_repeat_first_field == 1 )
-                        {
-                            i_duration *= 3;
-                        }
-                    }
-                    else
-                    {
-                        if( p_sys->i_picture_structure == 0x03 )
-                        {
-                            if( p_sys->i_progressive_frame && p_sys->i_repeat_first_field )
-                            {
-                                i_duration += i_duration / 2;
-                            }
-                        }
-                    }
-
-                    if( p_sys->b_low_delay || p_sys->i_picture_type == 0x03 )
-                    {
-                        /* Trivial case (DTS == PTS) */
-                        /* Correct interpolated dts when we receive a new pts/dts */
-                        if( p_sys->i_pts > 0 ) p_sys->i_interpolated_dts = p_sys->i_pts;
-                        if( p_sys->i_dts > 0 ) p_sys->i_interpolated_dts = p_sys->i_dts;
-                    }
-                    else
-                    {
-                        /* Correct interpolated dts when we receive a new pts/dts */
-                        if( p_sys->i_last_ref_pts > 0 )
-                            p_sys->i_interpolated_dts = p_sys->i_last_ref_pts;
-                        if( p_sys->i_dts > 0 ) p_sys->i_interpolated_dts = p_sys->i_dts;
-
-                        p_sys->i_last_ref_pts = p_sys->i_pts;
-                    }
-
-                    p_pic->i_dts    = p_sys->i_interpolated_dts;
-                    /* Set PTS only if I frame or come from stream */
-                    if( p_sys->i_pts > 0 )
-                    {
-                        p_pic->i_pts    = p_sys->i_pts;
-                    }
-                    else if( p_sys->i_picture_type == 0x03 )
-                    {
-                        p_pic->i_pts = p_pic->i_dts;
-                    }
-                    else
-                    {
-                        p_pic->i_pts = -1;
-                    }
-
-                    if( p_sys->b_low_delay || p_sys->i_picture_type == 0x03 )
-                    {
-                        /* Trivial case (DTS == PTS) */
-                        p_sys->i_interpolated_dts += i_duration;
-                    }
-                    else
-                    {
-                        p_sys->i_interpolated_dts += p_sys->i_old_duration;
-                        p_sys->i_old_duration = i_duration;
-                    }
-
-                    p_pic->i_length = p_sys->i_interpolated_dts - p_pic->i_dts;
-
-                    //msg_Dbg( p_dec, "pic: type=%d dts=%lld pts-dts=%lld", p_sys->i_picture_type, p_pic->i_dts, p_pic->i_pts - p_pic->i_dts);
-
-                    block_ChainAppend( &p_chain_out, p_pic );
+                /* Need more data */
+                return NULL;
+            }
 
-                }
+            /* Get the new fragment and set the pts/dts */
+            p_pic = block_New( p_dec, p_sys->i_offset );
+            block_BytestreamFlush( &p_sys->bytestream );
+            p_pic->i_pts = p_sys->bytestream.p_block->i_pts;
+            p_pic->i_dts = p_sys->bytestream.p_block->i_dts;
 
-                /* reset context */
-                p_sys->p_frame = NULL;
-                p_sys->b_frame_slice = VLC_FALSE;
-                p_sys->b_gop = VLC_FALSE;
-                p_sys->i_pts = 0;
-                p_sys->i_dts = 0;
+            block_GetBytes( &p_sys->bytestream, p_pic->p_buffer,
+                            p_pic->i_buffer );
+
+            /* don't reuse the same timestamps several times */
+            if( p_pic->i_buffer >= 4 && p_pic->p_buffer[3] == 0x00 )
+            {
+                /* We have a picture start code */
+                p_sys->bytestream.p_block->i_pts = 0;
+                p_sys->bytestream.p_block->i_dts = 0;
             }
 
-            if( p_frag->p_buffer[3] == 0xb8 )
+            p_sys->i_offset = 0;
+
+            /* Get picture if any */
+            if( !( p_pic = ParseMPEGBlock( p_dec, p_pic ) ) )
             {
-                if( p_sys->p_seq &&
-                    p_sys->i_seq_old > p_sys->i_frame_rate/p_sys->i_frame_rate_base )
-                {
-                    /* Usefull for mpeg1: repeat sequence header every second */
-                    block_ChainAppend( &p_sys->p_frame,
-                                       block_Duplicate( p_sys->p_seq ) );
-                    if( p_sys->p_ext )
-                    {
-                        block_ChainAppend( &p_sys->p_frame,
-                                           block_Duplicate( p_sys->p_ext ) );
-                    }
-
-                    p_sys->i_seq_old = 0;
-                }
-                p_sys->b_gop = VLC_TRUE;
+                p_sys->i_state = STATE_NOSYNC;
+                break;
             }
-            else if( p_frag->p_buffer[3] == 0xb3 )
+
+            /* If a discontinuity has been encountered, then wait till
+             * the next Intra frame before continuing with packetizing */
+            if( p_sys->b_discontinuity &&
+                p_sys->b_sync_on_intra_frame )
             {
-                static const int code_to_frame_rate[16][2] =
+                if( p_pic->i_flags & BLOCK_FLAG_TYPE_I )
                 {
-                    { 1, 1 },   /* invalid */
-                    { 24000, 1001 }, { 24, 1 }, { 25, 1 },       { 30000, 1001 },
-                    { 30, 1 },       { 50, 1 }, { 60000, 1001 }, { 60, 1 },
-                    { 1, 1 },        { 1, 1 },  { 1, 1 },        { 1, 1 },  /* invalid */
-                    { 1, 1 },        { 1, 1 },  { 1, 1 }                    /* invalid */
-                };
-
-                /* sequence header */
-                if( p_sys->p_seq )
-                {
-                    block_Release( p_sys->p_seq );
+                    msg_Dbg( p_dec, "synced on intra frame" );
+                    p_sys->b_discontinuity = VLC_FALSE;
+                    p_pic->i_flags |= BLOCK_FLAG_DISCONTINUITY;
                 }
-                if( p_sys->p_ext )
+                else
                 {
-                    block_Release( p_sys->p_ext );
-                    p_sys->p_ext = NULL;
+                    msg_Dbg( p_dec, "waiting on intra frame" );
+                    p_sys->i_state = STATE_NOSYNC;
+                    block_Release( p_pic );
+                    break;
                 }
-                p_sys->p_seq = block_Duplicate( p_frag );
-                p_sys->i_seq_old = 0;
+            }
+
+            /* We've just started the stream, wait for the first PTS.
+             * We discard here so we can still get the sequence header. */
+            if( p_sys->i_dts <= 0 && p_sys->i_pts <= 0 &&
+                p_sys->i_interpolated_dts <= 0 )
+            {
+                msg_Dbg( p_dec, "need a starting pts/dts" );
+                p_sys->i_state = STATE_NOSYNC;
+                block_Release( p_pic );
+                break;
+            }
+
+            /* When starting the stream we can have the first frame with
+             * a null DTS (i_interpolated_pts is initialized to 0) */
+            if( !p_pic->i_dts ) p_pic->i_dts = p_pic->i_pts;
+
+            /* So p_block doesn't get re-added several times */
+            *pp_block = block_BytestreamPop( &p_sys->bytestream );
+
+            p_sys->i_state = STATE_NOSYNC;
+
+            return p_pic;
+        }
+    }
+}
+
+/*****************************************************************************
+ * ParseMPEGBlock: Re-assemble fragments into a block containing a picture
+ *****************************************************************************/
+static block_t *ParseMPEGBlock( decoder_t *p_dec, block_t *p_frag )
+{
+    decoder_sys_t *p_sys = p_dec->p_sys;
+    block_t *p_pic = NULL;
+
+    /*
+     * Check if previous picture is finished
+     */
+    if( ( p_sys->b_frame_slice &&
+          (p_frag->p_buffer[3] == 0x00 || p_frag->p_buffer[3] > 0xaf) ) &&
+          p_sys->p_seq == NULL )
+    {
+        /* We have a picture but without a sequence header we can't
+         * do anything */
+        msg_Dbg( p_dec, "waiting for sequence start" );
+        if( p_sys->p_frame ) block_ChainRelease( p_sys->p_frame );
+        p_sys->p_frame = NULL;
+        p_sys->pp_last = &p_sys->p_frame;
+        p_sys->b_frame_slice = VLC_FALSE;
 
-                p_dec->fmt_out.video.i_width = ( p_frag->p_buffer[4] << 4)|(p_frag->p_buffer[5] >> 4 );
-                p_dec->fmt_out.video.i_height= ( (p_frag->p_buffer[5]&0x0f) << 8 )|p_frag->p_buffer[6];
+    }
+    else if( p_sys->b_frame_slice &&
+             (p_frag->p_buffer[3] == 0x00 || p_frag->p_buffer[3] > 0xaf) )
+    {
+        mtime_t i_duration;
 
-                p_sys->i_frame_rate = code_to_frame_rate[p_frag->p_buffer[7]&0x0f][0];
-                p_sys->i_frame_rate_base = code_to_frame_rate[p_frag->p_buffer[7]&0x0f][1];
+        p_pic = block_ChainGather( p_sys->p_frame );
 
+        i_duration = (mtime_t)( 1000000 * p_sys->i_frame_rate_base /
+                                p_sys->i_frame_rate );
 
-                p_sys->b_seq_progressive = VLC_TRUE;
-                p_sys->b_low_delay = VLC_TRUE;
+        if( !p_sys->b_seq_progressive && p_sys->i_picture_structure != 0x03 )
+        {
+            i_duration /= 2;
+        }
 
-                msg_Dbg( p_dec, "Size %dx%d fps=%.3f",
-                         p_dec->fmt_out.video.i_width,
-                         p_dec->fmt_out.video.i_height,
-                         (float)p_sys->i_frame_rate / (float)p_sys->i_frame_rate_base );
+        if( p_sys->b_seq_progressive )
+        {
+            if( p_sys->i_top_field_first == 0 &&
+                p_sys->i_repeat_first_field == 1 )
+            {
+                i_duration *= 2;
             }
-            else if( p_frag->p_buffer[3] == 0xb5 )
+            else if( p_sys->i_top_field_first == 1 &&
+                     p_sys->i_repeat_first_field == 1 )
             {
-                int i_type = p_frag->p_buffer[4] >> 4;
-                /* extention start code */
-                if( i_type == 0x01 )
-                {
-                    /* sequence extention */
-                    if( p_sys->p_ext)
-                    {
-                        block_Release( p_sys->p_ext );
-                    }
-                    p_sys->p_ext = block_Duplicate( p_frag );
-
-                    if( p_frag->i_buffer >= 10 )
-                    {
-                        p_sys->b_seq_progressive = p_frag->p_buffer[5]&0x08 ? VLC_TRUE : VLC_FALSE;
-                        p_sys->b_low_delay = p_frag->p_buffer[9]&0x80 ? VLC_TRUE : VLC_FALSE;
-                    }
-                }
-                else if( i_type == 0x08 )
-                {
-                    /* picture extention */
-                    p_sys->i_picture_structure = p_frag->p_buffer[6]&0x03;
-                    p_sys->i_top_field_first   = p_frag->p_buffer[7] >> 7;
-                    p_sys->i_repeat_first_field= (p_frag->p_buffer[7]>>1)&0x01;
-                    p_sys->i_progressive_frame = p_frag->p_buffer[8] >> 7;
-                }
+                i_duration *= 3;
             }
-            else if( p_frag->p_buffer[3] == 0x00 )
+        }
+        else
+        {
+            if( p_sys->i_picture_structure == 0x03 )
             {
-                /* picture */
-                p_sys->i_seq_old++;
-
-                if( p_frag->i_buffer >= 6 )
+                if( p_sys->i_progressive_frame && p_sys->i_repeat_first_field )
                 {
-                    p_sys->i_temporal_ref = ( p_frag->p_buffer[4] << 2 )|(p_frag->p_buffer[5] >> 6);
-                    p_sys->i_picture_type = ( p_frag->p_buffer[5] >> 3 )&0x03;
-                }
-                if( !p_sys->b_frame_slice )
-                {
-                    p_sys->i_dts = p_block->i_dts; p_block->i_dts = 0;
-                    p_sys->i_pts = p_block->i_pts; p_block->i_pts = 0;
+                    i_duration += i_duration / 2;
                 }
             }
-            else if( p_frag->p_buffer[3] >= 0x01 && p_frag->p_buffer[3] <= 0xaf )
+        }
+
+        if( p_sys->b_low_delay || p_sys->i_picture_type == 0x03 )
+        {
+            /* Trivial case (DTS == PTS) */
+            /* Correct interpolated dts when we receive a new pts/dts */
+            if( p_sys->i_pts > 0 ) p_sys->i_interpolated_dts = p_sys->i_pts;
+            if( p_sys->i_dts > 0 ) p_sys->i_interpolated_dts = p_sys->i_dts;
+        }
+        else
+        {
+            /* Correct interpolated dts when we receive a new pts/dts */
+            if( p_sys->i_last_ref_pts > 0 && !p_sys->b_second_field )
+                p_sys->i_interpolated_dts = p_sys->i_last_ref_pts;
+            if( p_sys->i_dts > 0 ) p_sys->i_interpolated_dts = p_sys->i_dts;
+
+            if( !p_sys->b_second_field )
+                p_sys->i_last_ref_pts = p_sys->i_pts;
+        }
+
+        p_pic->i_dts = p_sys->i_interpolated_dts;
+        p_sys->i_interpolated_dts += i_duration;
+
+        /* Set PTS only if we have a B frame or if it comes from the stream */
+        if( p_sys->i_pts > 0 )
+        {
+            p_pic->i_pts = p_sys->i_pts;
+        }
+        else if( p_sys->i_picture_type == 0x03 )
+        {
+            p_pic->i_pts = p_pic->i_dts;
+        }
+        else
+        {
+            p_pic->i_pts = 0;
+        }
+
+        switch ( p_sys->i_picture_type )
+        {
+        case 0x01:
+            p_pic->i_flags |= BLOCK_FLAG_TYPE_I;
+            break;
+        case 0x02:
+            p_pic->i_flags |= BLOCK_FLAG_TYPE_P;
+            break;
+        case 0x03:
+            p_pic->i_flags |= BLOCK_FLAG_TYPE_B;
+            break;
+        }
+
+        p_pic->i_length = p_sys->i_interpolated_dts - p_pic->i_dts;
+
+#if 0
+        msg_Dbg( p_dec, "pic: type=%d dts="I64Fd" pts-dts="I64Fd,
+        p_sys->i_picture_type, p_pic->i_dts, p_pic->i_pts - p_pic->i_dts);
+#endif
+
+        /* Reset context */
+        p_sys->p_frame = NULL;
+        p_sys->pp_last = &p_sys->p_frame;
+        p_sys->b_frame_slice = VLC_FALSE;
+
+        if( p_sys->i_picture_structure != 0x03 )
+        {
+            p_sys->b_second_field = !p_sys->b_second_field;
+        }
+        else
+        {
+            p_sys->b_second_field = 0;
+        }
+    }
+
+    /*
+     * Check info of current fragment
+     */
+    if( p_frag->p_buffer[3] == 0xb8 )
+    {
+        /* Group start code */
+        if( p_sys->p_seq &&
+            p_sys->i_seq_old > p_sys->i_frame_rate/p_sys->i_frame_rate_base )
+        {
+            /* Usefull for mpeg1: repeat sequence header every second */
+            block_ChainLastAppend( &p_sys->pp_last, block_Duplicate( p_sys->p_seq ) );
+            if( p_sys->p_ext )
             {
-                /* Slice */
-                p_sys->b_frame_slice = VLC_TRUE;
+                block_ChainLastAppend( &p_sys->pp_last, block_Duplicate( p_sys->p_ext ) );
             }
 
-            /* Append the block */
-            block_ChainAppend( &p_sys->p_frame, p_frag );
+            p_sys->i_seq_old = 0;
         }
-        p_sys->p_old = p_sys->p_start;
-        p_sys->p_start += 4;
     }
-}
+    else if( p_frag->p_buffer[3] == 0xb3 && p_frag->i_buffer >= 8 )
+    {
+        /* Sequence header code */
+        static const int code_to_frame_rate[16][2] =
+        {
+            { 1, 1 },  /* invalid */
+            { 24000, 1001 }, { 24, 1 }, { 25, 1 },       { 30000, 1001 },
+            { 30, 1 },       { 50, 1 }, { 60000, 1001 }, { 60, 1 },
+            /* Unofficial 15fps from Xing*/
+            { 15, 1001 },
+            /* Unofficial economy rates from libmpeg3 */
+            { 5, 1001 }, { 10, 1001 }, { 12, 1001 }, { 15, 1001 },
+            { 1, 1 },  { 1, 1 }  /* invalid */
+        };
+
+        if( p_sys->p_seq ) block_Release( p_sys->p_seq );
+        if( p_sys->p_ext ) block_Release( p_sys->p_ext );
+
+        p_sys->p_seq = block_Duplicate( p_frag );
+        p_sys->i_seq_old = 0;
+        p_sys->p_ext = NULL;
+
+        p_dec->fmt_out.video.i_width =
+            ( p_frag->p_buffer[4] << 4)|(p_frag->p_buffer[5] >> 4 );
+        p_dec->fmt_out.video.i_height =
+            ( (p_frag->p_buffer[5]&0x0f) << 8 )|p_frag->p_buffer[6];
+        p_sys->i_aspect_ratio_info = p_frag->p_buffer[7] >> 4;
+
+        /* TODO: MPEG1 aspect ratio */
+
+        p_sys->i_frame_rate = code_to_frame_rate[p_frag->p_buffer[7]&0x0f][0];
+        p_sys->i_frame_rate_base =
+            code_to_frame_rate[p_frag->p_buffer[7]&0x0f][1];
+
+        p_dec->fmt_out.video.i_frame_rate = p_sys->i_frame_rate;
+        p_dec->fmt_out.video.i_frame_rate_base = p_sys->i_frame_rate_base;
+
+        p_sys->b_seq_progressive = VLC_TRUE;
+        p_sys->b_low_delay = VLC_TRUE;
+
+        if ( !p_sys->b_inited )
+        {
+            msg_Dbg( p_dec, "size %dx%d fps=%.3f",
+                 p_dec->fmt_out.video.i_width, p_dec->fmt_out.video.i_height,
+                 p_sys->i_frame_rate / (float)p_sys->i_frame_rate_base );
+            p_sys->b_inited = 1;
+        }
+    }
+    else if( p_frag->p_buffer[3] == 0xb5 )
+    {
+        int i_type = p_frag->p_buffer[4] >> 4;
 
-static int mpgv_FindStartCode( uint8_t **pp_start, uint8_t *p_end )
-{
-    uint8_t *p = *pp_start;
+        /* Extension start code */
+        if( i_type == 0x01 )
+        {
+#if 0
+            static const int mpeg2_aspect[16][2] =
+            {
+                {0,1}, {1,1}, {4,3}, {16,9}, {221,100},
+                {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1},
+                {0,1}, {0,1}
+            };
+#endif
 
-    for( p = *pp_start; p < p_end - 4; p++ )
-    {
-        if( p[0] == 0 && p[1] == 0 && p[2] == 1 )
+            /* sequence extension */
+            if( p_sys->p_ext) block_Release( p_sys->p_ext );
+            p_sys->p_ext = block_Duplicate( p_frag );
+
+            if( p_frag->i_buffer >= 10 )
+            {
+                p_sys->b_seq_progressive =
+                    p_frag->p_buffer[5]&0x08 ? VLC_TRUE : VLC_FALSE;
+                p_sys->b_low_delay =
+                    p_frag->p_buffer[9]&0x80 ? VLC_TRUE : VLC_FALSE;
+            }
+
+            /* Do not set aspect ratio : in case we're transcoding,
+             * transcode will take our fmt_out as a fmt_in to libmpeg2.
+             * libmpeg2.c will then believe that the user has requested
+             * a specific aspect ratio, which she hasn't. Thus in case
+             * of aspect ratio change, we're screwed. --Meuuh
+             */
+#if 0
+            p_dec->fmt_out.video.i_aspect =
+                mpeg2_aspect[p_sys->i_aspect_ratio_info][0] *
+                VOUT_ASPECT_FACTOR /
+                mpeg2_aspect[p_sys->i_aspect_ratio_info][1];
+#endif
+
+        }
+        else if( i_type == 0x08 )
         {
-            *pp_start = p;
-            return VLC_SUCCESS;
+            /* picture extension */
+            p_sys->i_picture_structure = p_frag->p_buffer[6]&0x03;
+            p_sys->i_top_field_first   = p_frag->p_buffer[7] >> 7;
+            p_sys->i_repeat_first_field= (p_frag->p_buffer[7]>>1)&0x01;
+            p_sys->i_progressive_frame = p_frag->p_buffer[8] >> 7;
         }
     }
+    else if( p_frag->p_buffer[3] == 0x00 )
+    {
+        /* Picture start code */
+        p_sys->i_seq_old++;
 
-    *pp_start = p;
-    return VLC_EGENERIC;
-}
+        if( p_frag->i_buffer >= 6 )
+        {
+            p_sys->i_temporal_ref =
+                ( p_frag->p_buffer[4] << 2 )|(p_frag->p_buffer[5] >> 6);
+            p_sys->i_picture_type = ( p_frag->p_buffer[5] >> 3 ) & 0x03;
+        }
 
+        p_sys->i_dts = p_frag->i_dts;
+        p_sys->i_pts = p_frag->i_pts;
+    }
+    else if( p_frag->p_buffer[3] >= 0x01 && p_frag->p_buffer[3] <= 0xaf )
+    {
+        /* Slice start code */
+        p_sys->b_frame_slice = VLC_TRUE;
+    }
 
+    /* Append the block */
+    block_ChainLastAppend( &p_sys->pp_last, p_frag );
+
+    return p_pic;
+}