Change &WNOHANG to WNOHANG() or WNOHANG and then yell at whoever taught you to use &NAME for constants in hopes that this problematic but still common practice can be stomped out.

FYI, the reason you get this warning is that your sub reaper gets called with the name of the signal passed in to it and calling a subroutine (and constants from Perl modules are usually subroutines) with & and without parens means that the current value of @_ is reused as the arguments to that subroutine.

So you end up doing the equivalent of WNOHANG(@_) which, in your case, is similar to WNOHANG("CHLD"). Finally, for obscure reasons, the example code for implementing constants in XS modules, allows these so-called "constants" to take an optional numeric argument. But "CHLD" isn't numeric, hence the warning.

        - tye (but my friends call me "Tye")

In reply to (tye)Re: reaper subroutines by tye
in thread reaper subroutines by jalebie

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":


  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.