Tom Insam wrote: > Normally, that simply can't happen - we already know, having got here, > that the method exists. The (old) code was called if anything in the > method ever threw a 'method doesn't exist' error, and this is fixed, but > we could fix _that_ bug by just not looking - all errors are real errors > at that point.
Aha! The light goes on! Thanks for explaining that. Yes, you're right. > The case where we _do_ need this code is where I want to implement an > AUTOLOAD that will pretend to only implement some methods. Yes, it's a marginal case admittedly, but given that the code is already there to handle it then we might as well keep it in case anyone is relying on it. > Anyway, I have the CVS repos now, so I'll give you a patch against it to > test this situation once I see the merge go in. The code is checked in. There are some tests at the bottom of t/stash.t and t/stash-xs.t which test Dave's original patch. You may be able to tweak them a little. Cheers A _______________________________________________ templates mailing list [email protected] http://lists.template-toolkit.org/mailman/listinfo/templates
