[Home]

Summary:ASTERISK-02473: compilation against latest ilbc don't work
Reporter:oden (oden)Labels:
Date Opened:2004-09-26 18:55:44Date Closed:2004-10-14 18:53:48
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/CodecInterface
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:Hi.

I maintain the Mandrakelinux packages, and today I made a shared library out of the latest ilbc code found here:

http://www.ietf.org/internet-drafts/draft-ietf-avt-ilbc-codec-05.txt

But I get this error while trying to compile against it:

codec_ilbc.c: In function `lintoilbc_new':
codec_ilbc.c:64: error: too few arguments to function `initEncode'
codec_ilbc.c: In function `ilbctolin_new':
codec_ilbc.c:78: error: too few arguments to function `initDecode'
codec_ilbc.c: In function `lintoilbc_sample':
codec_ilbc.c:95: warning: assignment discards qualifiers from pointer target type
codec_ilbc.c: In function `ilbctolin_sample':
codec_ilbc.c:110: warning: assignment discards qualifiers from pointer target type
codec_ilbc.c: In function `ilbctolin_frameout':
codec_ilbc.c:128: warning: assignment discards qualifiers from pointer target type
codec_ilbc.c: In function `lintoilbc_frameout':
codec_ilbc.c:189: warning: assignment discards qualifiers from pointer target type
make[1]: *** [codec_ilbc.o] Error 1

If I use the two older headers from the bundled ilbc code it compiles just fine.
Comments:By: Brian West (bkw918) 2004-09-26 20:15:22

No need for that you use the code that we bring with asterisk not the latest or other code you find... it works fine.

By: oden (oden) 2004-09-26 20:26:44

The thing is that we recently had a bug related to the bundled ilbc in openh323, an upgrade of that code solved that problem. I was planning building openh323 against my new shared ilbc package as well as I now do with asterisk.

FYI. I use almost none of the bundled libraries for my asterisk packages, it seems to work fine.

By: Brian West (bkw918) 2004-09-26 20:46:52

Sounds like the API changed a bit.  Not much we can do at this point since what we support is whats included unless you can show me the headerfiles and the API changes.

bkw

By: Mark Spencer (markster) 2004-10-14 01:15:51

Fixed in CVS, updated to latest code in CVS anyway.

By: Russell Bryant (russell) 2004-10-14 18:53:48

fixes not inluded in the 1.0 branch