#type in FASTJavaVarDeclStatement should be a FASTJavaTypeName

[NicolasAnquetil]

it is currently defined as FASTJavaVariableExpression which is twice wrong

• this is a Type, not a variable (see FASTJavaTypeExpression is a subclass of FASTJavaVariableExpression ?! #39)
• this is not an expression but a FASTJavaTypeName

[Gabriel-Darbord]

It should be a TypeExpression, because only ClassTypeExpressions use TypeName.

JavaSmaCCProgramNodeImporterVisitor parseCodeMethodString: 'void f() { int a; }'

JavaSmaCCProgramNodeImporterVisitor parseCodeMethodString: 'void f() { String a; }'

[NicolasAnquetil]

humm not sure I got your point.
What I meant was:

• the type in FASTJavaVarDeclStatement is not an expression (it is not evaluated at exec time but at compile time
• it must be a type name "hardcoded"

So the type of a FASTJavaVarDeclStatement is a FASTJavaTypeName

[Gabriel-Darbord]

I'm not arguing about the semantics, whether or not it is an "expression."
Currently, among nodes representing a type, only ClassTypeExpression uses TypeName.
For all other types (array and primitives), they don't need to use TypeName because the name is inferred from the node type.
For example, BooleanTypeExpression is boolean, IntTypeExpression is int, etc.
Whereas ClassTypeExpressions need a TypeName to know what class is being referenced.