Fix javadoc errors

Author: ctrueden

src/main/java/org/scijava/Context.java

@@ -141,8 +141,9 @@ public Context(final boolean noServices, final boolean noPlugins) {
 	 * {@code new Context(DisplayService.class)}) yield the potentially confusing
 	 * warning:
 	 * </p>
-	 * <blockquote>Type safety: A generic array of Class<? extends Service> is
-	 * created for a varargs parameter</blockquote>
+	 * <blockquote>Type safety: A generic array of
+	 * {@code Class<? extends Service>} is created for a varargs
+	 * parameter</blockquote>
 	 * <p>
 	 * To avoid this, we have opted to use raw types and suppress the relevant
 	 * warnings here instead.
@@ -351,7 +352,7 @@ public Service getService(final String className) {
 	/**
 	 * Injects the application context into the given object. This does three
 	 * distinct things:
-	 * <ul>
+	 * <ol>
 	 * <li>If the given object has any non-final {@link Context} fields annotated
 	 * with @{@link Parameter}, sets the value of those fields to this context.
 	 * </li>
@@ -360,8 +361,8 @@ public Service getService(final String className) {
 	 * corresponding service available from this context.</li>
 	 * <li>Calls {@link EventService#subscribe(Object)} with the object to
 	 * register any @{@link EventHandler} annotated methods as event subscribers.
-	 * </li> .</li>
-	 * </ul>
+	 * </li>
+	 * </ol>
 	 *
 	 * @param o The object to which the context should be assigned.
 	 * @throws IllegalStateException If the object already has a context.

src/main/java/org/scijava/Identifiable.java

@@ -49,7 +49,7 @@
  * <p>
  * In this way, objects themselves do not need to be {@link Serializable}, nor
  * do multiple potentially equivalent objects need to be synthesized and then
- * compared using {@link #equals}.
+ * compared using {@link Object#equals}.
  * </p>
  * 
  * @author Curtis Rueden

src/main/java/org/scijava/SciJava.java

@@ -74,8 +74,9 @@ public SciJava(final boolean empty) {
 	 * {@code new SciJava(LogService.class)}) yield the potentially confusing
 	 * warning:
 	 * </p>
-	 * <blockquote>Type safety: A generic array of Class<? extends Service> is
-	 * created for a varargs parameter</blockquote>
+	 * <blockquote>Type safety: A generic array of
+	 * {@code Class<? extends Service>} is created for a varargs
+	 * parameter</blockquote>
 	 * <p>
 	 * To avoid this, we have opted to use raw types and suppress the relevant
 	 * warning here instead.

src/main/java/org/scijava/annotations/Index.java

@@ -45,14 +45,15 @@
 /**
  * Makes the annotation indexes accessible.
  * <p>
- * You would call it like this:<br />
- * <code>
- * for (IndexItem<MyAnnotation> item : Index.load(MyAnnotation.class)) {<br />
- *    // do something with item.annotation() and/or item.className()<br />
- * }<br />
- * </code>
+ * You would call it like this:
  * </p>
  * 
+ * <pre>
+ * for (IndexItem&lt;MyAnnotation&gt; item : Index.load(MyAnnotation.class)) {
+ *   // do something with item.annotation() and/or item.className()
+ * }
+ * </pre>
+ * 
  * @author Johannes Schindelin
  */
 public class Index<A extends Annotation> implements Iterable<IndexItem<A>> {
@@ -74,8 +75,6 @@ public static <A extends Annotation> Index<A> load(final Class<A> annotation)
 
 	/**
 	 * Loads the index of all classes annotated with the specified annotation.
-	 * <p>
-	 * </p>
 	 * 
 	 * @param annotation the annotation type
 	 * @param loader the class loader to use when loading {@link Class}-type

src/main/java/org/scijava/convert/ConversionRequest.java

@@ -33,29 +33,27 @@
 
 import java.lang.reflect.Type;
 
+import org.scijava.plugin.HandlerService;
 import org.scijava.util.GenericUtils;
 
 /**
- * Currency for use in {@link Converter} and {@link ConvertService}
- * methods.
+ * Currency for use in {@link Converter} and {@link ConvertService} methods.
  * <p>
  * {@link #ConversionRequest} provides a variety of constructors. Note that only
  * one destination type needs to be set (e.g. either a {@link Type} or a
  * {@link Class}).
  * </p>
  * <p>
- * Only the {@link Class} source needs to be set for {@link Converter}
- * lookup, such as through
- * {@link ConvertService#getHandler(ConversionRequest)}. However, to perform
- * an actual conversion, e.g. using
+ * Only the {@link Class} source needs to be set for {@link Converter} lookup,
+ * such as through {@link HandlerService#getHandler(Object)}. However, to
+ * perform an actual conversion, e.g. using
  * {@link Converter#convert(ConversionRequest)}, you must provide an
  * {@link Object} source.
  * </p>
  * <p>
- * NB: once a {@link Converter} has been acquired, the
- * {@code ConversionRequest} used for lookup can be reused to cast to the same
- * destination type, simply by updating the source object using the
- * {@link #setSourceObject(Object)} method.
+ * NB: once a {@link Converter} has been acquired, the {@code ConversionRequest}
+ * used for lookup can be reused to cast to the same destination type, simply by
+ * updating the source object using the {@link #setSourceObject(Object)} method.
  * </p>
  *
  * @author Mark Hiner

src/main/java/org/scijava/convert/ConvertService.java

@@ -66,22 +66,22 @@ public interface ConvertService extends
 	Object convert(ConversionRequest request);
 
 	/**
-	 * @see #getHandler(ConversionRequest)
+	 * @see HandlerService#supports(Object)
 	 */
 	Converter<?, ?> getHandler(Object src, Class<?> dest);
 
 	/**
-	 * @see #getHandler(ConversionRequest)
+	 * @see HandlerService#supports(Object)
 	 */
 	Converter<?, ?> getHandler(Object src, Type dest);
 
 	/**
-	 * @see #supports(ConversionRequest)
+	 * @see HandlerService#supports(Object)
 	 */
 	boolean supports(Object src, Class<?> dest);
 
 	/**
-	 * @see #supports(ConversionRequest)
+	 * @see HandlerService#supports(Object)
 	 */
 	boolean supports(Object src, Type dest);
 
@@ -106,28 +106,28 @@ public interface ConvertService extends
 	// -- Deprecated API --
 
 	/**
-	 * @see #getHandler(ConversionRequest)
+	 * @see HandlerService#getHandler(Object)
 	 * @deprecated Use {@link #getHandler(Object, Class)}
 	 */
 	@Deprecated
 	Converter<?, ?> getHandler(Class<?> src, Class<?> dest);
 
 	/**
-	 * @see #getHandler(ConversionRequest)
+	 * @see HandlerService#getHandler(Object)
 	 * @deprecated Use {@link #getHandler(Object, Type)}
 	 */
 	@Deprecated
 	Converter<?, ?> getHandler(Class<?> src, Type dest);
 
 	/**
-	 * @see #supports(ConversionRequest)
+	 * @see HandlerService#supports(Object)
 	 * @deprecated Use {@link #supports(Object, Class)}
 	 */
 	@Deprecated
 	boolean supports(Class<?> src, Class<?> dest);
 
 	/**
-	 * @see #supports(ConversionRequest)
+	 * @see HandlerService#supports(Object)
 	 * @deprecated Use {@link #supports(Object, Type)}
 	 */
 	@Deprecated

src/main/java/org/scijava/convert/Converter.java

@@ -50,9 +50,8 @@
 public interface Converter<I, O> extends HandlerPlugin<ConversionRequest> {
 
 	/**
-	 * Checks whether a given {@ConversionRequest} can be
-	 * processed, by converting the desired
-	 * {@link ConversionRequest#sourceClass()} to its
+	 * Checks whether a given {@link ConversionRequest} can be processed, by
+	 * converting the desired {@link ConversionRequest#sourceClass()} to its
 	 * {@link ConversionRequest#destClass()} or
 	 * {@link ConversionRequest#destType()}.
 	 *

src/main/java/org/scijava/input/KeyCode.java

@@ -442,7 +442,7 @@ public enum KeyCode {
 	/** TODO. */
 	DEAD_SEMIVOICED_SOUND(0x8f),
 
-	/** Ampersand ('&'). */
+	/** Ampersand ('&'). */
 	AMPERSAND(0x96),
 
 	/** Asterisk ('*'). */

src/main/java/org/scijava/menu/ShadowMenu.java

@@ -172,7 +172,8 @@ public MenuEntry getMenuEntry() {
 
 	/**
 	 * Gets how deep into the menu structure this node is. For example, "File"
-	 * would be at depth 1, whereas "Exit" (of "File>Exit") would be at depth 2.
+	 * would be at depth 1, whereas "Exit" (of "File>Exit") would be at depth
+	 * 2.
 	 */
 	public int getMenuDepth() {
 		return menuDepth;

src/main/java/org/scijava/plugin/Plugin.java

@@ -79,10 +79,10 @@
 
 	/**
 	 * Abbreviated menu path defining where the plugin is shown in the menu
-	 * structure. Uses greater than signs (>) as a separator; e.g.:
-	 * "Image > Overlay > Properties..." defines a "Properties..." menu item
-	 * within the "Overlay" submenu of the "Image" menu. Use either
-	 * {@link #menuPath} or {@link #menu} but not both.
+	 * structure. Uses greater than signs ({@code >}) as a separator; e.g.: "Image
+	 * > Overlay > Properties..." defines a "Properties..." menu item within
+	 * the "Overlay" submenu of the "Image" menu. Use either {@link #menuPath} or
+	 * {@link #menu} but not both.
 	 */
 	String menuPath() default "";