Description André Pankraz 2011-10-15 19:17:47 EDT

Build Identifier: Build id: M20110909-1335

I build an AST with the API.
If I use PackageDeclaration.annotations().add... and CompilationUnit.rewrite() I get invalid source code because of the missing space.

In ASTRewriteFlattener.visit(PackageDeclaration node) is this line wrong:

			visitList(node, PackageDeclaration.ANNOTATIONS_PROPERTY, String.valueOf(' '));

correct:

	public boolean visit(PackageDeclaration node) {
		if (node.getAST().apiLevel() >= AST.JLS3) {
			ASTNode javadoc = getChildNode(node, PackageDeclaration.JAVADOC_PROPERTY);
			if (javadoc != null) {
				javadoc.accept(this);
			}
			visitList(node, PackageDeclaration.ANNOTATIONS_PROPERTY, String.valueOf(' '), Util.EMPTY_STRING, String.valueOf(' '));
		}
		this.result.append("package "); //$NON-NLS-1$
		getChildNode(node, PackageDeclaration.NAME_PROPERTY).accept(this);
		this.result.append(';');
		return false;
	}


Reproducible: Always

Steps to Reproduce:
1. Build AST with API via CU.recordModifications()
2. Add annotation to PackageDeclaration
3. CU.rewriteAST() to document -> result invalid