minor refactor: fix javadoc warnings

Author: paulk-asert

src/main/java/org/apache/groovy/util/concurrent/ConcurrentReferenceHashMap.java

@@ -1950,7 +1950,7 @@ protected abstract class HashIterator {
             advance();
         }
 
-        /** {@inheritDoc} */
+        /** @return {@code true} if more elements remain (delegates to {@link #hasNext()}) */
         public boolean hasMoreElements() {
             return hasNext();
         }
@@ -1981,7 +1981,7 @@ final void advance() {
             }
         }
 
-        /** {@inheritDoc} */
+        /** @return {@code true} if there is a next live entry to iterate to */
         public boolean hasNext() {
             while (nextEntry != null) {
                 if (nextEntry.key() != null) {
@@ -2009,7 +2009,7 @@ HashEntry<K, V> nextEntry() {
             return lastReturned;
         }
 
-        /** {@inheritDoc} */
+        /** Removes the last entry returned by the iterator. */
         public void remove() {
             if (lastReturned == null) {
                 throw new IllegalStateException();

src/main/java/org/codehaus/groovy/ast/ClassNode.java

@@ -800,7 +800,6 @@ public FieldNode addFieldFirst(String name, int modifiers, ClassNode type, Expre
      *
      * @param node the {@link PropertyNode} to add, must not be {@code null}
      * @see #addField(FieldNode)
-     * @see #removeProperty(String)
      * @see PropertyNode
      */
     public void addProperty(PropertyNode node) {

src/main/java/org/codehaus/groovy/ast/GenericsType.java

@@ -262,7 +262,9 @@ public ClassNode getLowerBound() {
     }
 
     /**
-     * Returns the upper bounds for this wildcard or placeholder type (e.g., Bounds in "? extends Bound" or "T extends Bound1 & Bound2"),
+     * Returns the upper bounds for this wildcard or placeholder type
+     * (e.g., {@code Bound} in {@code ? extends Bound} or
+     * {@code Bound1 & Bound2} in {@code T extends Bound1 & Bound2}),
      * or null if this type has no upper bounds.
      *
      * @return array of upper bound {@link ClassNode}s, or null

src/main/java/org/codehaus/groovy/ast/GroovyCodeVisitor.java

@@ -363,7 +363,7 @@ default void visit(final Statement statement) {
     void visitFieldExpression(FieldExpression expression);
 
     /**
-     * Visits a method pointer expression (object.&methodName).
+     * Visits a method pointer expression ({@code object.&methodName}).
      *
      * @param expression the {@link MethodPointerExpression} to process
      */

src/main/java/org/codehaus/groovy/ast/decompiled/ClassStub.java

@@ -39,7 +39,7 @@
  * <p><strong>Thread Safety:</strong> ClassStub instances are typically constructed once and then
  * treated as immutable. They are safe for concurrent read access after initialization.
  *
- * @see AsmDecompiler#parseClass(String, int, String, String, String[])
+ * @see AsmDecompiler#parseClass(java.net.URL)
  * @see DecompiledClassNode#DecompiledClassNode(ClassStub, AsmReferenceResolver)
  */
 public class ClassStub extends MemberStub {

src/main/java/org/codehaus/groovy/ast/decompiled/DecompiledClassNode.java

@@ -96,7 +96,7 @@
  * <p>Groovy embeds compilation timestamps in synthetic static field names for change detection.
  * {@link #getCompilationTimeStamp()} decodes this metadata using {@link org.codehaus.groovy.classgen.Verifier#getTimestampFromFieldName(String)}.
  *
- * @see AsmDecompiler#parseClass(String, int, String, String, String[])
+ * @see AsmDecompiler#parseClass(java.net.URL)
  * @see AsmReferenceResolver
  * @see ClassStub
  * @see LazyFieldNode

src/main/java/org/codehaus/groovy/ast/expr/ExpressionTransformer.java

@@ -41,7 +41,7 @@
  * <p>This is a functional interface and can be implemented as a lambda expression.
  *
  * @see org.codehaus.groovy.ast.ClassCodeExpressionTransformer for abstract transformer base class
- * @see TransformingCodeVisitor for visitor using transformers
+ * @see org.codehaus.groovy.ast.TransformingCodeVisitor for visitor using transformers
  */
 @FunctionalInterface
 public interface ExpressionTransformer {

src/main/java/org/codehaus/groovy/ast/stmt/ExpressionStatement.java

@@ -28,7 +28,6 @@
  * an expression must be treated as a statement (e.g., method calls, assignments, or other
  * side-effect-producing expressions). The expression is evaluated and its result is discarded.
  *
- * @throws IllegalArgumentException if the expression is null
  * @see {@link Expression}
  * @see {@link Statement}
  * @see {@link ReturnStatement}

subprojects/groovy-xml/src/main/java/groovy/xml/XmlUtil.java

@@ -767,7 +767,7 @@ public static void setFeatureQuietly(SAXParserFactory factory, String feature, b
     }
 
     /**
-     * Streams XML events from the supplied {@link Reader} using a hardened
+     * Streams XML events from the supplied {@link java.io.Reader} using a hardened
      * StAX {@link javax.xml.stream.XMLInputFactory}. The returned stream
      * lazily pulls events; closing the stream closes the underlying reader.
      * <p>
@@ -782,7 +782,7 @@ public static java.util.stream.Stream<javax.xml.stream.events.XMLEvent> events(j
     }
 
     /**
-     * Streams XML events from the supplied {@link Reader} using a hardened
+     * Streams XML events from the supplied {@link java.io.Reader} using a hardened
      * StAX {@link javax.xml.stream.XMLInputFactory}. The returned stream
      * lazily pulls events; closing the stream closes the underlying reader.
      *