[MPlayer-dev-eng] [PATCH] libmpcodecs/ad_faad.c compile fix

[Andre Schulz]

Björn Sandell wrote:
> On Tue, 04 Feb 2003 00:10:10 +0100
> Andre Schulz <andre at 0wnage.de> wrote:
> 
> 
>>I tried to compile mplayer with the faad2-1.1 library and it choked.
>>It turned out that the faacDecInit() and faacDecDecode() functions were 
>>just missing 1 argument. (the buffer length)
> 
> 
> Err, something dosen't add up here. From my faad.h
> 
> #define FAAD2_VERSION "1.1"
> 
> long FAADAPI faacDecInit(faacDecHandle hDecoder,
>                         unsigned char *buffer,
>                         unsigned long *samplerate,
>                         unsigned char *channels);
> 
> void* FAADAPI faacDecDecode(faacDecHandle hDecoder,
>                             faacDecFrameInfo *hInfo,
>                             unsigned char *buffer);
> 
> Have been using it for three months.
> 

Oops. Sorry.
You're right.
I had both faad2-1.1 and cvs sources but installed the latter. :|
The #define for the cvs version is: "1.2 beta"

Arpi: I don't know if you still would like a patch for this.
Anyway, I have hacked together a updated version which is attached to 
this mail.

Btw maybe I don't have enough experience in C but parsing the 
FAAD2_VERSION define into something usable is a pain in the ass.

Andre

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: MPlayer-20030204-faad2.patch
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20030204/52464c8d/attachment.asc>