Correct Javadoc warnings, prevent future ones

Base/src/main/java/io/deephaven/base/ArrayUtil.java

@@ -13,7 +13,7 @@ public class ArrayUtil {
 
     /**
      * The maximum array size is determined by the JVM and in practice is less than Integer.MAX_VALUE. (See note at
-     * {@link jdk.internal.util.ArraysSupport#SOFT_MAX_ARRAY_LENGTH}.)
+     * {@code jdk.internal.util.ArraysSupport#SOFT_MAX_ARRAY_LENGTH}.)
      */
     // We would like to use jdk.internal.util.ArraysSupport.MAX_ARRAY_LENGTH, but it is not exported
     public static final int MAX_ARRAY_SIZE = Integer.MAX_VALUE - 8;

Base/src/main/java/io/deephaven/base/CompareUtils.java

@@ -159,7 +159,7 @@ public static boolean doubleNotEquals2(double x, double y, double tolerance) {
     }
 
     /**
-     * You can't do double1 > double2 because floating point numbers are not exact values. Just make sure that x-y is
+     * You can't do double1 > double2 because floating point numbers are not exact values. Just make sure that x-y is
      * greater than some allowable error factor for equality
      *
      * @param x
@@ -191,8 +191,8 @@ public static boolean doubleGreaterEqual(double x, double y, double tolerance) {
     }
 
     /**
-     * You can't do double1 < double2 because floating point numbers are not exact values. Just make sure that y - x is
-     * greater than some allowable error factor for equality
+     * You can't do double1 &lt; double2 because floating point numbers are not exact values. Just make sure that y - x
+     * is greater than some allowable error factor for equality
      *
      * @param x
      * @param y

Base/src/main/java/io/deephaven/base/LowGarbageArrayIntegerMap.java

@@ -19,7 +19,7 @@
  * when expanding). This set only has one {@link Iterator}, which is reused. This set is not thread safe.
  * <P>
  * Note: This class extends {@link HashMap} rather than {@link Map} (or {@link AbstractMap}) only because one of the
- * fields where we want to use it ({@link sun.nio.ch.EPollSelectorImpl#fdToKey}) is (improperly) declared as a HashMap
+ * fields where we want to use it ({@code sun.nio.ch.EPollSelectorImpl#fdToKey}) is (improperly) declared as a HashMap
  * rather than a Map.
  */
 public class LowGarbageArrayIntegerMap<T> extends HashMap<Integer, T> {

Base/src/main/java/io/deephaven/base/LowGarbageArraySet.java

@@ -16,7 +16,7 @@
  * expanding). This set only has one {@link Iterator}, which is reused. This set is not thread safe.
  * <P>
  * Note: This class extends {@link HashSet} rather than {@link Set} (or {@link AbstractSet}) only because one of the
- * fields where we want to use it ({@link sun.nio.ch.SelectorImpl#keys}) is (improperly) declared as a HashSet rather
+ * fields where we want to use it ({@code sun.nio.ch.SelectorImpl#keys}) is (improperly) declared as a HashSet rather
  * than a Set.
  */
 public class LowGarbageArraySet<T> extends HashSet<T> {

Base/src/main/java/io/deephaven/base/MathUtil.java

@@ -75,7 +75,7 @@ public static int gcd(int a, int b) {
     };
 
     /**
-     * Compute 10^n as a int for 0 <= n <= 9.
+     * Compute 10^n as a int for 0 &lt;= n &lt;= 9.
      *
      * @param n the exponent
      * @return 10^n
@@ -95,9 +95,9 @@ public static int pow10(int n) {
     };
 
     /**
-     * Compute the number of base 10 digits in n's representation, for n >= 0.
+     * Compute the number of base 10 digits in n's representation, for n &gt;= 0.
      * 
-     * @param n an integer >= 0
+     * @param n an integer &gt;= 0
      * @return how many digits in n's base 10 representation.
      */
     public static int base10digits(int n) {

Base/src/main/java/io/deephaven/base/cache/OpenAddressedCanonicalizationCache.java

@@ -48,7 +48,7 @@ public interface Adapter<INPUT_TYPE, OUTPUT_TYPE> {
         int hashCode(@NotNull final INPUT_TYPE inputItem);
 
         /**
-         * Note: The following must be true: hashCode(inputItem) == outputItem.hashCode() && equals(inputItem,
+         * Note: The following must be true: hashCode(inputItem) == outputItem.hashCode() &amp;&amp; equals(inputItem,
          * outputItem)
          * 
          * @param inputItem The input item

Base/src/main/java/io/deephaven/base/stats/Group.java

@@ -95,7 +95,7 @@ public Item[] getItems() {
     }
 
     /**
-     * Update the histories of all items in this group, logging all updated intervals >= logInterval.
+     * Update the histories of all items in this group, logging all updated intervals >= logInterval.
      */
     public void update(ItemUpdateListener listener, long logInterval, long now, long appNow) {
         Item[] arr = itemsArray;

Base/src/main/java/io/deephaven/base/stats/Item.java

@@ -71,7 +71,7 @@ public String getCompactName() {
     }
 
     /**
-     * Update the history intervals for this item's value, logging updated intervals >= logInterval
+     * Update the history intervals for this item's value, logging updated intervals >= logInterval
      */
     public void update(ItemUpdateListener listener, long logInterval, long now, long appNow) {
         value.update(this, listener, logInterval, now, appNow);

Base/src/main/java/io/deephaven/base/stats/Stats.java

@@ -161,7 +161,7 @@ public synchronized static Group getGroup(String name) {
     }
 
     /**
-     * Update the histories of all items in all groups, logging all updated intervals >= logInterval.
+     * Update the histories of all items in all groups, logging all updated intervals >= logInterval.
      */
     public synchronized static void update(ItemUpdateListener listener, long now, long appNow, long logInterval) {
         for (Group g : groups) {

Base/src/main/java/io/deephaven/base/string/EncodingInfo.java

@@ -56,7 +56,8 @@ public String getEncodingName() {
     }
 
     /**
-     * Can this encoding info's charset be encoded or decoded by simple linear assignment of char->byte or byte->char.
+     * Can this encoding info's charset be encoded or decoded by simple linear assignment of char->byte or
+     * byte->char.
      * 
      * @return Whether this encoding info's charset is simple
      */

Base/src/main/java/io/deephaven/base/string/cache/MappedCompressedString.java

@@ -12,8 +12,8 @@
 /**
  * This is a ridiculously simple, light-as-I-can-make-it, but decidedly single-purpose data structure. Specifically,
  * it's a CompressedString with an embedded (to avoid reference or Object instance overhead) open-addressed
- * SimpleReference<Object>-identity -> int hash map with load factor 1 (100%) and no public operations other than
- * "putIfAbsent".
+ * SimpleReference&lt;Object&gt;-identity -&gt; int hash map with load factor 1 (100%) and no public operations other
+ * than "putIfAbsent".
  *
  * The reason for requiring that key objects be SimpleReferences is to allow for O(1) automatic slot reclamation across
  * many MappedCompressedString instances at the same time. A given source of mappings creates a single SimpleReference
@@ -23,8 +23,9 @@
  *
  * Unfortunately, I haven't figured out a way to make this allow concurrent gets.
  *
- * The intended use is in Deephaven import code, for storing SymbolManager -> SymbolId mappings on the CompressedString
- * that represents the Symbol itself, typically inside of a (bounded) StringCache of MappedCompressedString instances.
+ * The intended use is in Deephaven import code, for storing SymbolManager -&gt; SymbolId mappings on the
+ * CompressedString that represents the Symbol itself, typically inside of a (bounded) StringCache of
+ * MappedCompressedString instances.
  *
  * Note that this uses io.deephaven.base.reference.SimpleReference instead of java.lang.ref.Reference so that unit tests
  * can avoid being required to use the concrete sub-classes of Reference, which all come with GC-related side-effects.
@@ -92,7 +93,7 @@ public final synchronized int capacity() {
     }
 
     /**
-     * Add the specified <key, value> pair if no mapping already exists for key.
+     * Add the specified &lt;key, value&gt; pair if no mapping already exists for key.
      * 
      * @param key A non-null Reference to an arbitrary object whose reachability determines mapping validity.
      * @param potentialValue The value to insert if none already exists. Must not equal NULL_MAPPING_VALUE.

Base/src/main/java/io/deephaven/base/string/cache/StringCache.java

@@ -15,7 +15,7 @@ public interface StringCache<STRING_LIKE_TYPE extends CharSequence> {
     /**
      * Get a hint about this cache's capacity and behavior.
      * 
-     * @return -1 : This is an unbounded cache. 0 : This "cache" doesn't actually perform any caching. >0 : Actual
+     * @return -1 : This is an unbounded cache. 0 : This "cache" doesn't actually perform any caching. &gt;0 : Actual
      *         capacity bound.
      */
     int capacity();

Base/src/main/java/io/deephaven/base/string/cache/StringCompatible.java

@@ -59,7 +59,7 @@ public interface StringCompatible extends CharSequence, Comparable<CharSequence>
     /**
      * Implementations MUST compare StringCompatibles and Strings char-by-char.
      * 
-     * @return 0, <0, or >0 if that compares equal-to, less-than, or greater-than this.
+     * @return 0, &lt;0, or &gt;0 if that compares equal-to, less-than, or greater-than this.
      */
     @Override
     int compareTo(@NotNull final CharSequence that);

Base/src/main/java/io/deephaven/base/system/AsyncSystem.java

@@ -82,7 +82,7 @@ public static void exitUncaught(Thread thread, Throwable throwable, int status,
     }
 
     /**
-     * Equivalent to {@code exitCaught(thread, throwable, status, out, null).
+     * Equivalent to {@code exitCaught(thread, throwable, status, out, null)}.
      */
     public static void exitCaught(Thread thread, Throwable throwable, int status, PrintStream out) {
         exitCaught(thread, throwable, status, out, null);