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.
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
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