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-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
