New ffmpeg CVS versions

[Jacob Meuser]

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 at 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 at freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-multimedia
> To unsubscribe, send any mail to 
> "freebsd-multimedia-unsubscribe at freebsd.org"