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
