fix javadoc errors "reference not found" and tag position related errors

See #1392

Author: EcljpseB0T

ds/org.eclipse.pde.ds.annotations/src/org/eclipse/pde/ds/internal/annotations/ComponentPropertyTester.java

@@ -31,8 +31,9 @@
 import org.eclipse.pde.internal.ds.core.IDSConstants;
 
 /**
- * Tests if a type (or any nested type) has a {@link Component} annotation with
- * no <code>name</code> attribute.
+ * Tests if a type (or any nested type) has a
+ * {@link org.osgi.service.component.annotations.Component} annotation with no
+ * <code>name</code> attribute.
  */
 @SuppressWarnings("restriction")
 public class ComponentPropertyTester extends PropertyTester {

ds/org.eclipse.pde.ds.core/src/org/eclipse/pde/internal/ds/core/IDSModel.java

@@ -23,7 +23,7 @@
  *
  * @since 3.4
  * @see IDSComponent
- * @see IDSFactory
+ * @see IDSDocumentFactory
  */
 public interface IDSModel extends IModelChangeProvider, IModel {
 

ui/org.eclipse.pde.bnd.ui/src/org/eclipse/pde/bnd/ui/model/repo/RepositoryTreeLabelProvider.java

@@ -159,10 +159,11 @@ public void update(ViewerCell cell) {
 
 	/**
 	 * Return the text to be shown as a tooltip.
-	 * <p/>
+	 * <p>
 	 * TODO allow markdown to be used. Not sure how to create a rich text
 	 * tooltip though. Would also be nice if we could copy/paste from the
 	 * tooltip like in the JDT.
+	 * </p>
 	 */
 	@Override
 	public String getToolTipText(Object element) {

ui/org.eclipse.pde.launching/src/org/eclipse/pde/internal/launching/launcher/LaunchArgumentsHelper.java

@@ -63,7 +63,7 @@ public class LaunchArgumentsHelper {
 
 	/**
 	 * Returns the location that will be used as the workspace when launching or
-	 * an empty string if the user has specified the <code>-data &#64none</code>
+	 * an empty string if the user has specified the {@code -data @none}
 	 * argument for no workspace.  Will replace variables, so this method should
 	 * only be called when variable substitution (may prompt the user) is appropriate.
 	 *

ui/org.eclipse.pde.runtime/src/org/eclipse/pde/internal/runtime/registry/model/Property.java

@@ -56,16 +56,17 @@ public void setValue(String value) {
 	 *
 	 * <p>
 	 * The {@link Property}s are compared according to the following rules:
+	 * </p>
 	 * <ul>
 	 * <li>objectClass is always less than everything else</li>
 	 * <li>properties with names starting with "service." are considered "less"
 	 * than other properties.</li>
 	 * <li>regular properties are considered "more" than other properties</li>
 	 * </ul>
-	 * </p>
 	 *
 	 * <p>
 	 * When sorting an array of properties with the following code:
+	 * </p>
 	 *
 	 * <pre>
 	 * Property[] properties = ...;
@@ -78,7 +79,6 @@ public void setValue(String value) {
 	 * <li>service.id</li>
 	 * <li>service.id</li>
 	 * </ul>
-	 * </p>
 	 *
 	 * @param other
 	 *            other property to be compared against

ui/org.eclipse.pde.ui.tests/src/org/eclipse/pde/ui/tests/target/TargetDefinitionResolutionTests.java

@@ -70,7 +70,7 @@ public void testInvalidBundleContainers() throws Exception {
 	 * Tests that if we find a bundle with a bad or missing manifest when
 	 * resolving we create the correct status.
 	 *
-	 * @see TargetBundle.STATUS_INVALID_MANIFEST
+	 * @see TargetBundle#STATUS_INVALID_MANIFEST
 	 */
 	@Test
 	public void testInvalidManifest() throws Exception {