[FFmpeg-devel] dvdata.h mess, are there still objections to fixing it

Reimar Döffinger Reimar.Doeffinger
Thu Oct 15 20:17:01 CEST 2009


Hello,
I remember that dvdata.h was discussed in the past.
There is a huge amount of data in that header, and it is included 3
times.
That means that the FFmpeg binary contains
dv_idct_factor_hd1080, dv_idct_factor_hd720, work_chunks_dv100pali,
work_chunks_dv100ntsci, dv_idct_factor_sd etc. 3 times, making about
233kB of .bss wasted for no good reason at all.
Preferably someone should check which ones are actually, really,
performance relevant and that compiler optimizes away and keep only
those in the header, but in case nobody volunteers, are there any
objections to moving dv_profiles and all the non-const stuff to a
separate .c file (while adding ff_ prefix)?



More information about the ffmpeg-devel mailing list