From 66c1dcbf47fcf05a517fe3ac7180d07dbcb86960 Mon Sep 17 00:00:00 2001
From: Sebastian Bazley <sebb@apache.org>
Date: Wed, 18 Jul 2012 14:14:29 +0000
Subject: [PATCH] Redundant semi-colons

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/logging/trunk@1362959 13f79535-47bb-0310-9956-ffa450edef68
---
 src/java/org/apache/commons/logging/LogFactory.java       | 6 +++---
 src/java/org/apache/commons/logging/LogSource.java        | 4 ++--
 src/java/org/apache/commons/logging/impl/SimpleLog.java   | 8 ++++----
 .../org/apache/commons/logging/tccl/NullTCCLTestCase.java | 2 +-
 4 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/java/org/apache/commons/logging/LogFactory.java b/src/java/org/apache/commons/logging/LogFactory.java
index 342c260..93c6413 100644
--- a/src/java/org/apache/commons/logging/LogFactory.java
+++ b/src/java/org/apache/commons/logging/LogFactory.java
@@ -511,7 +511,7 @@ public abstract class LogFactory {
                         + ": [" + trim(e.getMessage())
                         + "]. Trying alternative implementations...");
             }
-            ;  // ignore
+            // ignore
         } catch(RuntimeException e) {
             // This is not consistent with the behaviour when a bad LogFactory class is
             // specified in a services file.
@@ -588,7 +588,7 @@ public abstract class LogFactory {
                         + ": [" + trim(ex.getMessage())
                         + "]. Trying alternative implementations...");
                 }
-                ; // ignore
+                // ignore
             }
         }
 
@@ -922,7 +922,7 @@ public abstract class LogFactory {
                  * we can make a distinction.
                  */
                 if (e.getTargetException() instanceof SecurityException) {
-                    ;  // ignore
+                    // ignore
                 } else {
                     // Capture 'e.getTargetException()' exception for details
                     // alternate: log 'e.getTargetException()', and pass back 'e'.
diff --git a/src/java/org/apache/commons/logging/LogSource.java b/src/java/org/apache/commons/logging/LogSource.java
index 730ca12..58c47b5 100644
--- a/src/java/org/apache/commons/logging/LogSource.java
+++ b/src/java/org/apache/commons/logging/LogSource.java
@@ -115,7 +115,7 @@ public class LogSource {
                     setLogImplementation
                             ("org.apache.commons.logging.impl.NoOpLog");
                 } catch (Throwable u) {
-                    ;
+                    // ignored
                 }
             }
         } else {
@@ -135,7 +135,7 @@ public class LogSource {
                     setLogImplementation
                             ("org.apache.commons.logging.impl.NoOpLog");
                 } catch (Throwable u) {
-                    ;
+                    // ignored
                 }
             }
         }
diff --git a/src/java/org/apache/commons/logging/impl/SimpleLog.java b/src/java/org/apache/commons/logging/impl/SimpleLog.java
index 45ea63a..aaa8794 100644
--- a/src/java/org/apache/commons/logging/impl/SimpleLog.java
+++ b/src/java/org/apache/commons/logging/impl/SimpleLog.java
@@ -140,7 +140,7 @@ public class SimpleLog implements Log, Serializable {
         try {
             prop = System.getProperty(name);
         } catch (SecurityException e) {
-            ; // Ignore
+            // Ignore
         }
         return (prop == null) ? simpleLogProps.getProperty(name) : prop;
     }
@@ -661,7 +661,7 @@ public class SimpleLog implements Log, Serializable {
                     classLoader = (ClassLoader)method.invoke(Thread.currentThread(), 
                             (Class[]) null);
                 } catch (IllegalAccessException e) {
-                    ;  // ignore
+                    // ignore
                 } catch (InvocationTargetException e) {
                     /**
                      * InvocationTargetException is thrown by 'invoke' when
@@ -680,7 +680,7 @@ public class SimpleLog implements Log, Serializable {
                      * we can make a distinction.
                      */
                     if (e.getTargetException() instanceof SecurityException) {
-                        ;  // ignore
+                        // ignore
                     } else {
                         // Capture 'e.getTargetException()' exception for details
                         // alternate: log 'e.getTargetException()', and pass back 'e'.
@@ -690,7 +690,7 @@ public class SimpleLog implements Log, Serializable {
                 }
             } catch (NoSuchMethodException e) {
                 // Assume we are running on JDK 1.1
-                ;  // ignore
+                // ignore
             }
         }
 
diff --git a/src/test/org/apache/commons/logging/tccl/NullTCCLTestCase.java b/src/test/org/apache/commons/logging/tccl/NullTCCLTestCase.java
index 8aded64..d7b5271 100644
--- a/src/test/org/apache/commons/logging/tccl/NullTCCLTestCase.java
+++ b/src/test/org/apache/commons/logging/tccl/NullTCCLTestCase.java
@@ -28,7 +28,7 @@ import junit.framework.TestCase;
  */
 public class NullTCCLTestCase extends TestCase {
 
-    public static Test suite() throws Exception {;
+    public static Test suite() throws Exception {
         PathableTestSuite suite = new PathableTestSuite(NullTCCLTestCase.class, null);
         return suite;
     }