No, it should be void. The C function doesn't return anything.

#define XSRETURN(off)                                   \
    STMT_START {                                        \
        const IV tmpXSoff = (off);                      \
        PL_stack_sp = PL_stack_base + ax + (tmpXSoff - 1);      \
        return;                                         \
    } STMT_END
[download]

There are two ways of returning to Perl from an XS function:

• Place it on the Perl stack.

  void foo() {
     ...
     mXPUSHs(sv);
  }
  [download]

• Return it from the C function. The typemap will place it on the Perl stack after any conversion and mortalizing it performs.

  SV* foo() {
     ...
     return sv;
  }
  [download]

The OP's code uses the former approach.

Indeed. Thanks for the correction.

---

With the rise and rise of 'Social' network sites: 'Computers are making people easier to use everyday'

Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.

"Science is about questioning the status quo. Questioning authority".

In the absence of evidence, opinion is indistinguishable from prejudice.