[FFmpeg-devel] [PATCH] H.264 data tables cleanup
Thu Nov 20 10:14:37 CET 2008
On Thu, Nov 20, 2008 at 12:29:50AM -0800, Jason Garrett-Glaser wrote:
> On Thu, Nov 20, 2008 at 12:08 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Thu, Nov 20, 2008 at 12:49:03AM +0100, Diego Biurrun wrote:
> >> So I've (re)started working on splitting off svq3.c from h264.c.
> >> Here are some simple first steps:
> >> - Remove unused tables from h264data.h.
> > these tables are unused because we dont have an encoder ...
> > putting them under appropriate ifdef or in a seperate encoder
> > specific (not compiled) header seems better than removing them
> > even if there are no plans for an encoder ...
> What's the point though? If someone wants them at some time in the
> future, can't they just restore them from svn history? Why keep
> around code that has no plans to be used, even if ifdef'd out?
because when someone might want to write a encoder or bitstream generator
he will likely not find such tables and functions when they have been removed
even if they are somewhere in the history.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel