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

Owner: Nobody in particular
Requestors: wyant [...]

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

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

Please report any issues with to