[ 
https://issues.apache.org/jira/browse/DERBY-4415?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rick Hillegas updated DERBY-4415:
---------------------------------

    Attachment: derby-4415-01-ab-customASTprinters.diff

Thanks for the quick feedback, Knut. I agree that a separate ASTVisitor 
interface is a great idea. Attaching a new, simpler version of this patch: 
derby-4415-01-ab-customASTprinters.diff.

Touches the following files:

A      java/engine/org/apache/derby/iapi/sql/compile/ASTVisitor.java

Interface for user-written node Visitors. Provides initialization and teardown 
methods.


M      java/engine/org/apache/derby/iapi/sql/conn/LanguageConnectionContext.java
M      
java/engine/org/apache/derby/impl/sql/conn/GenericLanguageConnectionContext.java

Adds methods for poking a user-written node Visitor into the compiler's state 
variable.


M      java/engine/org/apache/derby/impl/sql/GenericStatement.java

Adds calls to the user-written node Visitor if one has been supplied. The 
Visitor is called for each processing phase: parse, bind, and optimize.


> Make it easy to plug custom AST printers into the compiler
> ----------------------------------------------------------
>
>                 Key: DERBY-4415
>                 URL: https://issues.apache.org/jira/browse/DERBY-4415
>             Project: Derby
>          Issue Type: Improvement
>          Components: SQL
>            Reporter: Rick Hillegas
>            Priority: Minor
>         Attachments: ASTInspector.java, 
> derby-4415-01-aa-customASTprinters.diff, 
> derby-4415-01-ab-customASTprinters.diff, XmlASTPrinter.java
>
>
> It would be nice to be able to tell the compiler to use user-written 
> formatters for pretty-printing Derby's abstract syntax trees. For instance, 
> it would be nice to be able to plug in the AST printers attached to 
> DERBY-791, including the Eclipse-based tree grapher written by Kay Röpke.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to