From 3e4348ec2ca66f056ed8260cf420099c08403b7f Mon Sep 17 00:00:00 2001
From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Mon, 6 Sep 2010 16:53:34 +0000
Subject: [PATCH] Reduce visibility, remove unused/unnecessary methods prior to
 fixing bz49884

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@993096 13f79535-47bb-0310-9956-ffa450edef68
---
 .../org/apache/catalina/core/AsyncContextImpl.java | 28 +++++-----------------
 1 file changed, 6 insertions(+), 22 deletions(-)

diff --git a/java/org/apache/catalina/core/AsyncContextImpl.java b/java/org/apache/catalina/core/AsyncContextImpl.java
index 22c532074..cf3745255 100644
--- a/java/org/apache/catalina/core/AsyncContextImpl.java
+++ b/java/org/apache/catalina/core/AsyncContextImpl.java
@@ -101,7 +101,7 @@ public class AsyncContextImpl implements AsyncContext {
 
     @Override
     public void dispatch() {
-        HttpServletRequest sr = (HttpServletRequest)getServletRequest();
+        HttpServletRequest sr = (HttpServletRequest)getRequest();
         String path = sr.getRequestURI();
         String cpath = sr.getContextPath();
         if (cpath.length()>1) path = path.substring(cpath.length());
@@ -171,12 +171,12 @@ public class AsyncContextImpl implements AsyncContext {
 
     @Override
     public ServletRequest getRequest() {
-        return getServletRequest();
+        return servletRequest;
     }
 
     @Override
     public ServletResponse getResponse() {
-        return getServletResponse();
+        return servletResponse;
     }
 
     @Override
@@ -277,14 +277,6 @@ public class AsyncContextImpl implements AsyncContext {
         }
     }
 
-    public ServletRequest getServletRequest() {
-        return servletRequest;
-    }
-
-    public ServletResponse getServletResponse() {
-        return servletResponse;
-    }
-
     @Override
     public boolean hasOriginalRequestAndResponse() {
         return hasOriginalRequestAndResponse;
@@ -294,11 +286,7 @@ public class AsyncContextImpl implements AsyncContext {
         this.hasOriginalRequestAndResponse = hasOriginalRequestAndResponse;
     }
 
-    public boolean isCompleted() {
-        return (state.get()==AsyncState.NOT_STARTED);
-    }
-
-    public void doInternalDispatch() throws ServletException, IOException {
+    protected void doInternalDispatch() throws ServletException, IOException {
         if (this.state.compareAndSet(AsyncState.TIMING_OUT,
                 AsyncState.TIMING_OUT_NEED_COMPLETE)) {
             log.debug("TIMING OUT!");
@@ -360,8 +348,8 @@ public class AsyncContextImpl implements AsyncContext {
         }
     }
     
-    public void doInternalComplete(boolean error) {
-        if (isCompleted()) return;
+    private void doInternalComplete(boolean error) {
+        if (state.get()==AsyncState.NOT_STARTED) return;
         if (state.compareAndSet(AsyncState.STARTED, AsyncState.NOT_STARTED)) {
             //this is the same as
             //request.startAsync().complete();
@@ -392,10 +380,6 @@ public class AsyncContextImpl implements AsyncContext {
         return state.get();
     }
     
-    protected void setState(AsyncState st) {
-        state.set(st);
-    }
-    
     @Override
     public long getTimeout() {
         return timeout;
-- 
2.11.0