This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 8.5.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit 470339e34ffea488863c043cc1b216392c8312d6
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Fri Sep 15 22:54:26 2023 +0100
Better Javadoc / comments
---
java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItem.java | 6 +++---
.../tomcat/util/http/fileupload/disk/DiskFileItemFactory.java | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItem.java
b/java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItem.java
index 7fcd7f7653..1f6a70af95 100644
--- a/java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItem.java
+++ b/java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItem.java
@@ -52,8 +52,8 @@ import org.apache.tomcat.util.http.fileupload.util.Streams;
* {@link #getInputStream()} and process the file without attempting to load
* it into memory, which may come handy with large files.
*
- * <p>Temporary files, which are created for file items, should be
- * deleted later on.</p>
+ * <p>Temporary files, which are created for file items, will be deleted when
+ * the associated request is recycled.</p>
*
* @since 1.1
*/
@@ -192,7 +192,7 @@ public class DiskFileItem
public InputStream getInputStream()
throws IOException {
if (!isInMemory()) {
- // Uses old code to avoid JVM bug
+ // Avoid JVM bug
// https://bz.apache.org/bugzilla/show_bug.cgi?id=65710
return new FileInputStream(dfos.getFile());
}
diff --git
a/java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItemFactory.java
b/java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItemFactory.java
index a6bd67e943..6d61c0d351 100644
--- a/java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItemFactory.java
+++ b/java/org/apache/tomcat/util/http/fileupload/disk/DiskFileItemFactory.java
@@ -47,12 +47,12 @@ import
org.apache.tomcat.util.http.fileupload.FileItemFactory;
* implementation in an environment with local, untrusted users,
* {@link #setRepository(File)} MUST be used to configure a repository location
* that is not publicly writable. In a Servlet container the location
identified
- * by the ServletContext attribute {@code java.servlet.context.tempdir}
+ * by the ServletContext attribute {@code javax.servlet.context.tempdir}
* may be used.
* </p>
*
- * <p>Temporary files, which are created for file items, should be
- * deleted later on.</p>
+ * <p>Temporary files, which are created for file items, will be deleted when
+ * the associated request is recycled.</p>
*
* @since 1.1
*/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
