This queue is for tickets about the UNIVERSAL-isa CPAN distribution.

Report information
The Basics
Id:
97468
Status:
open
Priority:
Low/Low
Queue:

People
Owner:
Nobody in particular
Requestors:
mst [...] shadowcat.co.uk
Cc:
AdminCc:

BugTracker
Severity:
(no value)
Broken in:
(no value)
Fixed in:
1.20140927



CC: exodist@cpan.org
Subject: Verbose warnings causes false positives
Date: Thu, 24 Jul 2014 19:37:20 +0000
To: bugs-UNIVERSAL-isa@rt.cpan.org
From: Matt S Trout <mst@shadowcat.co.uk>
Adding a ninth entry to t/warnings.t of - $warning = ''; $bar->isa('Bar'); is( $warning, '', 'No warning on actual method call' ); reveals that we *do* still warn on an actual method call. This produces fun failures like http://www.cpantesters.org/cpan/report/3ee883b4-10fa-11e4-97f6-9b514fc05230 because newer Test::Builder versions now actually use ->isa to check the types of things internally. -- Matt S Trout - Shadowcat Systems - Perl consulting with a commit bit and a clue http://shadowcat.co.uk/blog/matt-s-trout/ http://twitter.com/shadowcat_mst/ Email me now on mst (at) shadowcat.co.uk and let's chat about how our CPAN commercial support, training and consultancy packages could help your team.
Subject: Re: [rt.cpan.org #97468] Verbose warnings causes false positives
Date: Thu, 24 Jul 2014 12:48:12 -0700
To: Matt S Trout via RT <bug-UNIVERSAL-isa@rt.cpan.org>
From: Karen Etheridge <ether@cpan.org>
And when in verbose mode, the warning message isn't even correct. It should simply be "UNIVERSAL::isa called as method on <invocant>", so the developer won't think he's going crazy by triggering an impossible situation.
Subject: Re: [rt.cpan.org #97468] Verbose warnings causes false positives
Date: Thu, 24 Jul 2014 19:57:17 +0000
To: Karen Etheridge via RT <bug-UNIVERSAL-isa@rt.cpan.org>
From: Matt S Trout <mst@shadowcat.co.uk>
On Thu, Jul 24, 2014 at 03:48:32PM -0400, Karen Etheridge via RT wrote:
Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=97468 > > > > And when in verbose mode, the warning message isn't even correct. It should > simply be "UNIVERSAL::isa called as method on <invocant>", so the developer > won't think he's going crazy by triggering an impossible situation.
Um, no. That case shouldn't even warn at all. -- Matt S Trout - Shadowcat Systems - Perl consulting with a commit bit and a clue http://shadowcat.co.uk/blog/matt-s-trout/ http://twitter.com/shadowcat_mst/ Email me now on mst (at) shadowcat.co.uk and let's chat about how our CPAN commercial support, training and consultancy packages could help your team.
Subject: Re: [rt.cpan.org #97468] Verbose warnings causes false positives
Date: Thu, 24 Jul 2014 13:32:33 -0700
To: Matt S Trout via RT <bug-UNIVERSAL-isa@rt.cpan.org>
From: Karen Etheridge <ether@cpan.org>
On Thu, Jul 24, 2014 at 03:57:29PM -0400, Matt S Trout via RT wrote:
Show quoted text
> > And when in verbose mode, the warning message isn't even correct. It should > > simply be "UNIVERSAL::isa called as method on <invocant>", so the developer > > won't think he's going crazy by triggering an impossible situation.
> > Um, no. That case shouldn't even warn at all.
Ah, I see, I misunderstood what the verbose mode did -- I was thinking it warned about *all calls to isa*, regardless of the invocant -- in which case the warning that we were hitting in Test::More was actually correct.
In the absence of some sort of XS or B:: magic that I'm not aware of, there is no way to disambiguate between: UNIVERSAL::isa($obj, 'something'); and $obj->isa('something') from within UNIVERSAL::isa. *All* calls to isa during verbose mode will warn. Consequently, I think the best we can do is hack the tests so they don't fail when Test::Builder calls isa() during its own tests.
RT-Send-CC: exodist@cpan.org
UNIVERSAL-isa-1.20140824 is released.
Oops, this shouldn't be closed. The problem still persists. The only thing solved by the last release is the failing tests in the presence of newer Test::Builder.
On 2014-08-23 22:55:10, ETHER wrote:
Show quoted text
> Oops, this shouldn't be closed. The problem still persists. > > The only thing solved by the last release is the failing tests in the > presence of newer Test::Builder.
The bug, as originally described in the first message, has been fixed. Resolving.


This service runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.