Skip Menu |

This queue is for tickets about the POE CPAN distribution.

Report information
The Basics
Id: 79886
Status: resolved
Priority: 0/
Queue: POE
People
Owner: Nobody in particular
Requestors: liste [...] awale.qc.ca
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Sep 27 15:00:26 2012 liste [...] awale.qc.ca - Ticket created
Subject: patch for SIG{__DIE__} behaviour
Date: Wed, 26 Sep 2012 10:46:27 -0400
To: "poe [...] perl.org" <poe [...] perl.org>
From: Philip Gwyn <liste [...] awale.qc.ca>
Hello, This is a patch that fixes $SIG{__DIE__} handling for perl 5.8.8. While __DIE__ starts out as undef, if you assign undef to __DIE__ in 5.8.8, it becomes an empty string (""). This patch includes a regression test. -Philip

Message body is not shown because sender requested not to inline it.

  Thu Sep 27 15:40:34 2012 RCAPUTO [...] cpan.org - Correspondence added
Thanks for the test case and patch. I applied a modified version as change 616bb1050e8f01150c6db1a02570d6dff9c0320c The modifications include using a dummy handler instead of undef/'' for the handler during callbacks. This lets us test whether the callback intended to set the handler false and honor it if so. I've modified the test case accordingly. It passes on my system, but I don't know whether it passes on 5.8.8. My fingers are crossed!
  Thu Sep 27 15:40:36 2012 The RT System itself - Status changed from 'new' to 'open'
  Thu Sep 27 15:40:36 2012 RCAPUTO [...] cpan.org - Status changed from 'open' to 'resolved'