Replace javadoc usage of <p/> since it only breaks paragraphs when using <p>

Part of me hates the dangling nature of paragraph html tags like this, but I get that this is a special case...

Author: Col-E

recaf-core/src/main/java/software/coley/recaf/behavior/PriorityKeys.java

@@ -2,7 +2,7 @@
 
 /**
  * Default keys for {@link PrioritySortable} implementations.
- * <p/>
+ * <p>
  * Searching for usages of these keys will show what listeners/classes fire in which order.
  *
  * @author Matt Coley

recaf-core/src/main/java/software/coley/recaf/cdi/EagerInitialization.java

@@ -12,11 +12,11 @@
  * Applied to beans to enable eager initialization, which is to say they and their dependencies get created as soon as
  * possible depending on the {@link #value() value of the intended} {@link InitializationStage}. This will result in
  * the bean's {@code @Inject} annotated constructor being called.
- * <p/>
+ * <p>
  * Alternatively, you could also observe the events {@link InitializationEvent} or {@link UiInitializationEvent}
  * in a method with {@link Observes}. This would allow you to separate the initialization logic from the constructor
  * and have it reside in a separate method.
- * <p/>
+ * <p>
  * <b>NOTE:</b> Beans are not eagerly initialized while in a test environment.
  *
  * @author Matt Coley

recaf-core/src/main/java/software/coley/recaf/info/builder/JvmClassInfoBuilder.java

@@ -122,7 +122,7 @@ public JvmClassInfoBuilder(@Nonnull byte[] bytecode, int readerFlags) {
 	/**
 	 * Copies over values by reading the contents of the class file in the reader.
 	 * Calls {@link #adaptFrom(byte[], int)} with {@code flags=0}.
-	 * <p/>
+	 * <p>
 	 * <b>IMPORTANT:</b> If {@link #skipValidationChecks(boolean)} is {@code false} and validation checks are active
 	 * extra steps are taken to ensure the class is fully ASM compliant. You will want to wrap this call in a try-catch
 	 * block handling {@link Throwable} to cover any potential ASM failure.
@@ -148,7 +148,7 @@ public JvmClassInfoBuilder adaptFrom(@Nonnull byte[] code) {
 	/**
 	 * Copies over values by reading the contents of the class file in the reader.
 	 * Calls {@link #adaptFrom(ClassReader, int)} with {@code flags}.
-	 * <p/>
+	 * <p>
 	 * <b>IMPORTANT:</b> If {@link #skipValidationChecks(boolean)} is {@code false} and validation checks are active
 	 * extra steps are taken to ensure the class is fully ASM compliant. You will want to wrap this call in a try-catch
 	 * block handling {@link Throwable} to cover any potential ASM failure.
@@ -176,7 +176,7 @@ public JvmClassInfoBuilder adaptFrom(@Nonnull byte[] code, int readerFlags) {
 	/**
 	 * Copies over values by reading the contents of the class file in the reader.
 	 * Calls {@link #adaptFrom(ClassReader, int)} with {@code flags=0}.
-	 * <p/>
+	 * <p>
 	 * <b>IMPORTANT:</b> If {@link #skipValidationChecks(boolean)} is {@code false} and validation checks are active
 	 * extra steps are taken to ensure the class is fully ASM compliant. You will want to wrap this call in a try-catch
 	 * block handling {@link Throwable} to cover any potential ASM failure.
@@ -201,7 +201,7 @@ public JvmClassInfoBuilder adaptFrom(@Nonnull ClassReader reader) {
 
 	/**
 	 * Copies over values by reading the contents of the class file in the reader.
-	 * <p/>
+	 * <p>
 	 * <b>IMPORTANT:</b> If {@link #skipValidationChecks(boolean)} is {@code false} and validation checks are active
 	 * extra steps are taken to ensure the class is fully ASM compliant. You will want to wrap this call in a try-catch
 	 * block handling {@link Throwable} to cover any potential ASM failure.

recaf-core/src/main/java/software/coley/recaf/path/PathNode.java

@@ -13,7 +13,7 @@
  * A <i>"modular"</i> value type for representing <i>"paths"</i> to content in a {@link Workspace}.
  * The path must contain all data in a <i>"chain"</i> such that it can have access from most specific portion
  * all the way up to the {@link Workspace} portion.
- * <p/>
+ * <p>
  * <b>NOTE: Regarding contents in embedded resources,</b> the path result of the methods like
  * {@link Workspace#findClass(String)} will contain the root {@link WorkspaceResource} but the exact {@link Bundle}.
  * To find the exact embedded resource of a result use {@link WorkspaceResource#resolveBundleContainer(Bundle)}.
@@ -44,7 +44,7 @@ public interface PathNode<V> extends Comparable<PathNode<?>> {
 	/**
 	 * Creates a copy of the path node with this child-most node's value being looked up for a newer
 	 * value in the associated workspace.
-	 * <p/>
+	 * <p>
 	 * <b>Note:</b> A {@link WorkspacePathNode} must be present.
 	 *
 	 * @return A new path node pointing to the same location,

recaf-core/src/main/java/software/coley/recaf/path/PathNodes.java

@@ -19,11 +19,11 @@
 
 /**
  * Utility methods for constructing paths.
- * <p/>
+ * <p>
  * Generally you should use this only when the path creation is
  * a <i>"one time"</i> action. For instance, if you want to make a path to a single method, use
  * {@link #memberPath(Workspace, WorkspaceResource, Bundle, ClassInfo, ClassMember)}.
- * <p/>
+ * <p>
  * However, if you want to make a path to <i>all methods in a class</i> then you would use
  * {@link #classPath(Workspace, WorkspaceResource, Bundle, ClassInfo)} to get a {@link ClassPathNode}
  * and then use {@link ClassPathNode#child(ClassMember)} for each member. This reduces the number of redundant
@@ -352,7 +352,7 @@ public static PathNode<?> unique(@Nonnull String identifier) {
 
 	/**
 	 * A path node that just holds an arbitrary string.
-	 * <p/>
+	 * <p>
 	 * Intended for use in the UI where displayed panels are intended to be navigable for tracking, not actually
 	 * navigable in terms of their relationship to some location in a workspace.
 	 *

recaf-core/src/main/java/software/coley/recaf/services/assembler/WorkspaceFieldValueLookup.java

@@ -13,7 +13,7 @@
 
 /**
  * Field value lookup for items in a {@link Workspace}.
- * <p/>
+ * <p>
  * Its very basic, only looking at static fields default values.
  *
  * @author Matt Coley

recaf-core/src/main/java/software/coley/recaf/services/deobfuscation/transform/generic/KotlinMetadataCollectionTransformer.java

@@ -41,7 +41,7 @@
 
 /**
  * A transformer that collects kotlin metadata and offers utilities to process it.
- * <p/>
+ * <p>
  * For deobfuscating descriptors it is recommended to use the following methods in order:
  * <ol>
  *     <li>{@link #mapKtDescriptor(KtType)} / {@link #mapKtDescriptor(KtFunction)}</li>

recaf-core/src/main/java/software/coley/recaf/services/deobfuscation/transform/generic/VariableFoldingTransformer.java

@@ -198,7 +198,7 @@ public void transform(@Nonnull JvmTransformerContext context, @Nonnull Workspace
 
 	/**
 	 * Populate variable liveness for the method.
-	 * <p/>
+	 * <p>
 	 * Liveness is the set of variables that are live at each instruction.
 	 * A good reference for this can be found <a href="https://users.cs.northwestern.edu/%7Esimonec/files/Teaching/CAT/slides/DFA_part1.pdf">here</a>.
 	 *
@@ -283,7 +283,7 @@ private static void populateLiveness(@Nonnull MethodNode method,
 
 	/**
 	 * Populate variable access states for the method.
-	 * <p/>
+	 * <p>
 	 * This tracks when variables are read from and written to, which is necessary
 	 * for determining when variable copies are redundant and can be folded.
 	 *

recaf-core/src/main/java/software/coley/recaf/services/mapping/Mappings.java

@@ -25,7 +25,7 @@ public interface Mappings {
 	 * Some mapping formats do not include field types since name overloading is illegal at the source level of Java.
 	 * It's valid in the bytecode but the mapping omits this info since it isn't necessary information for mapping
 	 * that does not support name overloading.
-	 * <p/>
+	 * <p>
 	 * This is mostly only relevant for usage of {@link MappingsAdapter} which
 	 *
 	 * @return {@code true} when field mappings include the type descriptor in their lookup information.

recaf-core/src/main/java/software/coley/recaf/services/mapping/UniqueKeyMappings.java

@@ -10,11 +10,11 @@
 /**
  * A simple {@link Mappings} implementation that assumes all classes, fields, and methods are uniquely identified.
  * Mapping outputs are keyed to these unique identifiers, rather than a full {@code name + desc} pair.
- * <p/>
+ * <p>
  * This mappings implementation can be useful to pass to {@link MappingApplier} for some scenarios like
  * Minecraft mappings, where each class, field, and method are uniquely identified. Minecraft's Forge, Fabric-Yarn and
  * Mod-Coder-Pack all have <i>"intermediate"</i> names that follow this pattern.
- * <p/>
+ * <p>
  * Example use case:
  * <pre>{@code
  * MappingApplier applier = ...;