[FFmpeg-cvslog] r21013 - trunk/libavcodec/h263data.h

[Michael Niedermayer]

On Mon, Jan 04, 2010 at 02:21:18PM +0100, diego wrote:
> Author: diego
> Date: Mon Jan  4 14:21:18 2010
> New Revision: 21013
> 
> Log:
> The H.263 wrong_run table is only for encoding; only enable it in such cases.
> patch by avcoder, ffmpeg gmail com

the table is static, its the compilers and linkers job to ommit it if it is
not used. puting all static functions & table under ifdefs is not a reasonable
choice. The work to maintain this and the reduced readablility is really a much
bigger issue than the zero advantage this leads to
please revert

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20100104/2dfb0193/attachment.pgp>