develooper Front page | perl.perl5.porters | Postings from May 2008

Re: 5.8.9-to-be on Irix

Thread Previous | Thread Next
From:
David Cantrell
Date:
May 15, 2008 07:57
Subject:
Re: 5.8.9-to-be on Irix
Message ID:
20080515145704.GA31165@bytemark.barnyard.co.uk
> >With 0.24 I noticed this:
> >    cc-1174 cc: WARNING File = Syslog.c, Line = 367
> >      The variable "RETVAL" was declared but never referenced.
> >which might explain the failures.
> Does this warning appears in XS_Sys__Syslog_setlogmask_xs()? If not,  
> could you paste me the part of the C file where it occurs?

It's here:
  
XS(XS_Sys__Syslog_setlogmask_xs)
{
    dXSARGS;
    if (items != 1)
        Perl_croak(aTHX_ "Usage: Sys::Syslog::setlogmask_xs(mask)");
    {
        int     mask = (int)SvIV(ST(0));
        int     RETVAL;
        dXSTARG;
#line 129 "Syslog.xs"
        setlogmask(mask);
#line 320 "Syslog.c"
    }
    XSRETURN(1);
}

> So, could you try the version from the SVN and check whether this  
> fixes the problem?
>   $ svn co svn://svn.mongueurs.net/Sys-Syslog

All tests passed with both 5.8.8 and 5.8.9-to-be.  The warning about
RETVAL is still there, and your MANIFEST is out of date - 'make distclean'
says:
  Not in MANIFEST: t/cpan-rt-21516.t
  Not in MANIFEST: t/cpan-rt-21866.t
  Not in MANIFEST: t/cpan-rt-25488.t

> Thanks for your hard work. I owe you your favourite drinking for the  
> next Perl conference :-)

Pity I'm not going to be at YAPC::Europe this year :-)

-- 
David Cantrell | http://www.cantrell.org.uk/david

  Irregular English:
    ladies glow; gentlemen perspire; brutes, oafs and athletes sweat

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About