On Thu, Apr 14, 2011 at 01:27:28PM +0200, Luca Barbato wrote:
> On 04/14/2011 11:14 AM, Luca Barbato wrote:
> > From: Rudolf Polzer <[email protected]>
> > 
> > The mpegts muxer does not set the stream time base using
> > av_set_pts_info, but expects it to have the default value of 1/90000.
> > If the calling code changes stream pts before writing the header,
> > other muxers override the time base at that point (like mpegenc.c).
> > ---
> >  libavformat/mpegtsenc.c |    1 +
> >  1 files changed, 1 insertions(+), 0 deletions(-)
> > 
> > diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
> > index 40b906e..c89eb15 100644
> > --- a/libavformat/mpegtsenc.c
> > +++ b/libavformat/mpegtsenc.c
> > @@ -479,6 +479,7 @@ static int mpegts_write_header(AVFormatContext *s)
> >      /* assign pids to each stream */
> >      for(i = 0;i < s->nb_streams; i++) {
> >          st = s->streams[i];
> > +        av_set_pts_info(st, 64, 1, 90000);
> >          ts_st = av_mallocz(sizeof(MpegTSWriteStream));
> >          if (!ts_st)
> >              goto fail;
> 
> patch queued with av_set_pts_info(st, 33, 1, 90000); will be pushed
> tonight or tomorrow evening.

looks good

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to