Close out list elements

gselzer · ctrueden · commit 2a680debf77e · 2022-07-01T16:21:57.000-05:00
diff --git a/imagej/imagej-ops2/src/main/java/net/imagej/ops2/filter/tubeness/DefaultTubeness.java b/imagej/imagej-ops2/src/main/java/net/imagej/ops2/filter/tubeness/DefaultTubeness.java
@@ -65,17 +65,17 @@
  * the large eigenvalue is negative, we return its absolute value and otherwise
  * return 0.
  * <ul>
- * <li>Source image is filtered first by a gaussian with 𝜎 that sets its scale.
- * <li>The the Hessian matrix is calculated for each pixel.
+ * <li>Source image is filtered first by a gaussian with 𝜎 that sets its scale.</li>
+ * <li>The the Hessian matrix is calculated for each pixel.</li>
  * <li>We yield the eigenvalues of the Hessian matrix. The output of the
- * tubeness filter is a combination of these eigenvalues:
+ * tubeness filter is a combination of these eigenvalues:</li>
  * <ul>
  * <li>in 2D where <code>λ₂</code> is the largest eigenvalue:
  * <code>out = 𝜎 × 𝜎 × |λ₂|</code> if <code>λ₂</code> is negative, 0
- * otherwise.
+ * otherwise.</li>
  * <li>in 3D where <code>λ₂</code> and <code>λ₃</code> are the largest
  * eigenvalues:, <code>out = 𝜎 × 𝜎 × sqrt( λ₂ * λ₃ )</code> if <code>λ₂</code>
- * and <code>λ₃</code> are negative, 0 otherwise.
+ * and <code>λ₃</code> are negative, 0 otherwise.</li>
  * </ul>
  * </ul>
  * This results in enhancing filaments of roughly <code>𝜎 / sqrt(d)</code>
diff --git a/scijava/scijava-ops-api/src/main/java/org/scijava/ops/api/OpHistory.java b/scijava/scijava-ops-api/src/main/java/org/scijava/ops/api/OpHistory.java
@@ -7,9 +7,9 @@
  * Log describing each execution of an Op. This class is designed to answer two
  * questions:
  * <ol>
- * <li>What Op(s) produced and/or mutated this output?
+ * <li>What Op(s) produced and/or mutated this output?</li>
  * <li>Given an {@link Object} op, what {@link OpInfo}s (including dependencies)
- * were utilized to implement that Op's functionality?
+ * were utilized to implement that Op's functionality?</li>
  * </ol>
  * The answers to these two questions allow users to produce an entire
  * {@code List<Graph<OpInfo>>}, containing all of the information needed to
diff --git a/scijava/scijava-ops-api/src/main/java/org/scijava/ops/api/features/DependencyMatchingException.java b/scijava/scijava-ops-api/src/main/java/org/scijava/ops/api/features/DependencyMatchingException.java
@@ -12,8 +12,8 @@
  * {@code C} is found to satisfy {@code B}. This will result in an
  * {@link OpMatchingException} thrown for Op {@code A}. We would want to know:
  * <ol>
- * <li>that the request for {@code A} was not fulfilled
- * <li>that no match could be found for {@code C}, which was the cause for (1)
+ * <li>that the request for {@code A} was not fulfilled</li>
+ * <li>that no match could be found for {@code C}, which was the cause for (1)</li>
  * </ol>
  * This logic can be generalized for an arbitrarily long Op chain.
  *
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/impl/DefaultOpHistory.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/impl/DefaultOpHistory.java
@@ -17,9 +17,9 @@
  * questions:
  * <ol>
  * <li>Given an {@link Object} output (e.g. a {@code List<String>}), what Op(s)
- * mutated that output?
+ * mutated that output?</li>
  * <li>Given an {@link Object} op, what {@link OpInfo}s were utilized to
- * implement that Op's functionality?
+ * implement that Op's functionality?</li>
  * </ol>
  * The answers to these two questions allow users to produce an entire
  * {@code List<Graph<OpInfo>>}, containing all of the information needed to
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/impl/LambdaTypeBaker.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/impl/LambdaTypeBaker.java
@@ -87,8 +87,8 @@ private static <T> void ensureImplementation(T originalOp, Type reifiedType) {
 	/**
 	 * Creates a Class given an Op and the reified {@link Type}. This class:
 	 * <ul>
-	 * <li>is of the same functional type as the given Op
-	 * <li>knows its generic type by nature of being a {@link GenericTyped}
+	 * <li>is of the same functional type as the given Op</li>
+	 * <li>knows its generic type by nature of being a {@link GenericTyped}</li>
 	 * </ul>
 	 * 
 	 * @param originalOp - the Op that will be simplified
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/AdaptationMatchingRoutine.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/AdaptationMatchingRoutine.java
@@ -56,9 +56,9 @@ public void checkSuitability(MatchingConditions conditions)
 	 * examine the applicability of all {@code adapt} Ops with the correct output
 	 * type. We need to check all of them because we do not know whether:
 	 * <ul>
-	 * <li>The dependencies will exist for a particular {@code adapt} Op
+	 * <li>The dependencies will exist for a particular {@code adapt} Op</li>
 	 * <li>The Op we want exists with the correct type for the input of the
-	 * {@code adapt} Op.
+	 * {@code adapt} Op.</li>
 	 * </ul>
 	 * 
 	 * @param conditions the {@link MatchingConditions} the return must satisfy
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/OpAdaptationInfo.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/OpAdaptationInfo.java
@@ -154,9 +154,9 @@ public String version() {
 	 * For an adapted Op, we define the implementation name as the concatenation
 	 * of:
 	 * <ol>
-	 * <li>The signature of the <b>adaptor</b> {@link InfoChain}
-	 * <li>The adaptation delimiter
-	 * <li>The implementation name of the <b>original info</b>
+	 * <li>The signature of the <b>adaptor</b> {@link InfoChain}</li>
+	 * <li>The adaptation delimiter</li>
+	 * <li>The implementation name of the <b>original info</b></li>
 	 * </ol>
 	 * <p>
 	 * For example, for a source {@code com.example.foo.Bar@1.0.0} with adaptor
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/OpClassInfo.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/OpClassInfo.java
@@ -210,9 +210,9 @@ public String version() {
 	 * For a {@link Class}, we define the implementation as the concatenation
 	 * of:
 	 * <ol>
-	 * <li>The fully qualified name of the class
+	 * <li>The fully qualified name of the class</li>
 	 * <li>The version of the class containing the field, with a preceding
-	 * {@code @}
+	 * {@code @}</li>
 	 * </ol>
 	 * <p>
 	 * For example, for a field class {@code com.example.foo.Bar}, you might have
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/OpFieldInfo.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/OpFieldInfo.java
@@ -206,10 +206,10 @@ public String version() {
 	 * For an {@link OpField}, we define the implementation as the concatenation
 	 * of:
 	 * <ol>
-	 * <li>The fully qualified name of the class containing the field
-	 * <li>The method field
+	 * <li>The fully qualified name of the class containing the field</li>
+	 * <li>The method field</li>
 	 * <li>The version of the class containing the field, with a preceding
-	 * {@code @}
+	 * {@code @}</li>
 	 * </ol>
 	 * <p>
 	 * For example, for a field {@code baz} in class
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/OpMethodInfo.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/matcher/impl/OpMethodInfo.java
@@ -406,11 +406,11 @@ public String version() {
 	 * For an {@link OpMethod}, we define the implementation as the concatenation
 	 * of:
 	 * <ol>
-	 * <li>The fully qualified name of the class containing the method
-	 * <li>The method name
-	 * <li>The method parameters
+	 * <li>The fully qualified name of the class containing the method</li>
+	 * <li>The method name</li>
+	 * <li>The method parameters</li>
 	 * <li>The version of the class containing the method, with a preceding
-	 * {@code @}
+	 * {@code @}</li>
 	 * </ol>
 	 * <p>
 	 * For example, for a method {@code baz(Double in1, String in2)} in class
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/simplify/SimplificationMetadata.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/simplify/SimplificationMetadata.java
@@ -201,11 +201,11 @@ public InfoChain copyOpChain() {
 	/**
 	 * The constructor to the simplified Op takes:
 	 * <ol>
-	 * <li>All input simplifiers
-	 * <li>All input focusers
-	 * <li>The output simplifier
-	 * <li>The output focuser
-	 * <li>The original Op
+	 * <li>All input simplifiers</li>
+	 * <li>All input focusers</li>
+	 * <li>The output simplifier</li>
+	 * <li>The output focuser</li>
+	 * <li>The original Op</li>
 	 * </ol>
 	 * This Op returns the {@link Class}es of those arguments <b>in that
 	 * order</b>. Note that the {@link Class} of all arugments other than the
@@ -231,11 +231,11 @@ public Class<?>[] constructorClasses() {
 	/**
 	 * The constructor to the simplified Op takes:
 	 * <ol>
-	 * <li>All input simplifiers
-	 * <li>All input focusers
-	 * <li>The output simplifier
-	 * <li>The output focuser
-	 * <li>The original Op
+	 * <li>All input simplifiers</li>
+	 * <li>All input focusers</li>
+	 * <li>The output simplifier</li>
+	 * <li>The output focuser</li>
+	 * <li>The original Op</li>
 	 * </ol>
 	 * This Op returns those arguments <b>in that order</b>.
 	 * 
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/simplify/SimplificationUtils.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/simplify/SimplificationUtils.java
@@ -47,9 +47,9 @@ public class SimplificationUtils {
 	 * {@link SimplifiedOpRef}s. This method assumes that:
 	 * <ul>
 	 * <li>{@code originalOpRefType} is (or is a subtype of) some
-	 * {@link FunctionalInterface}
+	 * {@link FunctionalInterface}</li>
 	 * <li>all {@link TypeVariable}s declared by that {@code FunctionalInterface}
-	 * are present in the signature of that interface's single abstract method.
+	 * are present in the signature of that interface's single abstract method.</li>
 	 * </ul>
 	 * 
 	 * @param originalOpType - the {@link Type} declared by the source
@@ -143,8 +143,8 @@ public static int findMutableArgIndex(Class<?> c) {
 	 * {@code inferFrom}. We can then use these mappings to resolve the type
 	 * variables of {@code unresolvedType}. This method assumes that:
 	 * <ul>
-	 * <li> {@code inferFrom} is assignable to {@code mutatorInferFrom}
-	 * <li> There are no vacuous generics in {@code unresolvedType}
+	 * <li> {@code inferFrom} is assignable to {@code mutatorInferFrom}</li>
+	 * <li> There are no vacuous generics in {@code unresolvedType}</li>
 	 * </ul>
 	 * 
 	 * @param inferFrom - the concrete input/output type that we would like to pass to the mutator.
@@ -219,10 +219,10 @@ public static List<OpInfo> getFocusers(OpEnvironment env, Type t) {
 	/**
 	 * Creates a Class given an Op and a set of {@link Simplifier}s. This class:
 	 * <ul>
-	 * <li>is of the same functional type as the given Op
+	 * <li>is of the same functional type as the given Op</li>
 	 * <li>has type arguments that are of the simplified form of the type
 	 * arguments of the given Op (these arguments are dictated by the list of
-	 * {@code Simplifier}s.
+	 * {@code Simplifier}s.</li>
 	 * <li>
 	 * 
 	 * @param originalOp - the Op that will be simplified
@@ -415,10 +415,10 @@ private static String createConstructor(CtClass cc, SimplificationMetadata metad
 	 * must:
 	 * <ol>
 	 * <li>Simplify all inputs using the {@link Function}s provided by the
-	 * {@link SimplifiedOpRef}
+	 * {@link SimplifiedOpRef}</li>
 	 * <li>Focus the simplified inputs using the {@link Function}s provided by the
-	 * {@link SimplifiedOpInfo}
-	 * <li>Call the {@code Op} using the focused inputs.
+	 * {@link SimplifiedOpInfo}</li>
+	 * <li>Call the {@code Op} using the focused inputs.</li>
 	 * </ol>
 	 * <b>NB</b> The Javassist compiler
 	 * <a href="https://www.javassist.org/tutorial/tutorial3.html#generics">does
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/simplify/SimplifiedOpInfo.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/simplify/SimplifiedOpInfo.java
@@ -119,17 +119,17 @@ public double priority() {
 	 * following:
 	 * <ul>
 	 * <li>{@link Priority#VERY_LOW} to ensure that simplifications are not chosen
-	 * over a direct match.
+	 * over a direct match.</li>
 	 * <li>The {@link OpInfo#priority} of the source info to ensure that a
 	 * simplification of a higher-priority Op wins out over a simplification of a
-	 * lower-priority Op, all else equal.
+	 * lower-priority Op, all else equal.</li>
 	 * <li>a penalty defined as a lossiness heuristic of this simplification. This
-	 * penalty is the sum of:
+	 * penalty is the sum of:</li>
 	 * <ul>
 	 * <li>the loss undertaken by converting each of the Op's inputs from the ref
-	 * type to the info type
+	 * type to the info type</li>
 	 * <li>the loss undertaken by converting each of the Op's outputs from the info
-	 * type to the ref type
+	 * type to the ref type</li>
 	 * </ul>
 	 * </ul>
 	 */
@@ -273,12 +273,12 @@ public String version() {
 	 * For a simplified Op, we define the implementation as the concatenation
 	 * of:
 	 * <ol>
-	 * <li>The signature of all input simplifiers
-	 * <li>The signature of all input focusers
-	 * <li>The signature of the output simplifier
-	 * <li>The signature of the output focuser
-	 * <li>The signature of the output copier
-	 * <li>The id of the source Op
+	 * <li>The signature of all input simplifiers</li>
+	 * <li>The signature of all input focusers</li>
+	 * <li>The signature of the output simplifier</li>
+	 * <li>The signature of the output focuser</li>
+	 * <li>The signature of the output copier</li>
+	 * <li>The id of the source Op</li>
 	 * </ol>
 	 * <p>
 	 */
diff --git a/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/struct/OpRetypingMemberParser.java b/scijava/scijava-ops-engine/src/main/java/org/scijava/ops/engine/struct/OpRetypingMemberParser.java
@@ -25,10 +25,10 @@ public class OpRetypingMemberParser implements
 	 * <p>
 	 * This method makes a couple of assumptions:
 	 * <ol>
-	 * <li>That {@code srcStruct} is valid
+	 * <li>That {@code srcStruct} is valid</li>
 	 * <li>That there are {@code inputs.length} input {@link Member}s in
-	 * {@code srcStruct}
-	 * <li>That there is <b>one</b> output {@link Member} in {@code srcStruct}
+	 * {@code srcStruct}</li>
+	 * <li>That there is <b>one</b> output {@link Member} in {@code srcStruct}</li>
 	 * </ol>
 	 * We should consider adding the evalutation of these assumptions
 	 * 
diff --git a/scijava/scijava-ops-engine/src/test/java/org/scijava/ops/engine/JavadocParameterTest.java b/scijava/scijava-ops-engine/src/test/java/org/scijava/ops/engine/JavadocParameterTest.java
@@ -235,12 +235,12 @@ public void testJavadocMethodInplaceWithDepedency() {
 	/**
 	 * Asserts that the {@link OpInfo} has as inputs:
 	 * <ul>
-	 * <li> foo - the first input
-	 * <li> bar - the second input
+	 * <li> foo - the first input</li>
+	 * <li> bar - the second input</li>
 	 * </ul>
 	 * and as output:
 	 * <ul>
-	 * <li> output - foo + bar
+	 * <li> output - foo + bar</li>
 	 * </ul>
 	 */
 	private void isSuitableScrapedOpMethodInfo(OpInfo info) {
@@ -266,12 +266,12 @@ private void isSuitableScrapedOpMethodInfo(OpInfo info) {
 	/**
 	 * Asserts that the {@link OpInfo} has as inputs:
 	 * <ul>
-	 * <li> input1
-	 * <li> input2
+	 * <li> input1</li>
+	 * <li> input2</li>
 	 * </ul>
 	 * and as output:
 	 * <ul>
-	 * <li> output
+	 * <li> output</li>
 	 * </ul>
 	 */
 	private void isSuitableGenericOpMethodInfo(OpInfo info) {
diff --git a/scijava/scijava-ops-engine/src/test/java/org/scijava/ops/engine/OpBuilderTest.java b/scijava/scijava-ops-engine/src/test/java/org/scijava/ops/engine/OpBuilderTest.java
@@ -51,34 +51,34 @@
  * Input TYPES are given (IT):
  * <p>
  * <ol>
- * <li>The output is unspecified (OU):
+ * <li>The output is unspecified (OU):</li>
  * <ol type="a">
- * <li>match: Function, Inplace
- * <li>run: none
+ * <li>match: Function, Inplace</li>
+ * <li>run: none</li>
  * </ol>
- * <li>The output type is given (OT):
+ * <li>The output type is given (OT):</li>
  * <ol type="a">
- * <li>match: Function, Computer
- * <li>run: none
+ * <li>match: Function, Computer</li>
+ * <li>run: none</li>
  * </ol>
  * </ol>
  * Input VALUES are given (IV) (N.B. this case applies for Arity0):
  * <p>
  * <ol>
- * <li>The output is unspecified (OU):
+ * <li>The output is unspecified (OU):</li>
  * <ol type="a">
- * <li>match: Function, Inplace
- * <li>run: apply, mutate
+ * <li>match: Function, Inplace</li>
+ * <li>run: apply, mutate</li>
  * </ol>
- * <li>The output type is given (OT):
+ * <li>The output type is given (OT):</li>
  * <ol type="a">
- * <li>match: Function, Computer
- * <li>run: apply
+ * <li>match: Function, Computer</li>
+ * <li>run: apply</li>
  * </ol>
- * <li>The output value is given (OV):
+ * <li>The output value is given (OV):</li>
  * <ol type="a">
- * <li>match: Computer
- * <li>run: compute
+ * <li>match: Computer</li>
+ * <li>run: compute</li>
  * </ol>
  * </ol>
  * 
diff --git a/scijava/scijava-ops-engine/templates/test/java/org/scijava/ops/engine/OpBuilderTest.vm b/scijava/scijava-ops-engine/templates/test/java/org/scijava/ops/engine/OpBuilderTest.vm
diff --git a/scijava/scijava-types/src/main/java/org/scijava/types/Any.java b/scijava/scijava-types/src/main/java/org/scijava/types/Any.java
