[Ffmpeg-devel] [PATCH] flacenc - md5

[Justin Ruggles]

Michael Niedermayer wrote:
> Hi
> 
> On Mon, Jul 03, 2006 at 02:22:33PM -0400, Justin Ruggles wrote:
> [...]
> 
>>I first thought of putting the lavc version in the vendor string like in
>>the vorbis encoder, but then realized this would break regression tests
>>at every version increment, so I just used "libavcodec" instead.  I've
> 
> 
> just use:
> 
> if(!(AVCodecContext->flags & CODEC_FLAG_BITEXACT)){
>     write full vendor string with version number
> }
> 
> [...]
> 

Good idea. :)
New patch attached. With the different string lengths I had to change
the code a bit to keep the padding the same size. The new version is
simpler, too.

I am not 100% sure about the big-endian section in md5_accumulate.  I
don't have a big-endian system to test on.  Just let me know if it
doesn't work or if it could be done in a better or simpler way.

-Justin
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: flac-md5.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060704/e9bc6fe1/attachment.asc>