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 16:32:11 UTC
[tomcat] branch 9.0.x updated: CheckStyle Javadoc checks += JavadocMethod
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 d97b343258 CheckStyle Javadoc checks += JavadocMethod
d97b343258 is described below
commit d97b34325818695527f67b6a920b049d92733683
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Thu Jan 12 16:10:41 2023 +0000
CheckStyle Javadoc checks += JavadocMethod
---
java/org/apache/jasper/compiler/Node.java | 4 ++++
res/checkstyle/checkstyle.xml | 3 +++
.../classes/compressionFilters/CompressionResponseStream.java | 5 +++++
3 files changed, 12 insertions(+)
diff --git a/java/org/apache/jasper/compiler/Node.java b/java/org/apache/jasper/compiler/Node.java
index 510fe9edb1..842f150b14 100644
--- a/java/org/apache/jasper/compiler/Node.java
+++ b/java/org/apache/jasper/compiler/Node.java
@@ -2413,6 +2413,8 @@ abstract class Node implements TagConstants {
/**
* This method provides a place to put actions that are common to all
* nodes. Override this in the child visitor class if need to.
+ *
+ * @param n The node to visit
*/
@SuppressWarnings("unused")
protected void doVisit(Node n) throws JasperException {
@@ -2421,6 +2423,8 @@ abstract class Node implements TagConstants {
/**
* Visit the body of a node, using the current visitor
+ *
+ * @param n The node to visit
*/
protected void visitBody(Node n) throws JasperException {
if (n.getBody() != null) {
diff --git a/res/checkstyle/checkstyle.xml b/res/checkstyle/checkstyle.xml
index bc25876b03..6d345bc4ef 100644
--- a/res/checkstyle/checkstyle.xml
+++ b/res/checkstyle/checkstyle.xml
@@ -81,6 +81,9 @@
<module name="InvalidJavadocPosition"/>
<module name="JavadocBlockTagLocation"/>
<module name="JavadocContentLocationCheck"/>
+ <module name="JavadocMethod">
+ <property name="accessModifiers" value="public,protected"/>
+ </module>
<!-- Miscellaneous -->
<!-- ~5500 errors
diff --git a/webapps/examples/WEB-INF/classes/compressionFilters/CompressionResponseStream.java b/webapps/examples/WEB-INF/classes/compressionFilters/CompressionResponseStream.java
index 7485c5204d..6b6e850091 100644
--- a/webapps/examples/WEB-INF/classes/compressionFilters/CompressionResponseStream.java
+++ b/webapps/examples/WEB-INF/classes/compressionFilters/CompressionResponseStream.java
@@ -122,6 +122,9 @@ public class CompressionResponseStream extends ServletOutputStream {
/**
* Set the compressionThreshold number and create buffer for this size
+ *
+ * @param compressionThreshold Responses above this size in bytes will be
+ * compressed
*/
protected void setCompressionThreshold(int compressionThreshold) {
this.compressionThreshold = compressionThreshold;
@@ -133,6 +136,8 @@ public class CompressionResponseStream extends ServletOutputStream {
/**
* The compression buffer size to avoid chunking
+ *
+ * @param compressionBuffer The compression buffer size in bytes
*/
protected void setCompressionBuffer(int compressionBuffer) {
this.compressionBuffer = compressionBuffer;
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org