You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2024/09/02 13:21:54 UTC
(tomcat) branch 10.1.x updated: Fix unfortunate typo ("no" was missing)
This is an automated email from the ASF dual-hosted git repository.
remm 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 0a40f40e12 Fix unfortunate typo ("no" was missing)
0a40f40e12 is described below
commit 0a40f40e1266fdb4b3bff5456446323ce6f13373
Author: remm <re...@apache.org>
AuthorDate: Mon Sep 2 15:16:40 2024 +0200
Fix unfortunate typo ("no" was missing)
Now flagged by Coverity, so still thinking about how to do it better if
feasible.
---
java/org/apache/catalina/core/StandardContext.java | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/java/org/apache/catalina/core/StandardContext.java b/java/org/apache/catalina/core/StandardContext.java
index d181a99a7b..3b7b61dc4f 100644
--- a/java/org/apache/catalina/core/StandardContext.java
+++ b/java/org/apache/catalina/core/StandardContext.java
@@ -2136,10 +2136,8 @@ public class StandardContext extends ContainerBase implements Context, Notificat
@Override
public ServletContext getServletContext() {
- /*
- * This method is called (multiple times) during context start which is single threaded so there is concurrency
- * issue here.
- */
+ // This method is called multiple times during context start which is single threaded
+ // so there is no concurrency issue
if (context == null) {
context = new ApplicationContext(this);
if (altDDName != null) {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org