On Sun, Aug 21, 2005 at 01:59:34PM -0400, Sam Tregar wrote:
> On Sun, 21 Aug 2005, Philip M. Gollucci wrote:
> 
> > Hi all,
> > 
> > Nuff said!
> 
> Very cool!  One comment: don't bother with 'use constant' here.  The
> tiny amount of time saved by using a constant will be overwhelmed by
> the time used loading constant.pm and calling import().
> 
> > P.S.
> >   I believe its a side effect, but you need to enable
> >   PerlOptions +GlobalRequest or you get errors in the on_destory() method.
> 
> Can you confirm this and add it to the documentation?

(including the error message)

Then send me an updated patch.

Thanks Philip!

Tim.

Reply via email to