Many changes:

  - Split off the actual agent work into a separate src package in preparation for creating separate jars. Involved a lot of renaming
  - Renamed TransformCompilationUnitDeclaration to TransformEclipseAST, as this class will also be transforming e.g. MethodDeclaration objects.
  - Expanded the patching to also patch in transform calls when the parser fills in the Statement array for existing constructors, methods, and initializers.
  - Redesigned the ClassLoaderWorkaround class quite a bit.
  - Positioning should not work correctly ('jump to method' should jump to the getter annotation).
(Apparently, Clinit objects are always fully parsed in the original run, so no need to patch anything there).

1 files changed, 62 insertions, 0 deletions

diff --git a/src/lombok/eclipse/HandleGetter_ecj.java b/src/lombok/eclipse/HandleGetter_ecj.java
new file mode 100644
index 00000000..f71376a3
--- /dev/null
+++ b/src/lombok/eclipse/HandleGetter_ecj.java
@@ -0,0 +1,62 @@
+package lombok.eclipse;
+
+import java.lang.reflect.Modifier;
+
+import lombok.transformations.TransformationsUtil;
+
+import org.eclipse.jdt.internal.compiler.ast.AbstractMethodDeclaration;
+import org.eclipse.jdt.internal.compiler.ast.Annotation;
+import org.eclipse.jdt.internal.compiler.ast.ASTNode;
+import org.eclipse.jdt.internal.compiler.ast.Expression;
+import org.eclipse.jdt.internal.compiler.ast.FieldDeclaration;
+import org.eclipse.jdt.internal.compiler.ast.MethodDeclaration;
+import org.eclipse.jdt.internal.compiler.ast.ReturnStatement;
+import org.eclipse.jdt.internal.compiler.ast.SingleNameReference;
+import org.eclipse.jdt.internal.compiler.ast.Statement;
+import org.eclipse.jdt.internal.compiler.ast.TypeDeclaration;
+import org.eclipse.jdt.internal.compiler.ast.TypeReference;
+import org.eclipse.jdt.internal.compiler.lookup.MethodScope;
+
+public class HandleGetter_ecj {
+	public void apply(Annotation annotation, TypeDeclaration type, FieldDeclaration field) {
+		TypeReference fieldType = field.type;
+		String getterName = TransformationsUtil.toGetterName(new String(field.name), nameEquals(fieldType.getTypeName(), "boolean"));
+		
+		for ( AbstractMethodDeclaration method : type.methods ) {
+			if ( method.selector != null && new String(method.selector).equals(getterName) ) return;
+		}
+		
+		MethodDeclaration method = new MethodDeclaration(type.compilationResult);
+		method.modifiers = Modifier.PUBLIC;
+		method.returnType = field.type;
+		method.annotations = null;
+		method.arguments = null;
+		method.selector = getterName.toCharArray();
+		method.binding = null;
+		method.thrownExceptions = null;
+		method.typeParameters = null;
+		method.scope = new MethodScope(type.scope, method, false);
+		method.bits |= ASTNode.Bit24;
+		Expression fieldExpression = new SingleNameReference(field.name, (field.declarationSourceStart << 32) | field.declarationSourceEnd);
+		Statement returnStatement = new ReturnStatement(fieldExpression, field.sourceStart, field.sourceEnd);
+		method.bodyStart = method.declarationSourceStart = method.sourceStart = annotation.sourceStart;
+		method.bodyEnd = method.declarationSourceEnd = method.sourceEnd = annotation.sourceEnd;
+		method.statements = new Statement[] { returnStatement };
+		AbstractMethodDeclaration[] newArray = new AbstractMethodDeclaration[type.methods.length + 1];
+		System.arraycopy(type.methods, 0, newArray, 0, type.methods.length);
+		newArray[type.methods.length] = method;
+		type.methods = newArray;
+	}
+	
+	private boolean nameEquals(char[][] typeName, String string) {
+		StringBuilder sb = new StringBuilder();
+		boolean first = true;
+		for ( char[] elem : typeName ) {
+			if ( first ) first = false;
+			else sb.append('.');
+			sb.append(elem);
+		}
+		
+		return string.contentEquals(sb);
+	}
+}