You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2015/12/30 15:41:45 UTC

svn commit: r1722336 - /sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestDispatcher.java

Author: cziegeler
Date: Wed Dec 30 14:41:45 2015
New Revision: 1722336

URL: http://svn.apache.org/viewvc?rev=1722336&view=rev
Log:
Remove obsolete comment

Modified:
    sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestDispatcher.java

Modified: sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestDispatcher.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestDispatcher.java?rev=1722336&r1=1722335&r2=1722336&view=diff
==============================================================================
--- sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestDispatcher.java (original)
+++ sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/request/SlingRequestDispatcher.java Wed Dec 30 14:41:45 2015
@@ -61,6 +61,7 @@ public class SlingRequestDispatcher impl
         this.path = resource.getPath();
     }
 
+    @Override
     public void include(ServletRequest request, ServletResponse sResponse)
             throws ServletException, IOException {
 
@@ -117,6 +118,7 @@ public class SlingRequestDispatcher impl
         }
     }
 
+    @Override
     public void forward(ServletRequest request, ServletResponse response)
             throws ServletException, IOException {
 
@@ -168,13 +170,6 @@ public class SlingRequestDispatcher impl
 
     private void dispatch(ServletRequest request, ServletResponse sResponse,
             boolean include) throws ServletException, IOException {
-
-        /**
-         * TODO: I have made some quick fixes in this method for SLING-221 and
-         * SLING-222, but haven't had time to do a proper review. This method
-         * might deserve a more extensive rewrite.
-         */
-
         SlingHttpServletRequest cRequest = RequestData.unwrap(request);
         RequestData rd = RequestData.getRequestData(cRequest);
         String absPath = getAbsolutePath(cRequest, path);