You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by lu...@apache.org on 2011/10/07 09:26:26 UTC
svn commit: r1179948 - in /commons/proper/math/trunk: checkstyle.xml
src/main/java/org/apache/commons/math/util/FastMath.java
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
+
}