On 3/14/02 9:27 AM, "Jason van Zyl" <[EMAIL PROTECTED]> wrote:
> On Thu, 2002-03-14 at 03:23, Christian Trutz wrote: >> >> Hello velocitians, >> >> I experimentize a little with ParserVisitors on AST and I think the following >> patch is usefull: >> >> it allows processing the AST nodes with a visitor, until now: >> >> template.merge( context, writer ); >> >> now: >> >> template.merge( context, writer ); >> and >> template.merge( context, visitor ); >> >> are avaible. I think this is a trivial change, but it is usefull ... > > I actually don't like the idea of changing something so fundamental but > I would be curious if you tried any profiling to see if there was any > performance difference between using a visitor and the self-walking > tree. I can't imagine that it will be faster. To me this appears to be of academic interest - I would only be in favor of this if there was some kind of usecase that supported it. Given the model is fairly well used and tested at this point, I can't imagine what that use case would be. -- Geir Magnusson Jr. [EMAIL PROTECTED] System and Software Consulting My inner cowboy needs to yodel. -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
