You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2023/07/26 16:19:43 UTC
[tomcat] branch 10.1.x updated: Update comment
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new efbeae4f4a Update comment
efbeae4f4a is described below
commit efbeae4f4a86f0d9c49543a6a80ba0052259de14
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Jul 26 17:19:22 2023 +0100
Update comment
---
java/org/apache/coyote/http2/Http2UpgradeHandler.java | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/java/org/apache/coyote/http2/Http2UpgradeHandler.java b/java/org/apache/coyote/http2/Http2UpgradeHandler.java
index ec3e098c0d..811737bf83 100644
--- a/java/org/apache/coyote/http2/Http2UpgradeHandler.java
+++ b/java/org/apache/coyote/http2/Http2UpgradeHandler.java
@@ -1338,9 +1338,10 @@ class Http2UpgradeHandler extends AbstractStream implements InternalHttpUpgradeH
Stream pushStream;
- // Synchronized since PUSH_PROMISE frames have to be sent in order. Once
- // the stream has been created we need to ensure that the PUSH_PROMISE
- // is sent before the next stream is created for a PUSH_PROMISE.
+ /*
+ * Uses SocketWrapper lock since PUSH_PROMISE frames have to be sent in order. Once the stream has been created
+ * we need to ensure that the PUSH_PROMISE is sent before the next stream is created for a PUSH_PROMISE.
+ */
Lock lock = socketWrapper.getLock();
lock.lock();
try {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org