Skip Menu |

This queue is for tickets about the Safe-Isa CPAN distribution.

Report information
The Basics
Id: 91186
Status: rejected
Priority: 0/
Queue: Safe-Isa
People
Owner: Nobody in particular
Requestors: dagolden [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Wed Dec 04 18:33:09 2013 dagolden [...] cpan.org - Ticket created
Subject: should check defined Scalar::Util::blessed()
Because "0" is a possible class name, Safe::Isa should check for a defined value from blessed(). Or, if you don't want to support stupid class names like that, then you should at least document the limitation. David
  Wed Dec 04 18:55:07 2013 ether [...] cpan.org - Correspondence added
Subject: Re: [rt.cpan.org #91186] should check defined Scalar::Util::blessed()
Date: Wed, 4 Dec 2013 15:54:54 -0800
To: David Golden via RT <bug-Safe-Isa [...] rt.cpan.org>
From: Karen Etheridge <ether [...] cpan.org>
On Wed, Dec 04, 2013 at 06:33:10PM -0500, David Golden via RT wrote: Show quoted text
> Because "0" is a possible class name, Safe::Isa should check for a defined value from blessed(). > Or, if you don't want to support stupid class names like that, then you should at least document the limitation.
Agreed. Patching.
  Wed Dec 04 18:55:07 2013 The RT System itself - Status changed from 'new' to 'open'
  Wed Dec 04 20:44:55 2013 rt-cpan [...] trout.me.uk - Correspondence added
A number of modules, for example audrey's Scalar::Defer, rely on this. This is not a bug, it's an intentional feature. Safe::Isa will not be changed.
  Wed Dec 04 20:44:56 2013 rt-cpan [...] trout.me.uk - Status changed from 'open' to 'rejected'
  Wed Dec 04 21:59:34 2013 dagolden [...] cpan.org - Correspondence added
On Wed Dec 04 20:44:55 2013, MSTROUT wrote: Show quoted text
> A number of modules, for example audrey's Scalar::Defer, rely on this. > > This is not a bug, it's an intentional feature.
Then please document it, as originally requested.