8303480: Miscellaneous fixes to mostly invisible doc comments

make/ide/idea/jdk/template/src/idea/JdkIdeaAntLogger.java

@@ -54,7 +54,7 @@ public final class JdkIdeaAntLogger extends DefaultLogger {
     /**
      * This is just a way to pass in customized binary string predicates;
      *
-     * TODO: replace with @code{BiPredicate<String, String>} and method reference when moving to 8
+     * TODO: replace with {@code BiPredicate<String, String>} and method reference when moving to 8
      */
     enum StringBinaryPredicate {
         CONTAINS() {

make/ide/idea/langtools/template/src/idea/LangtoolsIdeaAntLogger.java

@@ -54,7 +54,7 @@ public final class LangtoolsIdeaAntLogger extends DefaultLogger {
     /**
      * This is just a way to pass in customized binary string predicates;
      *
-     * TODO: replace with @code{BiPredicate<String, String>} and method reference when moving to 8
+     * TODO: replace with {@code BiPredicate<String, String>} and method reference when moving to 8
      */
     enum StringBinaryPredicate {
         CONTAINS() {

src/java.base/share/classes/java/lang/invoke/BootstrapMethodInvoker.java

@@ -255,7 +255,7 @@ private static Object invokeWithManyArguments(MethodHandle bootstrapMethod, Look
 
     /**
      * @return true iff the BSM method type exactly matches
-     *         {@see java.lang.invoke.StringConcatFactory#makeConcatWithConstants(MethodHandles.Lookup,
+     *         {@link java.lang.invoke.StringConcatFactory#makeConcatWithConstants(MethodHandles.Lookup,
      *                 String,MethodType,String,Object...))}
      */
     private static boolean isStringConcatFactoryBSM(MethodType bsmType) {
@@ -264,7 +264,7 @@ private static boolean isStringConcatFactoryBSM(MethodType bsmType) {
 
     /**
      * @return true iff the BSM method type exactly matches
-     *         {@see java.lang.invoke.LambdaMetafactory#metafactory(
+     *         {@link java.lang.invoke.LambdaMetafactory#metafactory(
      *          MethodHandles.Lookup,String,Class,MethodType,MethodHandle,MethodType)}
      */
     private static boolean isLambdaMetafactoryCondyBSM(MethodType bsmType) {
@@ -273,7 +273,7 @@ private static boolean isLambdaMetafactoryCondyBSM(MethodType bsmType) {
 
     /**
      * @return true iff the BSM method type exactly matches
-     *         {@see java.lang.invoke.LambdaMetafactory#metafactory(
+     *         {@link java.lang.invoke.LambdaMetafactory#metafactory(
      *          MethodHandles.Lookup,String,MethodType,MethodType,MethodHandle,MethodType)}
      */
     private static boolean isLambdaMetafactoryIndyBSM(MethodType bsmType) {
@@ -282,7 +282,7 @@ private static boolean isLambdaMetafactoryIndyBSM(MethodType bsmType) {
 
     /**
      * @return true iff the BSM method type exactly matches
-     *         {@see java.lang.invoke.LambdaMetafactory#altMetafactory(
+     *         {@link java.lang.invoke.LambdaMetafactory#altMetafactory(
      *          MethodHandles.Lookup,String,MethodType,Object[])}
      */
     private static boolean isLambdaMetafactoryAltMetafactoryBSM(MethodType bsmType) {

src/java.base/share/classes/java/security/AccessControlContext.java

@@ -186,7 +186,7 @@ public AccessControlContext(AccessControlContext acc,
     }
 
     /**
-     * package private to allow calls from (@code ProtectionDomain} without
+     * package private to allow calls from {@code ProtectionDomain} without
      * performing the security check for
      * {@linkplain SecurityConstants#CREATE_ACC_PERMISSION} permission
      */

src/java.base/share/classes/java/security/BasicPermission.java

@@ -260,7 +260,7 @@ public PermissionCollection newPermissionCollection() {
 
     /**
      * {@code readObject} is called to restore the state of the
-     * (@code BasicPermission} from a stream.
+     * {@code BasicPermission} from a stream.
      *
      * @param  s the {@code ObjectInputStream} from which data is read
      * @throws IOException if an I/O error occurs
@@ -323,8 +323,8 @@ final class BasicPermissionCollection
     private transient ConcurrentHashMap<String, Permission> perms;
 
     /**
-     * This is set to {@code true} if this (@code BasicPermissionCollection}
-     * contains a {code BasicPermission} with '*' as its permission name.
+     * This is set to {@code true} if this {@code BasicPermissionCollection}
+     * contains a {@code BasicPermission} with '*' as its permission name.
      *
      * @see #serialPersistentFields
      */

src/java.base/share/classes/jdk/internal/foreign/abi/CallingSequence.java

@@ -162,7 +162,7 @@ public boolean needsReturnBuffer() {
     /**
      * The size of the return buffer, if one is needed.
      *
-     * {@see #needsReturnBuffer}
+     * @see #needsReturnBuffer
      *
      * @return the return buffer size
      */

src/java.base/share/classes/jdk/internal/vm/VMSupport.java

@@ -66,7 +66,7 @@ private static byte[] serializePropertiesToByteArray(Properties p) throws IOExce
     }
 
     /**
-     * @returns a Properties object containing only the entries in {@code p}
+     * @return a Properties object containing only the entries in {@code p}
      *          whose key and value are both Strings
      */
     private static Properties onlyStrings(Properties p) {

src/java.base/share/classes/sun/util/locale/provider/LocaleNameProviderImpl.java

@@ -169,7 +169,7 @@ public String getDisplayVariant(String vrnt, Locale locale) {
     }
 
     /**
-     * @inheritDoc
+     * {@inheritDoc}
      */
     @Override
     public String getDisplayUnicodeExtensionKey(String key, Locale locale) {
@@ -180,7 +180,7 @@ public String getDisplayUnicodeExtensionKey(String key, Locale locale) {
     }
 
     /**
-     * @inheritDoc
+     * {@inheritDoc}
      */
     @Override
     public String getDisplayUnicodeExtensionType(String extType, String key, Locale locale) {

src/java.desktop/share/classes/javax/swing/plaf/synth/SynthComboBoxUI.java

@@ -645,7 +645,7 @@ private void updateOver(boolean o) {
         //------------------------------------------------------------------
 
         /**
-         * @inheritDoc
+         * {@inheritDoc}
          *
          * Ensures that isPressed() will return true if the combo is pressed,
          * or the arrowButton is pressed, <em>or</em> if the combo popup is
@@ -659,7 +659,7 @@ public boolean isPressed() {
         }
 
         /**
-         * @inheritDoc
+         * {@inheritDoc}
          *
          * Ensures that the armed state is in sync with the pressed state
          * if shouldActLikeButton is true. Without this method, the arrow
@@ -674,7 +674,7 @@ public boolean isArmed() {
         }
 
         /**
-         * @inheritDoc
+         * {@inheritDoc}
          *
          * Ensures that isRollover() will return true if the combo is
          * rolled over, or the arrowButton is rolled over.
@@ -685,7 +685,7 @@ public boolean isRollover() {
         }
 
         /**
-         * @inheritDoc
+         * {@inheritDoc}
          *
          * Forwards pressed states to the internal "pressed" field
          */
@@ -696,7 +696,7 @@ public void setPressed(boolean b) {
         }
 
         /**
-         * @inheritDoc
+         * {@inheritDoc}
          *
          * Forwards rollover states to the internal "over" field
          */
@@ -738,7 +738,7 @@ public void mouseClicked(MouseEvent e) {}
         //------------------------------------------------------------------
 
         /**
-         * @inheritDoc
+         * {@inheritDoc}
          *
          * Ensures that the combo box is repainted when the popup is closed.
          * This avoids a bug where clicking off the combo wasn't causing a repaint,

src/java.desktop/share/classes/javax/swing/plaf/synth/SynthComboPopup.java

@@ -67,7 +67,7 @@ protected void configureList() {
     }
 
     /**
-     * @inheritDoc
+     * {@inheritDoc}
      *
      * Overridden to take into account any popup insets specified in
      * SynthComboBoxUI

src/java.desktop/share/classes/sun/font/GlyphLayout.java

@@ -346,7 +346,7 @@ public static SDCache get(Font font, FontRenderContext frc) {
      * @param text the text, including optional context before start and after start + count
      * @param offset the start of the text to lay out
      * @param count the length of the text to lay out
-     * @param flags bidi and context flags {@see #java.awt.Font}
+     * @param flags bidi and context flags {@link java.awt.Font}
      * @param result a StandardGlyphVector to modify, can be null
      * @return the laid out glyphvector, if result was passed in, it is returned
      */

src/java.desktop/share/classes/sun/java2d/StateTrackableDelegate.java

@@ -93,15 +93,15 @@ private StateTrackableDelegate(State state) {
     }
 
     /**
-     * @inheritDoc
+     * {@inheritDoc}
      * @since 1.7
      */
     public State getState() {
         return theState;
     }
 
     /**
-     * @inheritDoc
+     * {@inheritDoc}
      * @since 1.7
      */
     public synchronized StateTracker getStateTracker() {

src/java.desktop/share/classes/sun/swing/AccumulativeRunnable.java

@@ -116,7 +116,7 @@ public final void run() {
      * appends arguments and sends this {@code Runnable} for the
      * execution if needed.
      * <p>
-     * This implementation uses {@see #submit} to send this
+     * This implementation uses {@link #submit} to send this
      * {@code Runnable} for execution.
      * @param args the arguments to accumulate
      */

src/java.desktop/share/classes/sun/swing/SwingUtilities2.java

@@ -2278,11 +2278,11 @@ public static boolean isFloatingPointScale(AffineTransform tx) {
 
     /**
      * Returns the client property for the given key if it is set; otherwise
-     * returns the {@L&F} property.
+     * returns the {@literal L&F} property.
      *
      * @param component the component
      * @param key an {@code String} specifying the key for the desired boolean value
-     * @return the boolean value of the client property if it is set or the {@L&F}
+     * @return the boolean value of the client property if it is set or the {@literal L&F}
      *         property in other case.
      */
     public static boolean getBoolean(JComponent component, String key) {

src/java.desktop/unix/classes/sun/awt/X11/XToolkit.java

@@ -2385,7 +2385,7 @@ public static long getEventNumber() {
     private static int oops_position = 0;
 
     /**
-     * @inheritDoc
+     * {@inheritDoc}
      */
     @Override
     protected boolean syncNativeQueue(long timeout) {

src/jdk.compiler/share/classes/com/sun/tools/javac/code/Type.java

@@ -594,7 +594,7 @@ public boolean isRaw() {
     /**
      * A compound type is a special class type whose supertypes are used to store a list
      * of component types. There are two kinds of compound types: (i) intersection types
-     * {@see IntersectionClassType} and (ii) union types {@see UnionClassType}.
+     * {@link IntersectionClassType} and (ii) union types {@link UnionClassType}.
      */
     public boolean isCompound() {
         return false;

src/jdk.compiler/share/classes/com/sun/tools/javac/code/Types.java

@@ -2862,7 +2862,7 @@ public boolean test(Type mt1, Type mt2, Types types) {
 
     /**
      * Merge multiple abstract methods. The preferred method is a method that is a subsignature
-     * of all the other signatures and whose return type is more specific {@see MostSpecificReturnCheck}.
+     * of all the other signatures and whose return type is more specific {@link MostSpecificReturnCheck}.
      * The resulting preferred method has a thrown clause that is the intersection of the merged
      * methods' clauses.
      */

src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Check.java

@@ -2445,7 +2445,7 @@ private void checkNonCyclic1(DiagnosticPosition pos, Type t, List<TypeVar> seen)
      *
      *  @param pos      Position to be used for error reporting.
      *  @param t        The type referred to.
-     *  @returns        True if the check completed on all attributed classes
+     *  @return        True if the check completed on all attributed classes
      */
     private boolean checkNonCyclicInternal(DiagnosticPosition pos, Type t) {
         boolean complete = true; // was the check complete?

src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Infer.java

@@ -656,7 +656,7 @@ public Type instantiateFunctionalInterface(DiagnosticPosition pos, Type funcInte
 
     /**
      * Infer record type for pattern matching. Given an expression type
-     * (@code expressionType}), and a given record ({@code patternTypeSymbol}),
+     * ({@code expressionType}), and a given record ({@code patternTypeSymbol}),
      * a parameterized type of {@code patternTypeSymbol} is inferred
      * according to JLS 18.5.5.
      *

src/jdk.compiler/share/classes/com/sun/tools/javac/comp/Resolve.java

@@ -3305,7 +3305,7 @@ static ReferenceLookupResult error(Symbol sym) {
 
     /**
      * This abstract class embodies the logic that converts one (bound lookup) or two (unbound lookup)
-     * {@code ReferenceLookupResult} objects into a (@code Symbol), which is then regarded as the
+     * {@code ReferenceLookupResult} objects into a {@code Symbol}, which is then regarded as the
      * result of method reference resolution.
      */
     abstract class ReferenceChooser {

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/debugger/win32/coff/DebugDirectoryEntry.java

@@ -46,7 +46,7 @@ public interface DebugDirectoryEntry {
 
   /** Format of debugging information: this field enables support of
       multiple debuggers. See
-      @link{sun.jvm.hotspot.debugger.win32.coff.DebugTypes}. */
+      {@link sun.jvm.hotspot.debugger.win32.coff.DebugTypes}. */
   public int getType();
 
   /** Size of the debug data (not including the debug directory itself). */
@@ -60,7 +60,7 @@ public interface DebugDirectoryEntry {
 
   /** If this debug directory entry is of type
       IMAGE_DEBUG_TYPE_CODEVIEW (see
-      @link{sun.jvm.hotspot.debugger.win32.coff.DebugTypes}), returns
+      {@link sun.jvm.hotspot.debugger.win32.coff.DebugTypes}), returns
       the contents as a DebugVC50 object; otherwise, returns null. */
   public DebugVC50 getDebugVC50();
 

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/utilities/LivenessPath.java

@@ -30,9 +30,9 @@
 
 /** Describes a path from an object back to the root which is keeping
     it alive. Elements of the path are (object, field) pairs, where
-    the object is expressed as a @link{sun.jvm.hotspot.oops.Oop}, and
+    the object is expressed as a {@link sun.jvm.hotspot.oops.Oop}, and
     where the field is expressed as a
-    @link{sun.jvm.hotspot.oops.FieldIdentifier}. If the element
+    {@link sun.jvm.hotspot.oops.FieldIdentifier}. If the element
     reflects a root, the Oop will be null. If the element is the end
     of the path, the FieldIdentifier will be null. */
 

src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/hotspot/Cleaner.java

@@ -98,9 +98,9 @@ private static synchronized void remove(Cleaner cl) {
     /**
      * Performs the cleanup action now that this object's referent has become weakly reachable.
      *
-     * @returns true if the clean up action cleared the referent of an oop handle and requires a
-     *          subsequent call to {@link CompilerToVM#releaseClearedOopHandles()} to reclaim the
-     *          resources of the handle itself
+     * @return true if the clean up action cleared the referent of an oop handle and requires a
+     *         subsequent call to {@link CompilerToVM#releaseClearedOopHandles()} to reclaim the
+     *         resources of the handle itself
      */
     abstract boolean doCleanup();
 

src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/hotspot/HotSpotCompiledCodeStream.java

@@ -629,7 +629,7 @@ private Object[] finalizeObjectPool() throws JVMCIError {
      * Determines if {@code name} or {@code method} are matched by
      * {@link Option#DumpSerializedCode}.
      *
-     * @returns the matched value or null if no match was made
+     * @return the matched value or null if no match was made
      */
     private static String shouldDump(String name, HotSpotCompiledNmethod nmethod) {
         String filter = Option.DumpSerializedCode.getString();

src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/hotspot/JFR.java

@@ -31,7 +31,7 @@
 
 /**
  * Helper methods for interacting with the Java Flight Recorder (JFR) to register events and notify
- * it when events occur. The JFR events are defined in {see @code
+ * it when events occur. The JFR events are defined in {@code
  * src/share/jfr/metadata/metadata.xml}.
  */
 public final class JFR {
@@ -50,7 +50,7 @@ public static long now() {
     }
 
     /**
-     * Helper methods for managing JFR CompilerPhase events. The events are defined in {see @code
+     * Helper methods for managing JFR CompilerPhase events. The events are defined in {@code
      * src/share/jfr/metadata/metadata.xml}.
      */
     public static final class CompilerPhaseEvent {
@@ -75,7 +75,7 @@ public static void write(long startTime, String phaseName, int compileId, int ph
     }
 
     /**
-     * Helper methods for managing JFR CompilerInlining events. The events are defined in {see @code
+     * Helper methods for managing JFR CompilerInlining events. The events are defined in {@code
      * src/share/jfr/metadata/metadata.xml}.
      */
     public static final class CompilerInliningEvent {

src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/meta/MethodHandleAccessProvider.java

@@ -66,8 +66,8 @@ enum IntrinsicMethod {
      * interpreter overhead. If the parameter forceBytecodeGeneration is set to true, the VM should
      * try to generate bytecodes before this method returns.
      *
-     * @returns {@code null} if {@code methodHandle} is not a {@link MethodHandle} or the invocation
-     *          target is not available at this time
+     * @return {@code null} if {@code methodHandle} is not a {@link MethodHandle} or the invocation
+     *         target is not available at this time
      * @throws NullPointerException if {@code methodHandle} is null
      */
     ResolvedJavaMethod resolveInvokeBasicTarget(JavaConstant methodHandle, boolean forceBytecodeGeneration);
@@ -77,7 +77,7 @@ enum IntrinsicMethod {
      * with the given constant member name. The member name is the last parameter of the
      * {@code linkTo*} method.
      *
-     * @returns {@code null} if the invocation target is not available at this time
+     * @return {@code null} if the invocation target is not available at this time
      * @throws NullPointerException if {@code memberName} is null
      * @throws IllegalArgumentException if {@code memberName} is not a
      *             {@code java.lang.invoke.MemberName}