Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-Deep CPAN distribution.

Report information
The Basics
Id: 85444
Status: rejected
Priority: 0/
Queue: Test-Deep
People
Owner: Nobody in particular
Requestors: victor [...] vsespb.ru
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sun May 19 12:06:42 2013 victor [...] vsespb.ru - Ticket created
Subject: superhash() does not localize $@
superhash() does not localize $@ so, you can't write code like this ok ! defined eval {die { a => 1, b => 2}; }; cmp_deeply $@, superhashof { a => ... }; I believe this is because perl 'require' (which is used inside superhash) will reset $@. I think that can be fixed using local $@
  Tue Aug 13 16:52:16 2013 ether [...] cpan.org - Correspondence added
On 2013-05-19 09:06:42, vsespb wrote: Show quoted text
> superhash() does not localize $@ > > so, you can't write code like this > > ok ! defined eval {die { a => 1, b => 2}; }; > cmp_deeply $@, superhashof { a => ... }; > > I believe this is because perl 'require' (which is used inside superhash) > will reset $@. I think that can be fixed using local $@
I don't think this is a bug. If you care about keeping the content of $@ beyond the next expression, copy it into a local lexical yourself and then test against that. Globals like $@ are fair game for being overwritten at any later time.
  Tue Aug 13 16:52:17 2013 The RT System itself - Status changed from 'new' to 'open'
  Tue Aug 13 17:16:34 2013 victor [...] vsespb.ru - Correspondence added
From: victor [...] vsespb.ru
Well, fair enough. I would treat it as whishlist On Wed Aug 14 00:52:16 2013, ETHER wrote: Show quoted text
> On 2013-05-19 09:06:42, vsespb wrote:
> > superhash() does not localize $@ > > > > so, you can't write code like this > > > > ok ! defined eval {die { a => 1, b => 2}; }; > > cmp_deeply $@, superhashof { a => ... }; > > > > I believe this is because perl 'require' (which is used inside > > superhash) > > will reset $@. I think that can be fixed using local $@
> > > I don't think this is a bug. If you care about keeping the content of > $@ beyond the next expression, copy it into a local lexical yourself > and then test against that. Globals like $@ are fair game for being > overwritten at any later time.
  Sat Nov 30 09:51:25 2013 rjbs [...] cpan.org - Correspondence added
This kind of $@ accounting is almost certainly not going to get done. -- rjbs
  Sat Nov 30 09:51:26 2013 rjbs [...] cpan.org - Status changed from 'open' to 'rejected'