[tomcat] branch master updated: Fix (hopefully) intermittent Gump failure

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/master by this push:
     new 9144e85  Fix (hopefully) intermittent Gump failure
9144e85 is described below

commit 9144e854def9fe213382a87fca6d9c68ff3ed531
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Mon Feb 17 16:57:57 2020 +0000

    Fix (hopefully) intermittent Gump failure
    
    result needs to be volatile because it is written from the container
    thread and read from the test thread (like done)
---
 test/org/apache/catalina/core/TestAsyncContextImpl.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/org/apache/catalina/core/TestAsyncContextImpl.java b/test/org/apache/catalina/core/TestAsyncContextImpl.java
index cd90776..80beac6 100644
--- a/test/org/apache/catalina/core/TestAsyncContextImpl.java
+++ b/test/org/apache/catalina/core/TestAsyncContextImpl.java
@@ -310,7 +310,7 @@ public class TestAsyncContextImpl extends TomcatBaseTest {
 
         private volatile boolean done = false;
 
-        private StringBuilder result;
+        private volatile StringBuilder result;
 
         public static final long THREAD_SLEEP_TIME = 1000;
 


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