Summary: | ASTERISK-02473: compilation against latest ilbc don't work | ||
Reporter: | oden (oden) | Labels: | |
Date Opened: | 2004-09-26 18:55:44 | Date Closed: | 2004-10-14 18:53:48 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | 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 |