This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new 4bc5c86  Fix JSP API Javadoc warnings for Java 16
4bc5c86 is described below

commit 4bc5c8662c168231090a2ad608620742d3eeac2a
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue Aug 31 17:55:09 2021 +0100

    Fix JSP API Javadoc warnings for Java 16
---
 java/javax/el/ELResolver.java                      | 19 ++++++++++++
 java/javax/servlet/jsp/JspContext.java             |  7 ++++-
 .../servlet/jsp/el/ImplicitObjectELResolver.java   |  4 +++
 java/javax/servlet/jsp/tagext/JspIdConsumer.java   | 10 +++++++
 .../javax/servlet/jsp/tagext/TagAttributeInfo.java | 34 +++++++++++++++++++---
 java/javax/servlet/jsp/tagext/TagSupport.java      | 16 ++++++----
 6 files changed, 79 insertions(+), 11 deletions(-)

diff --git a/java/javax/el/ELResolver.java b/java/javax/el/ELResolver.java
index 6a10cd1..0e438ec 100644
--- a/java/javax/el/ELResolver.java
+++ b/java/javax/el/ELResolver.java
@@ -118,8 +118,27 @@ public abstract class ELResolver {
     public abstract boolean isReadOnly(ELContext context, Object base,
             Object property);
 
+    /**
+     * Obtain the feature descriptors for the resolvable properties of the 
given
+     * object.
+     *
+     * @param context The context in which the examination takes place
+     * @param base The object to examine
+     *
+     * @return An iterator, possibly empty, of feature descriptors of the given
+     *         object
+     */
     public abstract Iterator<java.beans.FeatureDescriptor> 
getFeatureDescriptors(ELContext context, Object base);
 
+    /**
+     * Obtain the most common type that is acceptable for the given base 
object.
+     *
+     * @param context The context in which the examination takes place
+     * @param base The object to examine
+     *
+     * @return {code null} if the most common type cannot be determine,
+     *         otherwise the most common type
+     */
     public abstract Class<?> getCommonPropertyType(ELContext context,
             Object base);
 
diff --git a/java/javax/servlet/jsp/JspContext.java 
b/java/javax/servlet/jsp/JspContext.java
index 0b8221b..20b5f8b 100644
--- a/java/javax/servlet/jsp/JspContext.java
+++ b/java/javax/servlet/jsp/JspContext.java
@@ -227,7 +227,12 @@ public abstract class JspContext {
     @SuppressWarnings("dep-ann") // TCK signature test fails with annotation
     public abstract javax.servlet.jsp.el.ExpressionEvaluator 
getExpressionEvaluator();
 
-
+    /**
+     * Obtain the ELContext for this JSPContext. Each JSPContext has a 
dedicated
+     * ELCOntext.
+     *
+     * @return the ELContext for this JSPContext
+     */
     public abstract ELContext getELContext();
 
     /**
diff --git a/java/javax/servlet/jsp/el/ImplicitObjectELResolver.java 
b/java/javax/servlet/jsp/el/ImplicitObjectELResolver.java
index 431cdc8..027b254 100644
--- a/java/javax/servlet/jsp/el/ImplicitObjectELResolver.java
+++ b/java/javax/servlet/jsp/el/ImplicitObjectELResolver.java
@@ -39,6 +39,7 @@ import javax.servlet.jsp.JspContext;
 import javax.servlet.jsp.PageContext;
 
 /**
+ * Provides resolution in EL for the implicit variables of a JSP page.
  *
  * @since 2.1
  */
@@ -71,6 +72,9 @@ public class ImplicitObjectELResolver extends ELResolver {
 
     private static final int SESSION_SCOPE = 10;
 
+    /**
+     * Creates an instance of the implicit object resolver for EL.
+     */
     public ImplicitObjectELResolver() {
         super();
     }
diff --git a/java/javax/servlet/jsp/tagext/JspIdConsumer.java 
b/java/javax/servlet/jsp/tagext/JspIdConsumer.java
index f3d49bf..57fe4f1 100644
--- a/java/javax/servlet/jsp/tagext/JspIdConsumer.java
+++ b/java/javax/servlet/jsp/tagext/JspIdConsumer.java
@@ -16,6 +16,16 @@
  */
 package javax.servlet.jsp.tagext;
 
+/**
+ * Interface that allows tag handlers to be provided with a unique (within the
+ * scope of the web application) ID.
+ */
 public interface JspIdConsumer {
+
+    /**
+     * Set the unique ID for the tag handler.
+     *
+     * @param jspId The unique Id
+     */
     public void setJspId(String jspId);
 }
diff --git a/java/javax/servlet/jsp/tagext/TagAttributeInfo.java 
b/java/javax/servlet/jsp/tagext/TagAttributeInfo.java
index 550a679..a54c019 100644
--- a/java/javax/servlet/jsp/tagext/TagAttributeInfo.java
+++ b/java/javax/servlet/jsp/tagext/TagAttributeInfo.java
@@ -26,11 +26,11 @@ package javax.servlet.jsp.tagext;
  */
 
 public class TagAttributeInfo {
+
     /**
      * "id" is wired in to be ID. There is no real benefit in having it be
      * something else IDREFs are not handled any differently.
      */
-
     public static final String ID = "id";
 
     /**
@@ -128,7 +128,6 @@ public class TagAttributeInfo {
      *
      * @return the name of the attribute
      */
-
     public String getName() {
         return name;
     }
@@ -138,7 +137,6 @@ public class TagAttributeInfo {
      *
      * @return the type of the attribute
      */
-
     public String getTypeName() {
         return type;
     }
@@ -148,7 +146,6 @@ public class TagAttributeInfo {
      *
      * @return if the attribute can hold a request-time value.
      */
-
     public boolean canBeRequestTime() {
         return reqTime;
     }
@@ -240,22 +237,51 @@ public class TagAttributeInfo {
 
     private final String methodSignature;
 
+    /**
+     * Does the attribute expect to be passed a deferred method?
+     *
+     * @return {@code true} if a deferred method expression is expected,
+     *         otherwise {@code false}
+     */
     public boolean isDeferredMethod() {
         return deferredMethod;
     }
 
+    /**
+     * Does the attribute expect to be passed a deferred value?
+     *
+     * @return {@code true} if a deferred value expression is expected,
+     *         otherwise {@code false}
+     */
     public boolean isDeferredValue() {
         return deferredValue;
     }
 
+    /**
+     * Obtain the description for the attribute,
+     *
+     * @return the description
+     */
     public String getDescription() {
         return description;
     }
 
+    /**
+     * Obtain the type name, as a string, expected by this attribute.
+     *
+     * @return the type name, as a string
+     */
     public String getExpectedTypeName() {
         return expectedTypeName;
     }
 
+    /**
+     * If this is a deferred method attribute, obtain the expected method
+     * signature.
+     *
+     * @return The expected method signature or {@code null} if this attribute
+     *         is not a deferred method attribute
+     */
     public String getMethodSignature() {
         return methodSignature;
     }
diff --git a/java/javax/servlet/jsp/tagext/TagSupport.java 
b/java/javax/servlet/jsp/tagext/TagSupport.java
index ebad48d..762ff52 100644
--- a/java/javax/servlet/jsp/tagext/TagSupport.java
+++ b/java/javax/servlet/jsp/tagext/TagSupport.java
@@ -272,16 +272,20 @@ public class TagSupport implements IterationTag, 
Serializable {
         return values.keys();
     }
 
-    // private fields
+    /**
+     * The parent, if any, of thsi tag.
+     */
+    private Tag parent;
 
-    private   Tag         parent;
-    private   Hashtable<String, Object>   values;
     /**
-     * The value of the id attribute of this tag; or null.
+     * Map of object values keyed by Strings for this tag.
      */
-    protected String      id;
+    private Hashtable<String, Object> values;
 
-    // protected fields
+    /**
+     * The value of the id attribute of this tag; or null.
+     */
+    protected String id;
 
     /**
      * The PageContext.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to