svn commit: r1759224 - /tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java

[ma...@apache.org]

Author: markt
Date: Mon Sep  5 10:22:32 2016
New Revision: 1759224

URL: http://svn.apache.org/viewvc?rev=1759224&view=rev
Log:
MUST_DISPATCH is not a state that requires non-container threads to be paused.
MUST_DISPATCH transitions to DISPATCHING in postProcess(). A call to complete() or dispatch() with the state in DISPATCHING triggers an IllegalStateException. The behaviour is the same if the state is MUST_DISPATCH. Therefore, there is no difference in end result whether the non-container thread is paused or not. Therefore, don't pause it so the ISE is triggered earlier rather than later.

Modified:
    tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java

Modified: tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java?rev=1759224&r1=1759223&r2=1759224&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java [UTF-8] (original)
+++ tomcat/trunk/java/org/apache/coyote/AsyncStateMachine.java [UTF-8] Mon Sep  5 10:22:32 2016
@@ -117,7 +117,7 @@ public class AsyncStateMachine {
         MUST_COMPLETE(true,  true,  true,  false, false),
         COMPLETING   (true,  false, true,  false, false),
         TIMING_OUT   (true,  true,  false, false, false),
-        MUST_DISPATCH(true,  true,  false, true,  true),
+        MUST_DISPATCH(true,  true,  false, true,  false),
         DISPATCHING  (true,  false, false, true,  false),
         READ_WRITE_OP(true,  true,  false, false, true),
         ERROR        (true,  true,  false, false, false);



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