Beefy Boxes and Bandwidth Generously Provided by pair Networks
good chemistry is complicated,
and a little bit messy -LW


by syphilis (Chancellor)
on Sep 29, 2011 at 12:39 UTC ( #928545=note: print w/ replies, xml ) Need Help??

in reply to Re^10: XS: EXTEND/mPUSHi
in thread XS: EXTEND/mPUSHi

Heh - I must've looked at the C code a hundred times over the years, and never noticed the unreached
PUTBACK; return;
... though, as I write this, I'm suddenly struck with a contradictory sense of "deja vu".

That unreached code *is* confusing and/or annoying, and it would be nice to make that piece of xsubbp-generated code reachable (as you suggest) - which means amending in a way that makes me feel a bit uneasy.

Your 'do nothing' was probably the right call

Maybe ... I *have* been right before, y'know !! ... by accident, of course ( ... which would also be the case in this instance :-)

I'll chew this over for a while, as time permits. I've already checked that ikegami's suggested changes re the "no-op" and the "PUTBACK" insertions don't break the test suite. (However, they still leave us with that unreached "return" ... not that he suggested they would do otherwise.)

Thanks BrowserUk, ikegami.


Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://928545]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others making s'mores by the fire in the courtyard of the Monastery: (4)
As of 2016-07-27 10:05 GMT
Find Nodes?
    Voting Booth?
    What is your favorite alternate name for a (specific) keyboard key?

    Results (242 votes). Check out past polls.