commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From l..@apache.org
Subject svn commit: r1179948 - in /commons/proper/math/trunk: checkstyle.xml src/main/java/org/apache/commons/math/util/FastMath.java
Date Fri, 07 Oct 2011 07:26:26 GMT
Author: luc
Date: Fri Oct  7 07:26:26 2011
New Revision: 1179948

URL: http://svn.apache.org/viewvc?rev=1179948&view=rev
Log:
Added checkstyle filter for declaration orders in FastMath.

FastMath uses very large tables which are located at end of class for
better readability. this is not a standard layout, but is well suited
for this case, hence we explicitly suppress checkstyle warning for these
tables.

Modified:
    commons/proper/math/trunk/checkstyle.xml
    commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java

Modified: commons/proper/math/trunk/checkstyle.xml
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/checkstyle.xml?rev=1179948&r1=1179947&r2=1179948&view=diff
==============================================================================
--- commons/proper/math/trunk/checkstyle.xml (original)
+++ commons/proper/math/trunk/checkstyle.xml Fri Oct  7 07:26:26 2011
@@ -183,6 +183,11 @@
     <property name="onCommentFormat"  value="CHECKSTYLE\: resume IllegalCatch"/>
     <property name="checkFormat"      value="IllegalCatch"/>
   </module>
+  <module name="SuppressionCommentFilter">
+    <property name="offCommentFormat" value="CHECKSTYLE\: stop DeclarationOrder"/>
+    <property name="onCommentFormat"  value="CHECKSTYLE\: resume DeclarationOrder"/>
+    <property name="checkFormat"      value="DeclarationOrder"/>
+  </module>
 
 </module>
 

Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java?rev=1179948&r1=1179947&r2=1179948&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java (original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math/util/FastMath.java Fri
Oct  7 07:26:26 2011
@@ -3656,7 +3656,7 @@ public class FastMath {
 
     //  ===================================== large static tables ====================================
 
-
+    //CHECKSTYLE: stop DeclarationOrder
     /** Index of exp(0) in the array of integer exponentials. */
     static final int EXP_INT_TABLE_MAX_INDEX = 750;
 
@@ -9851,4 +9851,6 @@ public class FastMath {
         }
     }
 
+    //CHECKSTYLE: resume DeclarationOrder
+
 }



Mime
View raw message