On Sat, Aug 11, 2001 at 09:49:12AM -0400, Lyle Brooks wrote: > Ok, who the patch pumpkin?
That's me! > I've worked on a patch for this LOAD_PERL bug. I'm not exactly sure > what the patch procedure is. Do we have one? Let me know if we do, Excellent! Thanks for looking into this. I'm not yet convinced that creating a separate module for this task is the best solution. Perhaps a simple closure might be better? Anyway, I'll look it over tonight and either apply the patch as it is, or rework the essence into some other form. Either way the hard part is done, so thanks again. > BTW, in the course of testing this patch, I ran across a configuration > option (PLUGIN_FACTORY) that I didn't see documented anywhere and only > used in two files (t/leak.t and lib/Template/Plugins.pm). I doubt that if > PLUGIN_FACTORY and LOAD_PERL were both enabled, you'd get the desired > behavior. But that's left as an exercise to the reader. Hmm, yes, there may be a bit of dead wood in there, I'll have a look around and see what needs cleaning up. Cheers A
