[FFmpeg-devel] [PATCH] [RFC] avformat: use "genpts" option for output file.

[Benoit Fouet]

Hi,

----- Mail original -----
> On Mon, Nov 03, 2014 at 03:21:33PM +0100, Benoit Fouet wrote:
> > When "-fflags +genpts" is used for output file, use dts as pts.
> > 
> > Signed-off-by: Benoit Fouet <benoit.fouet at free.fr>
> > ---
> >  libavformat/mux.c | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/libavformat/mux.c b/libavformat/mux.c
> > index 5e45bd8..1d42451 100644
> > --- a/libavformat/mux.c
> > +++ b/libavformat/mux.c
> > @@ -884,6 +884,9 @@ int av_interleaved_write_frame(AVFormatContext
> > *s, AVPacket *pkt)
> >              ret = AVERROR(EINVAL);
> >              goto fail;
> >          }
> > +
> > +        if (pkt->pts == AV_NOPTS_VALUE && s->flags &
> > AVFMT_FLAG_GENPTS)
> > +            pkt->pts = pkt->dts;
> 
> does this set pts to invalid value for h264 with b frames or
> b pyramid ?
> 

No. The only thing I wanted to do was to provide a way for users to remux h264 elementary streams to .ts or .mkv
Maybe I could add a new option, like +copy_dts_to_pts instead of using the existing +genpts option?

-- 
Ben