[FFmpeg-devel] [PATCH] moves ff_set_mpeg4_time to mpegvideo_enc.c

Michael Niedermayer michaelni
Fri Jul 6 10:44:55 CEST 2007


Hi

On Fri, Jul 06, 2007 at 01:51:20AM +0200, Aurelien Jacobs wrote:
> Hi,
> 
> The attached patch moves ff_set_mpeg4_time to mpegvideo_enc.c because
> it is only used by this file. This allows to make the function static.
> Moreover this function is used for mpeg1/2 as well as mpeg4 so it
> definitely don't belong in h263.c and it really deserve a renaming
> (as suggested by the FIXME comment).
> 
> Is it OK ?

well ...

the function should be split, some of the things are needed for all
b frame encodings some are mpeg4 specific, later belong in h263.c
not mpegvideo*.c unless you split mpeg4 out of h263.c ...

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

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- 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/20070706/932ec608/attachment.pgp>



More information about the ffmpeg-devel mailing list