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