Summary: | FormalParameter in JDT DOM/AST documentation | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Jean-Claude Royer <Jean-Claude.Royer> | ||||
Component: | Core | Assignee: | Olivier Thomann <Olivier_Thomann> | ||||
Status: | VERIFIED FIXED | QA Contact: | |||||
Severity: | trivial | ||||||
Priority: | P3 | CC: | daniel_megert, jarthana, markus.kell.r, Olivier_Thomann | ||||
Version: | 3.6 | ||||||
Target Milestone: | 3.6 M6 | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Jean-Claude Royer
2009-11-23 08:58:57 EST
> In org.eclipse.jdt.core.dom, I found a mistake in the API documentation > CatchClause, MethodDeclaration and EnhancedForStatement doc headings are > refeering in their headings to FormalParameter ASTNode which did not exist. > In place the accessors say that it is SingleVariableDeclaration These sections of AST node Javadocs refer to the grammar in the Java Language Specification (JLS2 and JLS3). Some node classes have been named after the nonterminals in the grammar, but there's no 1:1 mapping between the grammar and the AST node types. > Another detail is that: the doc heading of TypeDeclaration refers to > Classdeclaration and InterfaceDeclaration which contains some > *BodyDeclarations. > But there is nothing to get/set the Interface/Class BodyDeclaration, the > getters are getFields/getMethods. See AbstractTypeDeclaration#bodyDeclarations() in the super class. (In reply to comment #1) > These sections of AST node Javadocs refer to the grammar in the Java Language > Specification (JLS2 and JLS3). Some node classes have been named after the > nonterminals in the grammar, but there's no 1:1 mapping between the grammar and > the AST node types. In this we should refrain from referencing directly grammar rules, but rather reference the correspoding ASTNode. At least it should be clear when the reference concerns a node class name or a grammar rule name. Created attachment 161128 [details]
Proposed fix
Released for 3.6M6. Verified for 3.6M6 by code inspection (Javadoc) |