Tels wrote: >-----BEGIN PGP SIGNED MESSAGE----- > >Moin, > >this mail has a patch to Text::Balanced as attachement. The last time I >sent this to p5p this was not applied because Text::Balanced is on CPAN, >so now I seperated the patch for Text::Balanced, and while I was at it, >added one more thing: the removal of SelfLoader. > >So this patch: > >* fixes the mistake in delayed loading of Carp, the code did use "use >Carp" while a "require Carp" is needed to not unec. load Calc. >(On blead, Calc.pm is now quite small, but older Perls have a bigger Calc, >and in any event, the code looks like it wanted to do "require" :) >* removes SelfLoader - it is not used by Text::Balanced at all > > Thanks. Applied as change 25134.
I added a croak() sub to match the carp() sub that you added. I couldn't see any reason why croak should be treated differently to carp in regard to how it requires the Carp module. I also left the $VERSION untouched at 1.95_01. The latest CPAN release is 1.95, and the version in bleadperl had already been modified compared to it. There is no need to bump the $VERSION multiple times between releases of bleadperl. ------------------------------------------------ Radan Computational Ltd. The information contained in this message and any files transmitted with it are confidential and intended for the addressee(s) only. If you have received this message in error or there are any problems, please notify the sender immediately. The unauthorized use, disclosure, copying or alteration of this message is strictly forbidden. Note that any views or opinions presented in this email are solely those of the author and do not necessarily represent those of Radan Computational Ltd. The recipient(s) of this message should check it and any attached files for viruses: Radan Computational will accept no liability for any damage caused by any virus transmitted by this email.
