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

Måns Rullgård mans
Thu Jun 28 02:59:21 CEST 2007


Benoit Fouet <benoit.fouet at purplelabs.com> writes:

> Hi,
>
> here is a first attempt to $subj
>
> what is done:
>  - remove everything flagged under CONFIG_ENCODERS from mpegvideo.c
> (apart from the DCT_common_init() part)
>  - put those parts in mpegvideo_enc.c

Does this in any way affect performance?  I can imagine calls from
encoding functions to common functions no longer being inlined due to
the split.  Then again, there might not be any relevant such calls.

>  - put in mpegvideo_common.heverything that's needed by both encoder and
> decoder

The usual naming convention would see common stuff in mpegvideo.h,
while encoder/decoder specific bits would go in mpegvideoenc.h and
mpegvideodec.h.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list