On 04/16/2012 10:10 PM, Ricardo Signes wrote: > * Father Chrysostomos via RT<perlbug-followup@perl.org> [2012-04-16T21:39:47] >> On Mon Apr 16 18:22:11 2012, sprout wrote: >>> Commit ad63d80fcd28c3 seems to have caused the first regression. It >>> apparently stops any error from being reported at all. >> >> And commit c8a14fb6c15f introduced the panic. >> >> Since this is a regression from 5.8 (and I was really hoping to get rid >> of all of those!), and since the fix is simple (I’ll probably have it >> done tonight), can this be fixed for 5.16? > > I hope so. I've just had a chance to look at the fix that Father Chrysostomos made for this and I am not happy with it at all. There is already a limit on the maximum size of any single element of a version object of 0x7FFFFFFF (see VERSION_MAX in util.c), so we should use something based on that as the upper limit rather than the magical number 10e50. This also means I need to bump the CPAN release with this fix too. Please, please cc: me on any change to the version code because I don't always see the p5p changes immediately. JohnThread Previous | Thread Next