You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by ga...@apache.org on 2014/08/27 02:38:36 UTC

git commit: Enforce no redundant modifiers via Checkstyle

Repository: jclouds
Updated Branches:
  refs/heads/master 1b824afbf -> 92d509184


Enforce no redundant modifiers via Checkstyle

This check also helps find utility classes which should not allow
instantiation.


Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/92d50918
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/92d50918
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/92d50918

Branch: refs/heads/master
Commit: 92d5091845efad00c3ed6a843c140d1c89f93c1e
Parents: 1b824af
Author: Andrew Gaul <ga...@apache.org>
Authored: Tue Aug 26 12:23:04 2014 -0700
Committer: Andrew Gaul <ga...@apache.org>
Committed: Tue Aug 26 17:38:21 2014 -0700

----------------------------------------------------------------------
 resources/checkstyle.xml | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/92d50918/resources/checkstyle.xml
----------------------------------------------------------------------
diff --git a/resources/checkstyle.xml b/resources/checkstyle.xml
index 6a67747..0266117 100644
--- a/resources/checkstyle.xml
+++ b/resources/checkstyle.xml
@@ -41,10 +41,7 @@
         </module>
         <module name="NoWhitespaceBefore"/>
         <module name="RedundantImport"/>
-        <module name="RedundantModifier">
-            <!-- VARIABLE_DEF disabled -->
-            <property name="tokens" value="METHOD_DEF, ANNOTATION_FIELD_DEF, INTERFACE_DEF"/>
-        </module>
+        <module name="RedundantModifier"/>
         <module name="SimplifyBooleanExpression"/>
         <module name="TypeName"/>
         <module name="UnnecessaryParentheses"/>