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