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: 49450
Status: resolved
Priority: 0/
Queue: PPI

People
Owner: Nobody in particular
Requestors: wyant [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 1.206
Fixed in: (no value)



Subject: Curlys in 'use constant 1.1 { One => 1 }' parsed as block, not constructor.
Download (untitled) / with headers
text/plain 514b
PPI version 1.206 parses the curlys in 'use constant { One => 1 }' as a PPI::Structure::Constructor, thanks to special-case code in PPI::Lexer. But in 'use constant 1.1 { One => 1 }' the curlys are still parsed as a PPI::Structure::Block. SVN revision 8860 contains a proposed fix for this (tweaking the special-case code), and some extra tests in t/ppi_statement_include.t to demonstrate the behavior. Thank you very much for PPI in general, and for your time and attention on this particular matter. Tom Wyant
Download (untitled) / with headers
text/plain 246b
Realized I screwed up the testing portion of SVN revision 8860. Put t/ppi_statement_include.t back the way it was, added a '=begin testing' section for _curly in lib/PPI/Lexer.pm, and the real code to t/ppi_lexer. The whole is SVN revision 8866.
And SVN revision 8867 because in 8866 I still didn't get the whitespace correct. Sigh.
Resolved


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

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