Added more docs.

Author: BrynCooke

src/main/java/com/tinkerpop/frames/modules/javahandler/JavaHandlerFactory.java

@@ -8,8 +8,16 @@
 
 /**
  * {@link JavaHandlerModule} uses this interface to create the concrete classes that will handle the method calls.
+ * Typically factories will use method.getDeclaringClass() to obtain the concrete class that is being called.
  * @author Bryn Cooke
  */
 public interface JavaHandlerFactory {
+	/**
+	 * @param graph The graph of the framed element
+	 * @param element The element being handled
+	 * @param method The method called
+	 * @param direction The drection of the edge if applicable
+	 * @return An object that contains the method being called. 
+	 */
 	public <T> T create(FramedGraph<?> graph, Element element, Method method, Direction direction);
 }

src/main/java/com/tinkerpop/frames/modules/javahandler/JavaHandlerModule.java

@@ -41,7 +41,7 @@ public class JavaHandlerModule implements Module {
 
 	private JavaHandlerFactory factory = new JavaHandlerFactory() {
 
-		
+		//We don't want to use the global class cache. Instead we cache the classes at the module level.
 		private LoadingCache<Class<?>, Class<?>> classCache = CacheBuilder.newBuilder().build(
 				new CacheLoader<Class<?>, Class<?>>() {
 
@@ -65,11 +65,10 @@ public <T> T create(final FramedGraph<?> graph, final Element element, final Met
 
 			try {
 				final Class<?> frameClass = method.getDeclaringClass();
-				Class<?> returnType = method.getReturnType();
 				Class<T> implClass = (Class<T>) classCache.get(frameClass);
 				T handler = implClass.newInstance();
 				((Proxy) handler).setHandler(new MethodHandler() {
-					private DefaultJavaHandlerImpl defaultJavahandlerImpl = new DefaultJavaHandlerImpl(graph, method, element);
+					private DefaultJavaHandlerImpl<Element> defaultJavahandlerImpl = new DefaultJavaHandlerImpl<Element>(graph, method, element);
 					private Object framedElement;
 
 					@Override