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

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

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



Subject: PPI misparses "pack'H*'" as Token::Word "pack'H" ...
Download (untitled) / with headers
text/plain 649b
The sample code turned up in Crypt::CBC. I do not know how many other cases of this sort of thing are in the wild. A proposed fix has been submitted as svn update 9292. I took the liberty of making the code hash-based, rather than adding things to the regexps, and while I was in it I added 'unpack', 'gt', 'ge', 'lt', and 'le', and tests for all the special cases. But I suspect a complete fix is going to require the name of every Perl built-in to be in the hash; whether there is a better way I know not. I think there is a B:: module that has a list of built-ins, but this would not work for (e.g.) analyzing Perl 5.010 code under Perl 5.008.
Download (untitled) / with headers
text/plain 137b
Confirming fixed in 1.215 pack'H*' PPI::Document PPI::Statement PPI::Token::Word 'pack' PPI::Token::Quote::Single ''H*''


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.