On Mon, Apr 11, 2005 at 01:34:11PM -0400, Stas Bekman ([EMAIL PROTECTED]) wrote: > Tim Bunce wrote: > [...] > >I've applied the patch (change #970) and make a few tweaks. I needed to > >add a 'const char *' entry to the typemap file for Perl 5.6. I've also > >reworked the way gcc flags get set and enabled more of them by default. > > The current svn now gives: > > cc -c -W -Wall -Wpointer-arith -Wmissing-noreturn -Wbad-function-cast > -Wno-comment -Wno-sign-compare -Wno-cast-qual -Wdisabled-optimization -g > -DVERSION=\"1.49\" -DXS_VERSION=\"1.49\" -fpic > "-I/home/stas/perl/5.8.6-ithread/lib/5.8.6/i686-linux-thread-multi/CORE" > Perl.c > Perl.xs:39: warning: unused parameter 'h' > Perl.c:86: warning: unused parameter 'cv' > Perl.c:122: warning: unused parameter 'cv' > Perl.c:157: warning: unused parameter 'cv'
Yup, that's with the checking for unused parms. I've just sent a patch to p5p for XSUBs to have the cv parameter be __attribute__((unused)) and not give that warning. -- Andy Lester => [EMAIL PROTECTED] => www.petdance.com => AIM:petdance
