Date: Tue, 3 May 2005 20:41:32 -0700 From: Jacob Meuser <jakemsr@jakemsr.com> To: freebsd-multimedia@freebsd.org Subject: Re: New ffmpeg CVS versions Message-ID: <20050504034132.GF27854@puff.jakemsr.gom> In-Reply-To: <20050504003425.A97449@kushnir1.kiev.ua> References: <20050502043610.51034.qmail@exxodus.fedaykin.here> <4275B154.5070007@ahze.net> <20050503070628.GF11140@puff.jakemsr.gom> <20050504003425.A97449@kushnir1.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, May 04, 2005 at 12:48:52AM +0300, Vladimir Kushnir wrote: > > > On Tue, 3 May 2005, Jacob Meuser wrote: > > <snip> > > > >notes on grab_bsdbktr.c > > > <snip> > > i think av_set_pts_info() needs to be moved up to before other > > parameters in 's' are set in grab_read_header(). > > Actually, with av_set_pts_info() as it is grabbing doesn't work even in > the present ffmpeg port. It should be > av_set_pts_info(st, 48, 1, 1000000); > not > av_set_pts_info(s1, 48, 1, 1000000); > (as av_set_pts_info() takes AVStream and not AVFormatContext). With this > change ffmpeg grabs even without any move. hmm, ok. I didn't notice that. thanks :) -- <jakemsr@jakemsr.com> > > > > i'm pretty sure the av_free() in grab_read_close() can result in > > double free errors, i.e. ffmpeg free()s the memory automagically. > > Yep, it does. > > Regards, > Vladimir > _______________________________________________ > freebsd-multimedia@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-multimedia > To unsubscribe, send any mail to > "freebsd-multimedia-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050504034132.GF27854>