Hi, ----- Mail original ----- > On Tue, Nov 04, 2014 at 01:15:02PM +0100, Benoit Fouet wrote: > > 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.fo...@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. > > how did you test this ? > can you show an example of b pyramid and the filled in PTS values >
Oh... I only tested stream copy. And this is the only thing I want to fix. I should ensure this is only used for this case, right? I think this can mess things up if used with "real" transcoding... -- Ben _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
