[FFmpeg-devel] [PATCH] split mpegvideo.c

Michael Niedermayer michaelni
Mon Jul 2 19:35:42 CEST 2007


Hi

On Mon, Jul 02, 2007 at 05:51:21PM +0200, Benoit Fouet wrote:
> Michael Niedermayer wrote:
> > Hi
> >
> > On Mon, Jul 02, 2007 at 05:30:34PM +0200, Benoit Fouet wrote:
> >   
> >> does that mean i have to benchmark calls to copy_picture or that the
> >> patch can be applied ?
> >>     
> >
> > no sense in benchmarking copy_picture()
> > though a "time ffmpeg -i foobar barfoo" cant hurt
> >
> > if theres no slowdown then apply it
> >
> > [...]
> >   
> >   
> 
> well, i tried out this morning, and the difference is the same between
> two calls to the same binary as to the two different binaries :)
> also, what are the prefered names for new files ?
> i proposed those 2 ones, but if it's not the usual naming convention,
> they could be changed...

iam fine with the names (or other names) also
its trivial to rename them later so id say dont delay the patch due to
file names

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070702/e339c42f/attachment.pgp>



More information about the ffmpeg-devel mailing list