Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 36080
Status: resolved
Priority: 0/
Queue: Perl-Critic
People
Owner: Nobody in particular
Requestors: EDAVIS [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.082
Fixed in: (no value)

History Show all quoted text
  Wed May 21 12:15:17 2008 EDAVIS [...] cpan.org - Ticket created
Subject: ControlStructures::ProhibitUnreachableCode false positive with //
This test program: #!/usr/bin/perl use warnings; use strict; use 5.010; my $home = $ENV{HOME} // die "HOME not set"; say 'hello'; produces a false positive from ControlStructures::ProhibitUnreachableCode: % perlcritic -4 test Unreachable code at line 7, column 1. Consider removing it. (Severity: 4) perl 5.10.0 on Fedora 9.
  Wed May 21 15:52:07 2008 perl [...] galumph.com - Correspondence added
Subject: Re: [rt.cpan.org #36080] ControlStructures::ProhibitUnreachableCode false positive with //
Date: Wed, 21 May 2008 14:51:38 -0500
To: bug-Perl-Critic [...] rt.cpan.org
From: Elliot Shank <perl [...] galumph.com>
EDAVIS via RT wrote: Show quoted text
> This test program: > > #!/usr/bin/perl > use warnings; > use strict; > use 5.010; > my $home = $ENV{HOME} // die "HOME not set"; > say 'hello'; > > produces a false positive from ControlStructures::ProhibitUnreachableCode: > > % perlcritic -4 test > Unreachable code at line 7, column 1. Consider removing it. (Severity: 4)
Confirmed. I've checked in a failing test for this. I'll have a look at it later.
  Wed May 21 15:52:10 2008 The RT System itself - Status changed from 'new' to 'open'
  Wed May 21 23:41:55 2008 perl [...] galumph.com - Correspondence added
Subject: Re: [rt.cpan.org #36080] ControlStructures::ProhibitUnreachableCode false positive with //
Date: Wed, 21 May 2008 22:41:40 -0500
To: bug-Perl-Critic [...] rt.cpan.org
From: Elliot Shank <perl [...] galumph.com>
Elliot Shank wrote: Show quoted text
> EDAVIS via RT wrote:
>> This test program: >> >> #!/usr/bin/perl >> use warnings; >> use strict; >> use 5.010; >> my $home = $ENV{HOME} // die "HOME not set"; >> say 'hello'; >> >> produces a false positive from >> ControlStructures::ProhibitUnreachableCode: >> >> % perlcritic -4 test >> Unreachable code at line 7, column 1. Consider removing it. >> (Severity: 4)
> > Confirmed. I've checked in a failing test for this. I'll have a look > at it later.
Fix checked in. It'll be in the next release.
  Thu May 22 04:17:13 2008 EDAVIS [...] cpan.org - Correspondence added
Show quoted text
>Fix checked in. It'll be in the next release.
Thanks!
  Sat May 24 16:19:05 2008 perl [...] galumph.com - Correspondence added
Subject: Re: [rt.cpan.org #36080] ControlStructures::ProhibitUnreachableCode false positive with //
Date: Sat, 24 May 2008 15:18:49 -0500
To: bug-Perl-Critic [...] rt.cpan.org
From: Elliot Shank <perl [...] galumph.com>
Released in 1.084 today.
  Tue May 27 12:57:25 2008 thaljef [...] cpan.org - Status changed from 'open' to 'resolved'