Code clean-up. Remove unnecessary ;'s id'd by Eclipse
authormarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Thu, 15 Jan 2009 13:31:05 +0000 (13:31 +0000)
committermarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Thu, 15 Jan 2009 13:31:05 +0000 (13:31 +0000)
git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@734700 13f79535-47bb-0310-9956-ffa450edef68

13 files changed:
java/org/apache/catalina/startup/Catalina.java
java/org/apache/catalina/startup/ExpandWar.java
java/org/apache/catalina/startup/PasswdUserDatabase.java
java/org/apache/catalina/startup/TldConfig.java
java/org/apache/catalina/valves/ErrorReportValve.java
java/org/apache/tomcat/util/digester/Digester.java
java/org/apache/tomcat/util/digester/Rule.java
java/org/apache/tomcat/util/http/Cookies.java
java/org/apache/tomcat/util/http/FastHttpDateFormat.java
java/org/apache/tomcat/util/modeler/BaseModelMBean.java
java/org/apache/tomcat/util/modeler/ManagedBean.java
java/org/apache/tomcat/util/net/DefaultServerSocketFactory.java
java/org/apache/tomcat/util/net/NioBlockingSelector.java

index df58eba..4c66f9b 100644 (file)
@@ -480,7 +480,7 @@ public class Catalina extends Embedded {
             inputStream = new FileInputStream(file);
             inputSource = new InputSource("file://" + file.getAbsolutePath());
         } catch (Exception e) {
-            ;
+            // Ignore
         }
         if (inputStream == null) {
             try {
@@ -490,7 +490,7 @@ public class Catalina extends Embedded {
                     (getClass().getClassLoader()
                      .getResource(getConfigFile()).toString());
             } catch (Exception e) {
-                ;
+                // Ignore
             }
         }
 
@@ -504,7 +504,7 @@ public class Catalina extends Embedded {
                 (getClass().getClassLoader()
                         .getResource("server-embed.xml").toString());
             } catch (Exception e) {
-                ;
+                // Ignore
             }
         }
         
index 9233970..068f72f 100644 (file)
@@ -174,7 +174,7 @@ public class ExpandWar {
                 try {
                     input.close();
                 } catch (Throwable t) {
-                    ;
+                    // Ignore
                 }
                 input = null;
             }
@@ -182,7 +182,7 @@ public class ExpandWar {
                 try {
                     jarFile.close();
                 } catch (Throwable t) {
-                    ;
+                    // Ignore
                 }
                 jarFile = null;
             }
index a99227e..3c24b33 100644 (file)
@@ -182,7 +182,7 @@ public final class PasswdUserDatabase
                 try {
                     reader.close();
                 } catch (IOException f) {
-                    ;
+                    // Ignore
                 }
                 reader = null;
             }
index 32ce34e..cb68589 100644 (file)
@@ -626,7 +626,7 @@ public final class TldConfig  implements LifecycleListener {
                 }
             }
         } catch (NamingException e) {
-            // Silent catch: it's valid that no /WEB-INF directory exists
+            // Silent catch: it's valid that no /WEB-INF directory exists
         }
     }
 
index 4b140d7..c4d762d 100644 (file)
@@ -117,7 +117,7 @@ public class ErrorReportValve
             try {
                 response.reset();
             } catch (IllegalStateException e) {
-                ;
+                // Ignore
             }
 
             response.sendError
@@ -130,7 +130,7 @@ public class ErrorReportValve
         try {
             report(request, response, throwable);
         } catch (Throwable tt) {
-            ;
+            // Ignore
         }
 
     }
@@ -167,7 +167,7 @@ public class ErrorReportValve
         try {
             report = sm.getString("http." + statusCode, message);
         } catch (Throwable t) {
-            ;
+            // Ignore
         }
         if (report == null)
             return;
@@ -254,9 +254,9 @@ public class ErrorReportValve
                 writer.write(sb.toString());
             }
         } catch (IOException e) {
-            ;
+            // Ignore
         } catch (IllegalStateException e) {
-            ;
+            // Ignore
         }
         
     }
index 6c7b4e4..36d464a 100644 (file)
@@ -1153,7 +1153,7 @@ public class Digester extends DefaultHandler {
                     new String(buffer, start, len) + ")");
         }
 
-        ;   // No processing required
+        // No processing required
 
     }
 
@@ -1173,7 +1173,7 @@ public class Digester extends DefaultHandler {
             saxLog.debug("processingInstruction('" + target + "','" + data + "')");
         }
 
-        ;   // No processing is required
+        // No processing is required
 
     }
 
@@ -1218,7 +1218,7 @@ public class Digester extends DefaultHandler {
             saxLog.debug("skippedEntity(" + name + ")");
         }
 
-        // No processing required
+        // No processing required
 
     }
 
@@ -2590,7 +2590,7 @@ public class Digester extends DefaultHandler {
     protected void initialize() {
 
         // Perform lazy initialization as needed
-        // Nothing required by default
+        // Nothing required by default
 
     }    
 
index 8583e67..8070ea3 100644 (file)
@@ -113,9 +113,7 @@ public abstract class Rule {
      *   parameters instead.
      */
     public void begin(Attributes attributes) throws Exception {
-
-        ;      // The default implementation does nothing
-
+        // The default implementation does nothing
     }
 
 
@@ -153,9 +151,7 @@ public abstract class Rule {
      *   instead.
      */
     public void body(String text) throws Exception {
-
-        ;      // The default implementation does nothing
-
+        // The default implementation does nothing
     }
 
 
@@ -190,9 +186,7 @@ public abstract class Rule {
      *   <code>namespace</code> and <code>name</code> parameters instead.
      */
     public void end() throws Exception {
-
-        ;      // The default implementation does nothing
-
+        // The default implementation does nothing
     }
 
 
@@ -223,9 +217,7 @@ public abstract class Rule {
      * called, to allow Rules to remove temporary data.
      */
     public void finish() throws Exception {
-
-        ;      // The default implementation does nothing
-
+        // The default implementation does nothing
     }
 
 
index 2739d2d..dba47c8 100644 (file)
@@ -370,7 +370,7 @@ public final class Cookies { // extends MultiMap {
             pos = nameEnd = getTokenEndPosition(bytes,pos,end);
 
             // Skip whitespace
-            while (pos < end && isWhiteSpace(bytes[pos])) {pos++; }; 
+            while (pos < end && isWhiteSpace(bytes[pos])) {pos++; } 
          
 
             // Check for an '=' -- This could also be a name-only
@@ -390,7 +390,7 @@ public final class Cookies { // extends MultiMap {
                 // Determine what type of value this is, quoted value,
                 // token, name-only with an '=', or other (bad)
                 switch (bytes[pos]) {
-                case '"':; // Quoted Value
+                case '"': // Quoted Value
                     isQuoted = true;
                     valueStart=pos + 1; // strip "
                     // getQuotedValue returns the position before 
@@ -413,7 +413,7 @@ public final class Cookies { // extends MultiMap {
                     valueStart = valueEnd = -1;
                     // The position is OK (On a delimiter)
                     break;
-                default:;
+                default:
                     if (!isSeparator(bytes[pos])) {
                         // Token
                         valueStart=pos;
@@ -429,7 +429,7 @@ public final class Cookies { // extends MultiMap {
                         log("Invalid cookie. Value not a token or quoted value");
                         while (pos < end && bytes[pos] != ';' && 
                                bytes[pos] != ',') 
-                            {pos++; };
+                            {pos++; }
                         pos++;
                         // Make sure no special avpairs can be attributed to 
                         // the previous cookie by setting the current cookie
@@ -450,7 +450,7 @@ public final class Cookies { // extends MultiMap {
             // in a good state.
   
             // Skip whitespace
-            while (pos < end && isWhiteSpace(bytes[pos])) {pos++; }
+            while (pos < end && isWhiteSpace(bytes[pos])) {pos++; }
 
 
             // Make sure that after the cookie we have a separator. This
@@ -557,7 +557,7 @@ public final class Cookies { // extends MultiMap {
      */
     public static final int getTokenEndPosition(byte bytes[], int off, int end){
         int pos = off;
-        while (pos < end && !isSeparator(bytes[pos])) {pos++; };
+        while (pos < end && !isSeparator(bytes[pos])) {pos++; }
         
         if (pos > end)
             return end;
index 8fd124c..ccd38ae 100644 (file)
@@ -190,7 +190,7 @@ public final class FastHttpDateFormat {
             try {
                 date = formats[i].parse(value);
             } catch (ParseException e) {
-                ;
+                // Ignore
             }
         }
         if (date == null) {
index e4192a0..d986eda 100644 (file)
@@ -231,8 +231,8 @@ public class BaseModelMBean implements DynamicMBean, MBeanRegistration, ModelMBe
             try {
                 response.add(new Attribute(names[i],getAttribute(names[i])));
             } catch (Exception e) {
-                // Not having a particular attribute in the response
-                // is the indication of a getter problem
+                // Not having a particular attribute in the response
+                // is the indication of a getter problem
             }
         }
         return (response);
@@ -474,7 +474,7 @@ public class BaseModelMBean implements DynamicMBean, MBeanRegistration, ModelMBe
             try {
                 setAttribute(item);
             } catch (Exception e) {
-                // Ignore all exceptions
+                // Ignore all exceptions
             }
         }
 
index 05d3090..9ea285c 100644 (file)
@@ -501,7 +501,7 @@ public class ManagedBean implements java.io.Serializable
                 object = mbean;
                 m = object.getClass().getMethod(getMethod, NO_ARGS_PARAM_SIG);
             } catch (NoSuchMethodException e) {
-                exception = e;;
+                exception = e;
             }
             if( m== null && resource != null ) {
                 try {
@@ -548,7 +548,7 @@ public class ManagedBean implements java.io.Serializable
                 object = bean;
                 m = object.getClass().getMethod(setMethod, signature);
             } catch (NoSuchMethodException e) {
-                exception = e;;
+                exception = e;
             }
             if( m== null && resource != null ) {
                 try {
@@ -608,7 +608,6 @@ public class ManagedBean implements java.io.Serializable
                 method = object.getClass().getMethod(aname, types);
             } catch (NoSuchMethodException e) {
                 exception = e;
-                ;
             }
             try {
                 if ((method == null) && (resource != null)) {
index 38b606d..c122990 100644 (file)
@@ -63,7 +63,7 @@ class DefaultServerSocketFactory extends ServerSocketFactory {
  
     public void handshake(Socket sock)
        throws IOException {
-       ; // NOOP
+        // NOOP
     }
            
         
index 85dcd70..86d83b7 100644 (file)
@@ -341,7 +341,7 @@ public class NioBlockingSelector {
                         KeyAttachment attachment = (KeyAttachment)sk.attachment();
                         try {
                             attachment.access();
-                            iterator.remove(); ;
+                            iterator.remove();
                             sk.interestOps(sk.interestOps() & (~sk.readyOps()));
                             if ( sk.isReadable() ) {
                                 countDown(attachment.getReadLatch());