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

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-logging.git


The following commit(s) were added to refs/heads/master by this push:
     new 736d1c3  Fix typos in comments
736d1c3 is described below

commit 736d1c3fa7584f63c59e8410773f2b15dc00a6f8
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Wed Aug 21 09:17:35 2024 -0400

    Fix typos in comments
---
 src/main/java/org/apache/commons/logging/LogFactory.java              | 4 ++--
 src/main/java/org/apache/commons/logging/impl/LogFactoryImpl.java     | 4 ++--
 .../java/org/apache/commons/logging/security/MockSecurityManager.java | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/commons/logging/LogFactory.java 
b/src/main/java/org/apache/commons/logging/LogFactory.java
index 5b09c22..d551f19 100644
--- a/src/main/java/org/apache/commons/logging/LogFactory.java
+++ b/src/main/java/org/apache/commons/logging/LogFactory.java
@@ -751,7 +751,7 @@ public abstract class LogFactory {
         // If the properties file exists, then its contents are used as
         // "attributes" on the LogFactory implementation class. One particular
         // property may also control which LogFactory concrete subclass is
-        // used, but only if other discovery mechanisms fail..
+        // used, but only if other discovery mechanisms fail.
         //
         // As the properties file (if it exists) will be used one way or
         // another in the end we may as well look for it first.
@@ -1118,7 +1118,7 @@ public abstract class LogFactory {
             }
         } catch (final SecurityException ex) {
             // We must be running in some very secure environment.
-            // We just have to assume output is not wanted..
+            // We just have to assume output is not wanted.
             return null;
         }
 
diff --git a/src/main/java/org/apache/commons/logging/impl/LogFactoryImpl.java 
b/src/main/java/org/apache/commons/logging/impl/LogFactoryImpl.java
index e52d465..bcaec92 100644
--- a/src/main/java/org/apache/commons/logging/impl/LogFactoryImpl.java
+++ b/src/main/java/org/apache/commons/logging/impl/LogFactoryImpl.java
@@ -367,7 +367,7 @@ public class LogFactoryImpl extends LogFactory {
                 } catch (final ClassNotFoundException 
originalClassNotFoundException) {
                     // The current class loader was unable to find the log 
adapter
                     // in this or any ancestor class loader. There's no point 
in
-                    // trying higher up in the hierarchy in this case..
+                    // trying higher up in the hierarchy in this case.
                     String msg = originalClassNotFoundException.getMessage();
                     logDiagnostic("The log adapter '" + logAdapterClassName + 
"' is not available via class loader " +
                                   objectId(currentCL) + ": " + trim(msg));
@@ -418,7 +418,7 @@ public class LogFactoryImpl extends LogFactory {
                 // other classes that could not be found. This simply means 
that
                 // the underlying logger library is not present in this or any
                 // ancestor class loader. There's no point in trying higher up
-                // in the hierarchy in this case..
+                // in the hierarchy in this case.
                 final String msg = e.getMessage();
                 logDiagnostic("The log adapter '" + logAdapterClassName +
                               "' is missing dependencies when loaded via class 
loader " + objectId(currentCL) +
diff --git 
a/src/test/java/org/apache/commons/logging/security/MockSecurityManager.java 
b/src/test/java/org/apache/commons/logging/security/MockSecurityManager.java
index 0486fb7..0704bff 100644
--- a/src/test/java/org/apache/commons/logging/security/MockSecurityManager.java
+++ b/src/test/java/org/apache/commons/logging/security/MockSecurityManager.java
@@ -113,7 +113,7 @@ public class MockSecurityManager extends SecurityManager {
                 //
                 // TODO: maybe check class is loaded via system loader or 
similar rather
                 // than checking name? Trusted domains may be different in 
alternative
-                // jvms..
+                // jvms.
             } else if 
(cname.startsWith("org.apache.commons.logging.security")) {
                 // this is the unit test code; treat this like an untrusted 
client
                 // app that is using JCL

Reply via email to