From: Gildas Bazin <gbazin@videolan.org>
Date: Wed, 27 Oct 2004 14:59:24 +0000 (+0000)
Subject: * modules/mux/mpeg: for video ES, always put the pts and dts in the pes header when... 
X-Git-Tag: 0.8.0~47
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=ea92845937166d277eea8dfaf4555cfea48c7a55;p=vlc

* modules/mux/mpeg: for video ES, always put the pts and dts in the pes header when they are both available (makes vlc a lot happier when demuxing the stream, especially for other video codecs than mpeg2).
---

diff --git a/modules/mux/mpeg/pes.c b/modules/mux/mpeg/pes.c
index 875132ee51..b342fedb39 100644
--- a/modules/mux/mpeg/pes.c
+++ b/modules/mux/mpeg/pes.c
@@ -47,7 +47,8 @@
 #define PES_PAYLOAD_SIZE_MAX 65500
 
 static inline int PESHeader( uint8_t *p_hdr, mtime_t i_pts, mtime_t i_dts,
-                             int i_es_size, int i_stream_id, int i_private_id,
+                             int i_es_size, es_format_t *p_fmt,
+                             int i_stream_id, int i_private_id,
                              vlc_bool_t b_mpeg2, vlc_bool_t b_data_alignment,
                              int i_header_size )
 {
@@ -93,7 +94,8 @@ static inline int PESHeader( uint8_t *p_hdr, mtime_t i_pts, mtime_t i_dts,
             {
                 int i_pts_dts;
 
-                if( i_pts > 0 && i_dts > 0 && i_pts != i_dts )
+                if( i_pts > 0 && i_dts > 0 &&
+                    ( i_pts != i_dts || p_fmt->i_cat == VIDEO_ES ) )
                 {
                     i_pts_dts = 0x03;
                     if ( !i_header_size ) i_header_size = 0xa;
@@ -161,7 +163,8 @@ static inline int PESHeader( uint8_t *p_hdr, mtime_t i_pts, mtime_t i_dts,
             {
                 int i_pts_dts;
 
-                if( i_pts > 0 && i_dts > 0 && i_pts != i_dts )
+                if( i_pts > 0 && i_dts > 0 &&
+                    ( i_pts != i_dts || p_fmt->i_cat == VIDEO_ES ) )
                 {
                     bits_write( &bits, 16, i_es_size + i_extra + 10 /* + stuffing */ );
                     i_pts_dts = 0x03;
@@ -227,10 +230,8 @@ static inline int PESHeader( uint8_t *p_hdr, mtime_t i_pts, mtime_t i_dts,
     }
 }
 
-int E_( EStoPES )( sout_instance_t *p_sout,
-                   block_t **pp_pes,
-                   block_t *p_es,
-                   int i_stream_id,
+int E_( EStoPES )( sout_instance_t *p_sout, block_t **pp_pes, block_t *p_es,
+                   es_format_t *p_fmt, int i_stream_id,
                    int b_mpeg2, int b_data_alignment, int i_header_size )
 {
     block_t *p_pes;
@@ -266,7 +267,7 @@ int E_( EStoPES )( sout_instance_t *p_sout,
     {
         i_pes_payload = __MIN( i_size, PES_PAYLOAD_SIZE_MAX );
         i_pes_header  = PESHeader( header, i_pts, i_dts, i_pes_payload,
-                                   i_stream_id, i_private_id, b_mpeg2,
+                                   p_fmt, i_stream_id, i_private_id, b_mpeg2,
                                    b_data_alignment, i_header_size );
         i_dts = 0; // only first PES has a dts/pts
         i_pts = 0;
@@ -314,7 +315,6 @@ int E_( EStoPES )( sout_instance_t *p_sout,
 
         i_dts += i_length;
     }
-    return( 0 );
-}
-
 
+    return 0;
+}
diff --git a/modules/mux/mpeg/pes.h b/modules/mux/mpeg/pes.h
index 5a3510f935..837dbbcb9e 100644
--- a/modules/mux/mpeg/pes.h
+++ b/modules/mux/mpeg/pes.h
@@ -22,7 +22,6 @@
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
  *****************************************************************************/
 
-
 #define PES_PROGRAM_STREAM_MAP          0xbc
 #define PES_PRIVATE_STREAM_1            0xbd
 #define PES_PADDING                     0xbe
@@ -33,8 +32,6 @@
 #define PES_DSMCC_STREAM                0xf2
 #define PES_ITU_T_H222_1_TYPE_E_STREAM  0xf8
 
-
-int E_( EStoPES )( sout_instance_t *p_sout,
-                   block_t **pp_pes, block_t *p_es,
-                   int i_stream_id, int b_mpeg2, int b_data_alignment,
-                   int i_header_size );
+int E_( EStoPES )( sout_instance_t *p_sout, block_t **pp_pes, block_t *p_es,
+                   es_format_t *p_fmt, int i_stream_id,
+                   int b_mpeg2, int b_data_alignment, int i_header_size );
diff --git a/modules/mux/mpeg/ps.c b/modules/mux/mpeg/ps.c
index 2ae38adaf1..f337f99c8e 100644
--- a/modules/mux/mpeg/ps.c
+++ b/modules/mux/mpeg/ps.c
@@ -371,7 +371,8 @@ static int Mux( sout_mux_t *p_mux )
 
         /* Get and mux a packet */
         p_data = block_FifoGet( p_input->p_fifo );
-        E_( EStoPES )( p_mux->p_sout, &p_data, p_data, p_stream->i_stream_id,
+        E_( EStoPES )( p_mux->p_sout, &p_data, p_data,
+                       p_input->p_fmt, p_stream->i_stream_id,
                        p_mux->p_sys->b_mpeg2, 0, 0 );
 
         block_ChainAppend( &p_ps, p_data );
diff --git a/modules/mux/mpeg/ts.c b/modules/mux/mpeg/ts.c
index 9d89ba3da1..42abb2c3a8 100644
--- a/modules/mux/mpeg/ts.c
+++ b/modules/mux/mpeg/ts.c
@@ -1095,6 +1095,7 @@ static int Mux( sout_mux_t *p_mux )
                                     p_spu->p_buffer[2] = ' ';
 
                                     E_(EStoPES)( p_mux->p_sout, &p_spu, p_spu,
+                                                 p_input->p_fmt,
                                                  p_stream->i_stream_id, 1,
                                                  0, 0 );
                                     p_data->p_next = p_spu;
@@ -1131,8 +1132,8 @@ static int Mux( sout_mux_t *p_mux )
                             p_data->i_pts = p_data->i_dts;
                         }
                         E_( EStoPES )( p_mux->p_sout, &p_data, p_data,
-                                       p_stream->i_stream_id, 1,
-                                       b_data_alignment, i_header_size );
+                                       p_input->p_fmt, p_stream->i_stream_id,
+                                       1, b_data_alignment, i_header_size );
 
                         BufferChainAppend( &p_stream->chain_pes, p_data );