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: 64747
Status: resolved
Priority: 0/
Queue: PPI
People
Owner: Nobody in particular
Requestors: siracusa [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Jan 13 09:59:40 2011 siracusa [...] gmail.com - Ticket created
Subject: Large __DATA__ sections quickly hit source code size limit
The hard-coded limit of 1048576 bytes for source code (in PPI::Tokenizer) is easily tripped by a small source file with a large __DATA__ section. It would be nice if one of the following things cold be done: * Replace the hard-coded 1048576 byte limit with something that can be overridden with an environment variable or package-scoped variable or something. ...or... * Disregard __DATA__ sections when enforcing source code size limits. ...or perhaps you could do both. Thanks.
  Fri May 20 11:35:53 2011 DROLSKY [...] cpan.org - Correspondence added
On Thu Jan 13 09:59:40 2011, JSIRACUSA wrote: Show quoted text
> The hard-coded limit of 1048576 bytes for source code (in > PPI::Tokenizer) is easily tripped by a > small source file with a large __DATA__ section. It would be nice if > one of the following things > cold be done: > > * Replace the hard-coded 1048576 byte limit with something that can be > overridden with an > environment variable or package-scoped variable or something. > > ...or... > > * Disregard __DATA__ sections when enforcing source code size limits. > > ...or perhaps you could do both. Thanks.
This is also problem for something I'm working on. I'm not sure how this limit was determined. For yucks, I changed it and ran the code using PPI (a dzil plugin). It worked fine.
  Fri May 20 11:35:54 2011 The RT System itself - Status changed from 'new' to 'open'
  Tue Jun 05 09:07:04 2012 GENSYM [...] cpan.org - Correspondence added
I would like to use PPI to strip off all POD and comments from a fatpacked (App::FatPacker) file to achieve a smaller size, but the hard coded limit prevents this...
  Sat Aug 16 16:05:19 2014 MITHALDU [...] cpan.org - Correspondence added
With the latest release of PPI this limit has been removed: https://metacpan.org/release/PPI Please let me know if you have further issues. :)
  Sat Aug 16 16:05:20 2014 MITHALDU [...] cpan.org - Status changed from 'open' to 'resolved'