You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2023/01/12 18:47:12 UTC
[tomcat] branch 10.1.x updated: CheckStyle Javadoc checks += JavadocMissingLeadingAsterisk
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new 4b38d6dbcf CheckStyle Javadoc checks += JavadocMissingLeadingAsterisk
4b38d6dbcf is described below
commit 4b38d6dbcfa06e1e0d6b7435da87632cee16572a
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Thu Jan 12 18:41:58 2023 +0000
CheckStyle Javadoc checks += JavadocMissingLeadingAsterisk
---
java/jakarta/servlet/jsp/tagext/TagData.java | 2 +-
java/org/apache/catalina/realm/JAASRealm.java | 26 ++++++++++++----------
java/org/apache/catalina/servlets/CGIServlet.java | 2 +-
.../TomcatURLStreamHandlerFactory.java | 1 -
java/org/apache/coyote/ajp/AjpProcessor.java | 2 +-
java/org/apache/naming/StringManager.java | 14 ++++++------
.../tomcat/dbcp/dbcp2/managed/package-info.java | 2 +-
.../apache/tomcat/jdbc/pool/PoolConfiguration.java | 10 ++++-----
res/checkstyle/checkstyle.xml | 1 +
9 files changed, 31 insertions(+), 29 deletions(-)
diff --git a/java/jakarta/servlet/jsp/tagext/TagData.java b/java/jakarta/servlet/jsp/tagext/TagData.java
index 989a7a83df..680acf3c74 100644
--- a/java/jakarta/servlet/jsp/tagext/TagData.java
+++ b/java/jakarta/servlet/jsp/tagext/TagData.java
@@ -50,7 +50,7 @@ public class TagData implements Cloneable {
*
* All values must be Strings except for those holding the
* distinguished object REQUEST_TIME_VALUE.
-
+ *
* @param atts the static attribute and values. May be null.
*/
public TagData(Object[] atts[]) {
diff --git a/java/org/apache/catalina/realm/JAASRealm.java b/java/org/apache/catalina/realm/JAASRealm.java
index f249ae4387..c6b1dcfbfa 100644
--- a/java/org/apache/catalina/realm/JAASRealm.java
+++ b/java/org/apache/catalina/realm/JAASRealm.java
@@ -97,21 +97,23 @@ import org.apache.tomcat.util.ExceptionUtils;
* with this name in the JAAS configuration file. Here is a hypothetical
* JAAS configuration file entry for a database-oriented login module that uses
* a Tomcat-managed JNDI database resource:
- * <blockquote><pre>Catalina {
-org.foobar.auth.DatabaseLoginModule REQUIRED
- JNDI_RESOURCE=jdbc/AuthDB
- USER_TABLE=users
- USER_ID_COLUMN=id
- USER_NAME_COLUMN=name
- USER_CREDENTIAL_COLUMN=password
- ROLE_TABLE=roles
- ROLE_NAME_COLUMN=name
- PRINCIPAL_FACTORY=org.foobar.auth.impl.SimplePrincipalFactory;
-};</pre></blockquote></li>
+ * <blockquote><pre>
+ * Catalina {
+ * org.foobar.auth.DatabaseLoginModule REQUIRED
+ * JNDI_RESOURCE=jdbc/AuthDB
+ * USER_TABLE=users
+ * USER_ID_COLUMN=id
+ * USER_NAME_COLUMN=name
+ * USER_CREDENTIAL_COLUMN=password
+ * ROLE_TABLE=roles
+ * ROLE_NAME_COLUMN=name
+ * PRINCIPAL_FACTORY=org.foobar.auth.impl.SimplePrincipalFactory;
+ * };
+ * </pre></blockquote></li>
* <li>To set the JAAS configuration file
* location, set the <code>CATALINA_OPTS</code> environment variable
* similar to the following:
-<blockquote><code>CATALINA_OPTS="-Djava.security.auth.login.config=$CATALINA_HOME/conf/jaas.config"</code></blockquote>
+ * <blockquote><code>CATALINA_OPTS="-Djava.security.auth.login.config=$CATALINA_HOME/conf/jaas.config"</code></blockquote>
* </li>
* <li>As part of the login process, JAASRealm registers its own <code>CallbackHandler</code>,
* called (unsurprisingly) <code>JAASCallbackHandler</code>. This handler supplies the
diff --git a/java/org/apache/catalina/servlets/CGIServlet.java b/java/org/apache/catalina/servlets/CGIServlet.java
index cb4a7421bb..d93c7dbb36 100644
--- a/java/org/apache/catalina/servlets/CGIServlet.java
+++ b/java/org/apache/catalina/servlets/CGIServlet.java
@@ -765,7 +765,7 @@ public final class CGIServlet extends HttpServlet {
*
* @return true if the request was parsed without error, false if there
* was a problem
-
+ *
* @throws UnsupportedEncodingException Unknown encoding
*/
protected boolean setupFromRequest(HttpServletRequest req)
diff --git a/java/org/apache/catalina/webresources/TomcatURLStreamHandlerFactory.java b/java/org/apache/catalina/webresources/TomcatURLStreamHandlerFactory.java
index f2dd35ba39..d58f9d3fef 100644
--- a/java/org/apache/catalina/webresources/TomcatURLStreamHandlerFactory.java
+++ b/java/org/apache/catalina/webresources/TomcatURLStreamHandlerFactory.java
@@ -86,7 +86,6 @@ public class TomcatURLStreamHandlerFactory implements URLStreamHandlerFactory {
* successfully disabled as a result of this call.
* <code>false</code> if the factory was already registered prior
* to this call.
-
*/
public static boolean disable() {
return !getInstanceInternal(false).isRegistered();
diff --git a/java/org/apache/coyote/ajp/AjpProcessor.java b/java/org/apache/coyote/ajp/AjpProcessor.java
index a7911e75f8..3d50ae89b5 100644
--- a/java/org/apache/coyote/ajp/AjpProcessor.java
+++ b/java/org/apache/coyote/ajp/AjpProcessor.java
@@ -565,7 +565,7 @@ public class AjpProcessor extends AbstractProcessor {
* @param message The message to populate
* @param block If there is no data available to read when this method is
* called, should this call block until data becomes available?
-
+ *
* @return true if the message has been read, false if no data was read
*
* @throws IOException any other failure, including incomplete reads
diff --git a/java/org/apache/naming/StringManager.java b/java/org/apache/naming/StringManager.java
index b20094d071..c34368acd1 100644
--- a/java/org/apache/naming/StringManager.java
+++ b/java/org/apache/naming/StringManager.java
@@ -92,13 +92,13 @@ public class StringManager {
}
/**
- Get a string from the underlying resource bundle or return
- null if the String is not found.
-
- @param key to desired resource String
- @return resource String matching <i>key</i> from underlying
- bundle or null if not found.
- @throws IllegalArgumentException if <i>key</i> is null.
+ * Get a string from the underlying resource bundle or return
+ * null if the String is not found.
+ *
+ * @param key to desired resource String
+ * @return resource String matching <i>key</i> from underlying
+ * bundle or null if not found.
+ * @throws IllegalArgumentException if <i>key</i> is null.
*/
public String getString(String key) {
if(key == null){
diff --git a/java/org/apache/tomcat/dbcp/dbcp2/managed/package-info.java b/java/org/apache/tomcat/dbcp/dbcp2/managed/package-info.java
index e930cb4cc6..a439660a21 100644
--- a/java/org/apache/tomcat/dbcp/dbcp2/managed/package-info.java
+++ b/java/org/apache/tomcat/dbcp/dbcp2/managed/package-info.java
@@ -29,7 +29,7 @@
* transaction is committed or rolled back the enlisted connections are
* committed or rolled back.
* </p>
-
+ *
* <p>
* This package supports full XADataSources and non-XA data sources using
* local transaction semantics. non-XA data sources commit and rollback as
diff --git a/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolConfiguration.java b/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolConfiguration.java
index fa78f9db41..6fbf62f7fb 100644
--- a/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolConfiguration.java
+++ b/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/PoolConfiguration.java
@@ -669,11 +669,11 @@ public interface PoolConfiguration {
* Returns true if the pool sweeper is enabled for the connection pool.
* The pool sweeper is enabled if any settings that require async intervention in the pool are turned on
* <code>
- boolean result = getTimeBetweenEvictionRunsMillis()>0;
- result = result && (isRemoveAbandoned() && getRemoveAbandonedTimeout()>0);
- result = result || (isTestWhileIdle() && getValidationQuery()!=null);
- return result;
- </code>
+ * boolean result = getTimeBetweenEvictionRunsMillis()>0;
+ * result = result && (isRemoveAbandoned() && getRemoveAbandonedTimeout()>0);
+ * result = result || (isTestWhileIdle() && getValidationQuery()!=null);
+ * return result;
+ * </code>
*
* @return true if a background thread is or will be enabled for this pool
*/
diff --git a/res/checkstyle/checkstyle.xml b/res/checkstyle/checkstyle.xml
index 3d285357fd..17dd0d5b96 100644
--- a/res/checkstyle/checkstyle.xml
+++ b/res/checkstyle/checkstyle.xml
@@ -84,6 +84,7 @@
<module name="JavadocMethod">
<property name="accessModifiers" value="public,protected"/>
</module>
+ <module name="JavadocMissingLeadingAsterisk"/>
<!-- Miscellaneous -->
<!-- ~5500 errors
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org