[FFmpeg-devel] [PATCH] fix libavcodec test programs

[Panagiotis Issaris]

Diego Biurrun schreef:
> On Wed, Jan 23, 2008 at 01:12:04AM +0100, Michael Niedermayer wrote:
>> On Wed, Jan 23, 2008 at 12:53:23AM +0100, Diego Biurrun wrote:
>>> Here is a patch to make the test programs in rangecoder.c and cabac.c
>>> compile again.  If it's OK I would commit in several steps.
>> ok
> 
> Applied.
> 
>>> h264.c contains another test program, but I fear it is beyond my
>>> immediate ability for repair.  It uses the functions h264_diff_dct_c
>>> and encode_nal which no longer exist.  Is it worth keeping or should it
>>> just be removed?
>> NOTHING should be removed without a very good reason, "not working" is never
>> a good reason!
> 
> Being in a state of disrepair without any hope of repair in the
> foreseeable future can be such a reason.
> 
> Anyway, here is a patch that disables all problematic parts and "fixes"
> compilation.  I searched for the missing functions, encode_nal() was
> removed in r7228, h264_diff_dct_c in r7244, both commits by takis.

Before my commit removing it, it was "#if 0"-ed out. And the code in 
encode_nal() was reused in h264enc.c, in h264_write_nal_unit().



With friendly regards,
Takis
--
vCard: http://issaris.org/pi.vcf
PGP key: http://issaris.org/pi.key