From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Tue, 7 Dec 2010 15:43:37 +0000 (+0000)
Subject: Add checkstyle rule for modifier order and fix remaining issues in current code base
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=02f1ed4b0771a4fb0c05632de1f63fb214337262;p=tomcat7.0

Add checkstyle rule for modifier order and fix remaining issues in current code base

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1043106 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/build.xml b/build.xml
index 3428cdcb4..06a22aa60 100644
--- a/build.xml
+++ b/build.xml
@@ -431,6 +431,8 @@
         <exclude name="output/**"/>
         <exclude name="modules/**"/>
         <exclude name="**/*.mdl"/>
+        <!-- Exclude auto-generated files with known errors -->
+        <exclude name="java/org/apache/el/parser/ELParser.java" />
         <!-- Exclude these else Gump runs validate on them -->
         <exclude name="**/org/apache/tomcat/dbcp/**"/>
         <exclude name="**/tomcat-deps/**"/>
diff --git a/checkstyle.xml b/checkstyle.xml
index 59d2e8095..325d24f07 100644
--- a/checkstyle.xml
+++ b/checkstyle.xml
@@ -44,9 +44,7 @@
     <module name="UnusedImports"/>
     
     <!--  Modifiers -->
-    <!-- ~250 errors
     <module name="ModifierOrder"/>
-    -->
     <!-- ~1750 errors
     <module name="RedundantModifier"/>
     -->
diff --git a/test/org/apache/catalina/valves/Benchmarks.java b/test/org/apache/catalina/valves/Benchmarks.java
index 004fe46e0..b46eb5321 100644
--- a/test/org/apache/catalina/valves/Benchmarks.java
+++ b/test/org/apache/catalina/valves/Benchmarks.java
@@ -182,7 +182,7 @@ public class Benchmarks extends TestCase {
         benchmark.doTest(5, tests);
     }
 
-    private static abstract class TimeDateElementBenchmarkTestBase {
+    private abstract static class TimeDateElementBenchmarkTestBase {
         protected static final String months[] = { "Jan", "Feb", "Mar", "Apr",
                 "May", "Jun", "Jul", "Aug", "Sep", "Oct", "Nov", "Dec" };
 
diff --git a/test/org/apache/tomcat/util/http/CookiesBaseTest.java b/test/org/apache/tomcat/util/http/CookiesBaseTest.java
index d8648a1fd..b5a492179 100644
--- a/test/org/apache/tomcat/util/http/CookiesBaseTest.java
+++ b/test/org/apache/tomcat/util/http/CookiesBaseTest.java
@@ -30,7 +30,7 @@ import org.apache.catalina.startup.TomcatBaseTest;
 
 /**
  * Base Test case for {@link Cookies}. <b>Note</b> because of the use of
- * <code>final static</code> constants in {@link Cookies}, each of these tests
+ * <code>static final</code> constants in {@link Cookies}, each of these tests
  * must be executed in a new JVM instance. The tests have been place in separate
  * classes to facilitate this when running the unit tests via Ant.
  */