This should work. On Sun, Jan 31, 2010 at 2:48 PM, Eugen Dedu <eugen.d...@pu-pm.univ-fcomte.fr > wrote:
> I have two warnings: > snoopy:~/softs/ekiga/opal/plugins/audio/celt$ make > [CC] celtcodec.c > celtcodec.c: In function ‘valid_for_sip’: > celtcodec.c:196: warning: pointer targets in passing argument 1 of > ‘strcasecmp’ differ in signedness > /usr/include/string.h:534: note: expected ‘const char *’ but argument is > of type ‘const unsigned char *’ > celtcodec.c: At top level: > celtcodec.c:202: warning: initialization from incompatible pointer type > [LD] ./celtcodec_pwplugin.so > > I removed the first one by removing "unsigned". Do you know how to fix > the 2nd warning? > > e schmidbauer wrote: > > Please try compiling with this file. I have not been able to test this > > version of celt because i do not have a linux machine available to cross > > compile. If you are successful at compiling, i can test it out. I will > get > > to the plugins/configure.ac file later. Please tell me if you get any > errors > > or warnings. Thank you. > > > > On Sun, Jan 31, 2010 at 8:15 AM, Eugen Dedu < > eugen.d...@pu-pm.univ-fcomte.fr > >> wrote: > > > >> e schmidbauer wrote: > >>> I've attached what i think will work for compiling opal with celt 0.7 > >>> Can you please test it to see if it compiles? > >> Thanks a lot for the patch. > >> > >> 1. When compiling, I have a few warnings, could you remove them too? > >> > >> snoopy:~/softs/ekiga/opal/plugins/audio/celt$ make > >> [CC] celtcodec.c > >> celtcodec.c: In function ‘celt_codec_encoder’: > >> celtcodec.c:146: warning: pointer targets in passing argument 4 of > >> ‘celt_encode’ differ in signedness > >> /usr/include/celt/celt.h:214: note: expected ‘unsigned char *’ but > >> argument is of type ‘char *’ > >> celtcodec.c: In function ‘celt_codec_decoder’: > >> celtcodec.c:176: warning: pointer targets in passing argument 2 of > >> ‘celt_decode’ differ in signedness > >> /usr/include/celt/celt.h:262: note: expected ‘const unsigned char *’ but > >> argument is of type ‘char *’ > >> [LD] ./celtcodec_pwplugin.so > >> > >> 2. Also, have you successfully made a communication with celt (0.7) > >> using your file? > >> > >> 3. Do you have the will to modify also plugins/configure.ac so that it > >> defines HAVE_CELT_0_7_0 and use that #define in celtcodec.c so that it > >> compiles with previous versions of celt too? If not, I will send your > >> patch to be committed anyway, but it is better with it... > >> > >> Thanks, > >> -- > >> Eugen > >> _______________________________________________ > >> Ekiga-devel-list mailing list > >> Ekiga-devel-list@gnome.org > >> http://mail.gnome.org/mailman/listinfo/ekiga-devel-list > >> > > > > > > ------------------------------------------------------------------------ > > > > _______________________________________________ > > Ekiga-devel-list mailing list > > Ekiga-devel-list@gnome.org > > http://mail.gnome.org/mailman/listinfo/ekiga-devel-list > > _______________________________________________ > Ekiga-devel-list mailing list > Ekiga-devel-list@gnome.org > http://mail.gnome.org/mailman/listinfo/ekiga-devel-list >
celtcodec.c
Description: Binary data
_______________________________________________ Ekiga-devel-list mailing list Ekiga-devel-list@gnome.org http://mail.gnome.org/mailman/listinfo/ekiga-devel-list
