(tomcat) 02/02: Remove TODOs and add comment on Ranges vs Range

[markt]

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

commit a3982f33978d5c3e07668858b1e0f813578dda75
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Nov 27 18:09:06 2024 +0000

    Remove TODOs and add comment on Ranges vs Range
---
 java/org/apache/catalina/servlets/DefaultServlet.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/java/org/apache/catalina/servlets/DefaultServlet.java 
b/java/org/apache/catalina/servlets/DefaultServlet.java
index db1cb6dbc9..ba9d4b0f51 100644
--- a/java/org/apache/catalina/servlets/DefaultServlet.java
+++ b/java/org/apache/catalina/servlets/DefaultServlet.java
@@ -1381,8 +1381,8 @@ public class DefaultServlet extends HttpServlet {
             return null;
         }
 
-        // TODO: Remove the internal representation and use Ranges
-        // Convert to internal representation
+        // Can't update to use Ranges as this is a protected method. Only 
backwards compatible API changes are allowed.
+        // Convert to Range
         Range range = new Range();
         range.start = contentRange.getStart();
         range.end = contentRange.getEnd();
@@ -1478,8 +1478,8 @@ public class DefaultServlet extends HttpServlet {
             return FULL;
         }
 
-        // TODO: Remove the internal representation and use Ranges
-        // Convert to internal representation
+        // Can't update to use Ranges as this is a protected method. Only 
backwards compatible API changes are allowed.
+        // Convert to Range
         ArrayList<Range> result = new ArrayList<>();
 
         List<long[]> rangeContext = new ArrayList<>();


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