1 files changed, 84 insertions, 20 deletions
diff --git a/src/lombok/javac/HandlerLibrary.java b/src/lombok/javac/HandlerLibrary.java
index 20cfed21..f73b9930 100644
--- a/src/lombok/javac/HandlerLibrary.java
+++ b/src/lombok/javac/HandlerLibrary.java
@@ -1,3 +1,24 @@
+/*
+ * Copyright © 2009 Reinier Zwitserloot and Roel Spilker.
+ * 
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ * 
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ * 
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
 package lombok.javac;
 
 import java.lang.annotation.Annotation;
@@ -10,7 +31,6 @@ import java.util.ServiceConfigurationError;
 import java.util.ServiceLoader;
 
 import javax.annotation.processing.Messager;
-import javax.lang.model.element.TypeElement;
 import javax.tools.Diagnostic;
 
 import lombok.core.PrintAST;
@@ -22,7 +42,12 @@ import lombok.core.AnnotationValues.AnnotationValueDecodeFail;
 import com.sun.tools.javac.tree.JCTree.JCAnnotation;
 import com.sun.tools.javac.tree.JCTree.JCCompilationUnit;
 
-
+/**
+ * This class tracks 'handlers' and knows how to invoke them for any given AST node.
+ * 
+ * This class can find the handlers (via SPI discovery) and will set up the given AST node, such as
+ * building an AnnotationValues instance.
+ */
 public class HandlerLibrary {
 	private final TypeLibrary typeLibrary = new TypeLibrary();
 	private final Map<String, AnnotationHandlerContainer<?>> annotationHandlers = new HashMap<String, AnnotationHandlerContainer<?>>();
@@ -30,6 +55,10 @@ public class HandlerLibrary {
 	private final Messager messager;
 	private boolean skipPrintAST = true;
 	
+	/**
+	 * Creates a new HandlerLibrary that will report any problems or errors to the provided messager.
+	 * You probably want to use {@link #load(Messager)} instead.
+	 */
 	public HandlerLibrary(Messager messager) {
 		this.messager = messager;
 	}
@@ -48,6 +77,11 @@ public class HandlerLibrary {
 		}
 	}
 	
+	/**
+	 * Creates a new HandlerLibrary that will report any problems or errors to the provided messager,
+	 * then uses SPI discovery to load all annotation and visitor based handlers so that future calls
+	 * to the handle methods will defer to these handlers.
+	 */
 	public static HandlerLibrary load(Messager messager) {
 		HandlerLibrary library = new HandlerLibrary(messager);
 		
@@ -57,6 +91,7 @@ public class HandlerLibrary {
 		return library;
 	}
 	
+	/** Uses SPI Discovery to find implementations of {@link JavacAnnotationHandler}. */
 	@SuppressWarnings("unchecked")
 	private static void loadAnnotationHandlers(HandlerLibrary lib) {
 		//No, that seemingly superfluous reference to JavacAnnotationHandler's classloader is not in fact superfluous!
@@ -78,36 +113,58 @@ public class HandlerLibrary {
 		}
 	}
 	
+	/** Uses SPI Discovery to find implementations of {@link JavacASTVisitor}. */
+	private static void loadVisitorHandlers(HandlerLibrary lib) {
+		//No, that seemingly superfluous reference to JavacASTVisitor's classloader is not in fact superfluous!
+		Iterator<JavacASTVisitor> it = ServiceLoader.load(JavacASTVisitor.class,
+				JavacASTVisitor.class.getClassLoader()).iterator();
+		while ( it.hasNext() ) {
+			try {
+				JavacASTVisitor handler = it.next();
+				lib.visitorHandlers.add(handler);
+			} catch ( ServiceConfigurationError e ) {
+				lib.javacWarning("Can't load Lombok visitor handler for javac", e);
+			}
+		}
+	}
+	
+	/** Generates a warning in the Messager that was used to initialize this HandlerLibrary. */
 	public void javacWarning(String message) {
 		javacWarning(message, null);
 	}
 	
+	/** Generates a warning in the Messager that was used to initialize this HandlerLibrary. */
 	public void javacWarning(String message, Throwable t) {
 		messager.printMessage(Diagnostic.Kind.WARNING, message + (t == null ? "" : (": " + t)));
 	}
 	
+	/** Generates an error in the Messager that was used to initialize this HandlerLibrary. */
 	public void javacError(String message) {
 		javacWarning(message, null);
 	}
 	
+	/** Generates an error in the Messager that was used to initialize this HandlerLibrary. */
 	public void javacError(String message, Throwable t) {
 		messager.printMessage(Diagnostic.Kind.ERROR, message + (t == null ? "" : (": " + t)));
 	}
 	
-	private static void loadVisitorHandlers(HandlerLibrary lib) {
-		//No, that seemingly superfluous reference to JavacASTVisitor's classloader is not in fact superfluous!
-		Iterator<JavacASTVisitor> it = ServiceLoader.load(JavacASTVisitor.class,
-				JavacASTVisitor.class.getClassLoader()).iterator();
-		while ( it.hasNext() ) {
-			try {
-				JavacASTVisitor handler = it.next();
-				lib.visitorHandlers.add(handler);
-			} catch ( ServiceConfigurationError e ) {
-				lib.javacWarning("Can't load Lombok visitor handler for javac", e);
-			}
-		}
-	}
-	
+	/**
+	 * Handles the provided annotation node by first finding a qualifying instance of
+	 * {@link JavacAnnotationHandler} and if one exists, calling it with a freshly cooked up
+	 * instance of {@link AnnotationValues}.
+	 * 
+	 * Note that depending on the printASTOnly flag, the {@link lombok.core.PrintAST} annotation
+	 * will either be silently skipped, or everything that isn't <code>PrintAST</code> will be skipped.
+	 * 
+	 * The HandlerLibrary will attempt to guess if the given annotation node represents a lombok annotation.
+	 * For example, if <code>lombok.*</code> is in the import list, then this method will guess that
+	 * <code>Getter</code> refers to <code>lombok.Getter</code>, presuming that {@link lombok.javac.handlers.HandleGetter}
+	 * has been loaded.
+	 * 
+	 * @param unit The Compilation Unit that contains the Annotation AST Node.
+	 * @param node The Lombok AST Node representing the Annotation AST Node.
+	 * @param annotation 'node.get()' - convenience parameter.
+	 */
 	public boolean handleAnnotation(JCCompilationUnit unit, JavacAST.Node node, JCAnnotation annotation) {
 		TypeResolver resolver = new TypeResolver(typeLibrary, node.getPackageDeclaration(), node.getImportStatements());
 		String rawType = annotation.annotationType.toString();
@@ -130,6 +187,9 @@ public class HandlerLibrary {
 		return handled;
 	}
 	
+	/**
+	 * Will call all registered {@link JavacASTVisitor} instances.
+	 */
 	public void callASTVisitors(JavacAST ast) {
 		for ( JavacASTVisitor visitor : visitorHandlers ) try {
 			ast.traverse(visitor);
@@ -138,14 +198,18 @@ public class HandlerLibrary {
 		}
 	}
 	
-	public boolean hasHandlerFor(TypeElement annotationType) {
-		return annotationHandlers.containsKey(annotationType.getQualifiedName().toString());
-	}
-	
+	/**
+	 * Lombok does not currently support triggering annotations in a specified order; the order is essentially
+	 * random right now. This lack of order is particularly annoying for the <code>PrintAST</code> annotation,
+	 * which is almost always intended to run last. Hence, this hack, which lets it in fact run last.
+	 * 
+	 * {@see #skipAllButPrintAST}
+	 */
 	public void skipPrintAST() {
 		skipPrintAST = true;
 	}
 	
+	/** {@see #skipPrintAST} */
 	public void skipAllButPrintAST() {
 		skipPrintAST = false;
 	}