[Ffmpeg-devel] [PATCH] ported SGI decoder to the new API

[Xiaohui Sun]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Diego Biurrun wrote:
> On Tue, Mar 27, 2007 at 03:34:39PM +0800, Xiaohui Sun wrote:
>>      I have updated the path, Thank you again for your suggestions about
>> my code.
>>
>> the new patch
>> --  ported SGI encoder
>> --  seperate encoder/decoder into two files
>> --  fixed some problems of the previous code
> 
>> --- libavcodec/allcodecs.c	(revision 8531)
>> +++ libavcodec/allcodecs.c	(working copy)
>> @@ -132,6 +132,7 @@
>>      REGISTER_DECODER(THEORA, theora);
>>      REGISTER_DECODER(TIERTEXSEQVIDEO, tiertexseqvideo);
>>      REGISTER_DECODER(TIFF, tiff);
>> +    REGISTER_ENCDEC(SGI, sgi);
> 
> This should be in alphabetical order and the '(' should be aligned.
> 
>> --- libavcodec/avcodec.h	(revision 8531)
>> +++ libavcodec/avcodec.h	(working copy)
>> @@ -148,6 +148,7 @@
>>      CODEC_ID_DSICINVIDEO,
>>      CODEC_ID_TIERTEXSEQVIDEO,
>>      CODEC_ID_TIFF,
>> +    CODEC_ID_SGI,
>>      CODEC_ID_GIF,
>>      CODEC_ID_FFH264,
>>      CODEC_ID_DXA,
> 
> Add the new ID at the end.
> 
> BTW, this should be documented.

Where should I put this document?




-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGCTSr+BwsLu3sVWwRAoOlAKCnxxdbF2NkdR60mBRAG9HKGqRIggCgkyWQ
jN96v1qh2cVnF7HpfrkYbik=
=jeUh
-----END PGP SIGNATURE-----