#1631: Exception handler throwing exception in NQP-rx causes segfault
--------------------+-------------------------------------------------------
Reporter: sorear | Owner: pmichaud
Type: bug | Status: new
Priority: normal | Milestone:
Component: nqp | Version: trunk
Severity: high | Keywords:
Lang: | Patch:
Platform: |
--------------------+-------------------------------------------------------
Comment(by jonathan):
pmichaud++ points out that while the patch does fix the problem at hand,
it also breaks the imo less important case of resumable exceptions. So we
need a fix that handles both before this ticket is closed.
{{{
19:48 <@pmichaud> jnthn: I'm a bit concerned about the fix for TT #1631
19:48 <@jnthn> Adding the pop_eh?
19:48 <@pmichaud> right.
19:48 <@jnthn> I don't think a CATCH should catch stuff thrown within it.
19:48 <@pmichaud> doing the pop_eh removes the handler.
19:48 <@ingy> greetings
19:48 <@jnthn> Right, that's what I intended.
19:48 <@ingy> jnthn: I agree
19:48 <@pmichaud> perhaps not, but triggering a CATCH shouldn't disable it
for
all subsequent exceptions, either.
19:49 <@pmichaud> die "1"; die "2"; CATCH { resume }
19:49 <@jnthn> Oh. :S
19:49 -!- cosimo [[email protected]] has quit [Remote host closed
the
connection]
19:50 <@jnthn> OTOH, I'd take getting basic exceptions and throws from
within
the block over resumable exceptions, if it's the choice
between
them.
19:50 <@jnthn> (until we work out a way to do both)
19:50 <@pmichaud> we have to be able to handle both, though.
19:52 <@pmichaud> we need a way to throw exceptions such that they skip
the
current handler.
19:53 <@pmichaud> (or the current handler otherwise ignores them)
}}}
--
Ticket URL: <https://trac.parrot.org/parrot/ticket/1631#comment:4>
Parrot <https://trac.parrot.org/parrot/>
Parrot Development
_______________________________________________
parrot-tickets mailing list
[email protected]
http://lists.parrot.org/mailman/listinfo/parrot-tickets
