Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the PPI CPAN distribution.

Report information
The Basics
Id: 23788
Status: resolved
Priority: 0/
Queue: PPI
People
Owner: Nobody in particular
Requestors: perl [...] galumph.com
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 1.199_01
Fixed in: (no value)

History Show all quoted text
  Mon Dec 04 23:03:40 2006 perl [...] galumph.com - Ticket created
Subject: PPI::Statement::location() returns undef for C<({})>.
For the expression ({}), the Statement around the Constructor returns undef for location(). Output from running ppidump from the Perl::Critic tools directory like "tools/ppidump '({})'": PPI::Document PPI::Statement PPI::Structure::List ( ... ) PPI::Statement PPI::Structure::Constructor { ... } Perl::Critic repository version 1033 contains a workaround. See diff: http://tinyurl.com/twup4 perl v5.8.8, Mac OS X 10.4.8.
  Mon Dec 04 23:31:32 2006 perl [...] galumph.com - Correspondence added
From: perl [...] galumph.com
On Mon Dec 04 23:03:40 2006, clonezone wrote: Show quoted text
> For the expression ({}), the Statement around the Constructor returns > undef for location().
This also applies to C<({} )>, but not C<( {})>. In other words, the problem only exists when there is no space between the opening parenthesis and the opening brace.
  Sat Feb 20 23:43:32 2010 adamk [...] cpan.org - Correspondence added
Resolved earlier, but I've added a ->location test to ensure this doesn't regress
  Sat Feb 20 23:43:39 2010 The RT System itself - Status changed from 'new' to 'open'
  Sat Feb 20 23:43:40 2010 adamk [...] cpan.org - Status changed from 'open' to 'resolved'