Avoid the custom SimpleNode 
----------------------------

                 Key: GEOT-2510
                 URL: http://jira.codehaus.org/browse/GEOT-2510
             Project: GeoTools
          Issue Type: Task
          Components: core cql
    Affects Versions: 2.6-M1
            Reporter: Mauricio Pazos
            Assignee: Mauricio Pazos
             Fix For: 2.6-M2


Related with the "Modules cleanup" tasks the maven jjtree-javacc plugin (Mojo 
version) failed compiling the cql module. The cql module was developed 
following the javacc "Simple" strategy  which generates only one parser node 
(SimpleNode) for all parsing rules. Thus, to identify what is the semantic 
action required, the compiler gets the node identifier. This is the reason why 
the SimpleNode was customised. This approach requires an special maven 
behaviour that was introduced in the jjtee-javacc plugin developed for geotools 
project.

In order to avoid the maintain of jjtree-javacc pluguin, the parser generation 
will be changed from SIMPLE to MULTI node, so jjtree will generate one class 
for each node declared in the parsing rule (specified in the correspondent jjt 
file)  

This solution requires to change all "getType" by "if node instanceof ..."  


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Register Now for Creativity and Technology (CaT), June 3rd, NYC. CaT
is a gathering of tech-side developers & brand creativity professionals. Meet
the minds behind Google Creative Lab, Visual Complexity, Processing, & 
iPhoneDevCamp asthey present alongside digital heavyweights like Barbarian
Group, R/GA, & Big Spaceship. http://www.creativitycat.com 
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to