Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-MinimumVersion CPAN distribution.

Report information
The Basics
Id: 84650
Status: resolved
Priority: 0/
Queue: Test-MinimumVersion

People
Owner: Nobody in particular
Requestors: tokuhirom+cpan [...] gmail.com
Cc:
AdminCc:

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



Subject: Can't testing too large file.
Perl::MinimumVersion->new returns undef if PPI parsing failed. It happen when parsing too large file. Then, $pmv becomes undef. It causes exception at line `my $explicit_minimum = $pmv->minimum_explicit_version || 0;`.
On 2013-04-15 17:02:01, TOKUHIROM wrote: Show quoted text
> Perl::MinimumVersion->new returns undef if PPI parsing failed. > It happen when parsing too large file. > Then, $pmv becomes undef. It causes exception at line `my > $explicit_minimum = $pmv->minimum_explicit_version || 0;`.
It would be useful to diag() this ("PPI failed to scan lib/Foo.pm: $!"), so the user knows that some files weren't scanned.
We now fail on documents we can't parse. We've also added a "skip" option to the all-file-checker. If it's not a help, please let me know, preferably via GitHub issues! -- rjbs