Daniel Rall wrote:
> 
> "Geir Magnusson Jr." <[EMAIL PROTECTED]> writes:
> 
> > > I have added support to ASTIdentifier by adding a PythonExecutor that will
> > > try to resolve reference from anything that subclasses PyObject. This is a
> > > pretty clean fit I think and doesn't require much code. I was about to
> > > extend ASTMethod to handle the method case, but here is the thing. I don't
> > > want to create a solid dependency on Jython in the Velocity codebase. What
> > > I would like from people are ideas about how to extend the AbstractExecutor
> > > notion to be more like plugable directive and then extend this notion to
> > > the AST{Identifier, Method, Reference?}
> >
> > If you can do it in the AST, why not just make a conventionally
> > introspectable wrapper class for the python stuff...?  Then you can drop
> > that right into the context, and no one is the wiser....
> 
> This is definitely the best way to handle this.  Your wrapper could go
> into our contrib/optional section.

+1

-- 
Geir Magnusson Jr.                           [EMAIL PROTECTED]
System and Software Consulting
Developing for the web?  See http://jakarta.apache.org/velocity/
You have a genius for suggesting things I've come a cropper with!

Reply via email to