Repository: incubator-groovy Updated Branches: refs/heads/GROOVY_2_4_X aef34862d -> e9dfe8c71
remove some checkstyle warnings (merged with changes) Project: http://git-wip-us.apache.org/repos/asf/incubator-groovy/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-groovy/commit/168d75fa Tree: http://git-wip-us.apache.org/repos/asf/incubator-groovy/tree/168d75fa Diff: http://git-wip-us.apache.org/repos/asf/incubator-groovy/diff/168d75fa Branch: refs/heads/GROOVY_2_4_X Commit: 168d75fafcce8b63c59459865e3925bca30503d1 Parents: aef3486 Author: Paul King <pa...@asert.com.au> Authored: Sun May 10 11:43:43 2015 +1000 Committer: Paul King <pa...@asert.com.au> Committed: Mon May 11 08:46:10 2015 +1000 ---------------------------------------------------------------------- gradle/quality.gradle | 8 +++++-- src/main/groovy/lang/ExpandoMetaClass.java | 1 - src/main/org/codehaus/groovy/ast/ASTNode.java | 1 - .../groovy/classgen/asm/InvocationWriter.java | 1 - .../classgen/asm/sc/StaticInvocationWriter.java | 1 - .../asm/sc/StaticTypesStatementWriter.java | 8 +++++-- .../groovy/runtime/ProxyGeneratorAdapter.java | 1 - .../transform/AnnotationCollectorTransform.java | 3 +-- .../StaticCompilationTransformer.java | 24 ++++++++++++++++---- .../stc/StaticTypeCheckingVisitor.java | 1 - .../trait/SuperCallTraitTransformer.java | 1 - .../transform/trait/TraitASTTransformation.java | 1 - .../codehaus/groovy/transform/trait/Traits.java | 1 - .../org/codehaus/groovy/vmplugin/v7/Java7.java | 3 +-- 14 files changed, 34 insertions(+), 21 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/gradle/quality.gradle ---------------------------------------------------------------------- diff --git a/gradle/quality.gradle b/gradle/quality.gradle index 7e7c215..245bf51 100644 --- a/gradle/quality.gradle +++ b/gradle/quality.gradle @@ -61,8 +61,11 @@ allprojects { configFile = file("$rootProject.projectDir/config/checkstyle/checkstyle.xml") configProperties = ['rootProject.projectDir': rootProject.projectDir] def reportFile = file("${buildDir}/reports/checkstyle/${name}.xml") + source = sourceSets.main.allJava.matching { + // TODO why doesn't this exclusion work? + exclude '**/generated-sources/**/*' + } reports { - // TODO exclude generated-sources include ( '**/*.java') xml { destination reportFile @@ -87,7 +90,8 @@ allprojects { def xml = new XmlSlurper().parse(reportFile.newReader('utf-8')) def files = [] xml.file.each { f -> - if (f.error.size()) { + // TODO remove generated-sources check once exclude above works + if (f.error.size() && !f.@name.toString().contains('generated-sources')) { files << [ name: f.@name.toString(), errors: f.error.collect { e -> http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/groovy/lang/ExpandoMetaClass.java ---------------------------------------------------------------------- diff --git a/src/main/groovy/lang/ExpandoMetaClass.java b/src/main/groovy/lang/ExpandoMetaClass.java index d0670a8..fb74414 100644 --- a/src/main/groovy/lang/ExpandoMetaClass.java +++ b/src/main/groovy/lang/ExpandoMetaClass.java @@ -55,7 +55,6 @@ import org.codehaus.groovy.runtime.metaclass.MixinInstanceMetaMethod; import org.codehaus.groovy.runtime.metaclass.OwnedMetaClass; import org.codehaus.groovy.runtime.metaclass.ThreadManagedMetaBeanProperty; import org.codehaus.groovy.util.FastArray; -import org.objectweb.asm.Opcodes; /** * ExpandoMetaClass is a MetaClass that behaves like an Expando, allowing the addition or replacement http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/ast/ASTNode.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/ast/ASTNode.java b/src/main/org/codehaus/groovy/ast/ASTNode.java index 887c8e8..3dc9691 100644 --- a/src/main/org/codehaus/groovy/ast/ASTNode.java +++ b/src/main/org/codehaus/groovy/ast/ASTNode.java @@ -22,7 +22,6 @@ import org.codehaus.groovy.GroovyBugError; import org.codehaus.groovy.util.ListHashMap; import java.util.Collections; -import java.util.LinkedHashMap; import java.util.Map; /** http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/classgen/asm/InvocationWriter.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/classgen/asm/InvocationWriter.java b/src/main/org/codehaus/groovy/classgen/asm/InvocationWriter.java index 9be85f1..321a5e7 100644 --- a/src/main/org/codehaus/groovy/classgen/asm/InvocationWriter.java +++ b/src/main/org/codehaus/groovy/classgen/asm/InvocationWriter.java @@ -36,7 +36,6 @@ import org.codehaus.groovy.runtime.ScriptBytecodeAdapter; import org.codehaus.groovy.runtime.typehandling.DefaultTypeTransformation; import org.codehaus.groovy.runtime.typehandling.ShortTypeHandling; import org.codehaus.groovy.syntax.SyntaxException; -import org.codehaus.groovy.transform.stc.StaticTypeCheckingSupport; import org.objectweb.asm.Label; import org.objectweb.asm.MethodVisitor; http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/classgen/asm/sc/StaticInvocationWriter.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/classgen/asm/sc/StaticInvocationWriter.java b/src/main/org/codehaus/groovy/classgen/asm/sc/StaticInvocationWriter.java index 21cc48b..fe50d75 100644 --- a/src/main/org/codehaus/groovy/classgen/asm/sc/StaticInvocationWriter.java +++ b/src/main/org/codehaus/groovy/classgen/asm/sc/StaticInvocationWriter.java @@ -28,7 +28,6 @@ import org.codehaus.groovy.ast.Parameter; import org.codehaus.groovy.ast.expr.*; import org.codehaus.groovy.ast.stmt.ExpressionStatement; import org.codehaus.groovy.ast.stmt.ForStatement; -import org.codehaus.groovy.ast.tools.WideningCategories; import org.codehaus.groovy.classgen.AsmClassGenerator; import org.codehaus.groovy.classgen.Verifier; import org.codehaus.groovy.classgen.asm.*; http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/classgen/asm/sc/StaticTypesStatementWriter.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/classgen/asm/sc/StaticTypesStatementWriter.java b/src/main/org/codehaus/groovy/classgen/asm/sc/StaticTypesStatementWriter.java index bb7d8b6..470793d 100644 --- a/src/main/org/codehaus/groovy/classgen/asm/sc/StaticTypesStatementWriter.java +++ b/src/main/org/codehaus/groovy/classgen/asm/sc/StaticTypesStatementWriter.java @@ -27,13 +27,17 @@ import org.codehaus.groovy.ast.expr.MethodCallExpression; import org.codehaus.groovy.ast.stmt.BlockStatement; import org.codehaus.groovy.ast.stmt.ForStatement; import org.codehaus.groovy.classgen.AsmClassGenerator; -import org.codehaus.groovy.classgen.asm.*; +import org.codehaus.groovy.classgen.asm.BytecodeVariable; +import org.codehaus.groovy.classgen.asm.CompileStack; +import org.codehaus.groovy.classgen.asm.MethodCaller; +import org.codehaus.groovy.classgen.asm.OperandStack; +import org.codehaus.groovy.classgen.asm.StatementWriter; +import org.codehaus.groovy.classgen.asm.TypeChooser; import org.codehaus.groovy.transform.stc.StaticTypeCheckingSupport; import org.objectweb.asm.Label; import org.objectweb.asm.MethodVisitor; import java.util.Enumeration; -import java.util.Iterator; import static org.objectweb.asm.Opcodes.*; http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/runtime/ProxyGeneratorAdapter.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/runtime/ProxyGeneratorAdapter.java b/src/main/org/codehaus/groovy/runtime/ProxyGeneratorAdapter.java index 463c44e..df53393 100644 --- a/src/main/org/codehaus/groovy/runtime/ProxyGeneratorAdapter.java +++ b/src/main/org/codehaus/groovy/runtime/ProxyGeneratorAdapter.java @@ -24,7 +24,6 @@ import groovy.lang.GroovyClassLoader; import groovy.lang.GroovyObject; import groovy.lang.GroovyRuntimeException; import groovy.transform.Trait; -import org.codehaus.groovy.ast.AnnotationNode; import org.codehaus.groovy.ast.ClassHelper; import org.codehaus.groovy.ast.ClassNode; import org.codehaus.groovy.classgen.asm.BytecodeHelper; http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/transform/AnnotationCollectorTransform.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/transform/AnnotationCollectorTransform.java b/src/main/org/codehaus/groovy/transform/AnnotationCollectorTransform.java index d392206..20067ce 100644 --- a/src/main/org/codehaus/groovy/transform/AnnotationCollectorTransform.java +++ b/src/main/org/codehaus/groovy/transform/AnnotationCollectorTransform.java @@ -28,7 +28,6 @@ import org.codehaus.groovy.ast.*; import org.codehaus.groovy.ast.expr.*; import org.codehaus.groovy.ast.stmt.ReturnStatement; import org.codehaus.groovy.ast.stmt.Statement; -import org.codehaus.groovy.control.CompilationUnit; import org.codehaus.groovy.control.SourceUnit; import org.codehaus.groovy.control.messages.SyntaxErrorMessage; import org.codehaus.groovy.syntax.SyntaxException; @@ -57,7 +56,7 @@ public class AnnotationCollectorTransform { } /** - * Class used by {@link CompilationUnit} to transform the alias class + * Class used by {@link org.codehaus.groovy.control.CompilationUnit} to transform the alias class * into what is needed by the compiler. This means removing invalid * modifiers, interfaces and superclasses, as well as adding a static * value method returning our serialized version of the data for processing http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/transform/sc/transformers/StaticCompilationTransformer.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/transform/sc/transformers/StaticCompilationTransformer.java b/src/main/org/codehaus/groovy/transform/sc/transformers/StaticCompilationTransformer.java index f84697e..67e5bea 100644 --- a/src/main/org/codehaus/groovy/transform/sc/transformers/StaticCompilationTransformer.java +++ b/src/main/org/codehaus/groovy/transform/sc/transformers/StaticCompilationTransformer.java @@ -18,17 +18,33 @@ */ package org.codehaus.groovy.transform.sc.transformers; -import org.codehaus.groovy.ast.*; -import org.codehaus.groovy.ast.expr.*; +import org.codehaus.groovy.ast.ClassCodeExpressionTransformer; +import org.codehaus.groovy.ast.ClassHelper; +import org.codehaus.groovy.ast.ClassNode; +import org.codehaus.groovy.ast.InnerClassNode; +import org.codehaus.groovy.ast.MethodNode; +import org.codehaus.groovy.ast.expr.BinaryExpression; +import org.codehaus.groovy.ast.expr.BooleanExpression; +import org.codehaus.groovy.ast.expr.CastExpression; +import org.codehaus.groovy.ast.expr.ClosureExpression; +import org.codehaus.groovy.ast.expr.ConstructorCallExpression; +import org.codehaus.groovy.ast.expr.Expression; +import org.codehaus.groovy.ast.expr.ListExpression; +import org.codehaus.groovy.ast.expr.MethodCallExpression; +import org.codehaus.groovy.ast.expr.RangeExpression; +import org.codehaus.groovy.ast.expr.StaticMethodCallExpression; +import org.codehaus.groovy.ast.expr.VariableExpression; import org.codehaus.groovy.ast.stmt.Statement; import org.codehaus.groovy.classgen.asm.sc.StaticTypesTypeChooser; import org.codehaus.groovy.control.SourceUnit; import org.codehaus.groovy.runtime.ScriptBytecodeAdapter; import org.codehaus.groovy.syntax.Types; -import org.codehaus.groovy.transform.sc.StaticCompilationVisitor; import org.codehaus.groovy.transform.stc.StaticTypeCheckingVisitor; -import java.util.*; +import java.util.Collections; +import java.util.HashMap; +import java.util.Iterator; +import java.util.Map; /** * Some expressions use symbols as aliases to method calls (<<, +=, ...). In static compilation, http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java b/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java index f856da4..b492aa3 100644 --- a/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java +++ b/src/main/org/codehaus/groovy/transform/stc/StaticTypeCheckingVisitor.java @@ -22,7 +22,6 @@ import groovy.lang.Closure; import groovy.lang.DelegatesTo; import groovy.lang.IntRange; import groovy.lang.Range; -import groovy.transform.SelfType; import groovy.transform.TypeChecked; import groovy.transform.TypeCheckingMode; import groovy.transform.stc.ClosureParams; http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/transform/trait/SuperCallTraitTransformer.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/transform/trait/SuperCallTraitTransformer.java b/src/main/org/codehaus/groovy/transform/trait/SuperCallTraitTransformer.java index bc07cb6..cfa9c0c 100644 --- a/src/main/org/codehaus/groovy/transform/trait/SuperCallTraitTransformer.java +++ b/src/main/org/codehaus/groovy/transform/trait/SuperCallTraitTransformer.java @@ -33,7 +33,6 @@ import org.codehaus.groovy.ast.expr.TupleExpression; import org.codehaus.groovy.ast.expr.VariableExpression; import org.codehaus.groovy.control.SourceUnit; import org.codehaus.groovy.syntax.Types; -import org.codehaus.groovy.transform.stc.StaticTypeCheckingSupport; import java.util.List; http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/transform/trait/TraitASTTransformation.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/transform/trait/TraitASTTransformation.java b/src/main/org/codehaus/groovy/transform/trait/TraitASTTransformation.java index 0da8140..028d943 100644 --- a/src/main/org/codehaus/groovy/transform/trait/TraitASTTransformation.java +++ b/src/main/org/codehaus/groovy/transform/trait/TraitASTTransformation.java @@ -58,7 +58,6 @@ import org.codehaus.groovy.syntax.Types; import org.codehaus.groovy.transform.ASTTransformationCollectorCodeVisitor; import org.codehaus.groovy.transform.AbstractASTTransformation; import org.codehaus.groovy.transform.GroovyASTTransformation; -import org.codehaus.groovy.transform.sc.StaticCompileTransformation; import java.lang.reflect.Modifier; import java.util.ArrayList; http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/transform/trait/Traits.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/transform/trait/Traits.java b/src/main/org/codehaus/groovy/transform/trait/Traits.java index 26f8a4b..a5d3a46 100644 --- a/src/main/org/codehaus/groovy/transform/trait/Traits.java +++ b/src/main/org/codehaus/groovy/transform/trait/Traits.java @@ -34,7 +34,6 @@ import org.codehaus.groovy.ast.expr.ListExpression; import org.codehaus.groovy.ast.tools.GenericsUtils; import org.codehaus.groovy.classgen.asm.BytecodeHelper; import org.codehaus.groovy.runtime.DefaultGroovyMethods; -import org.codehaus.groovy.transform.stc.Receiver; import java.lang.annotation.ElementType; import java.lang.annotation.Retention; http://git-wip-us.apache.org/repos/asf/incubator-groovy/blob/168d75fa/src/main/org/codehaus/groovy/vmplugin/v7/Java7.java ---------------------------------------------------------------------- diff --git a/src/main/org/codehaus/groovy/vmplugin/v7/Java7.java b/src/main/org/codehaus/groovy/vmplugin/v7/Java7.java index 51a5673..3ec9279 100644 --- a/src/main/org/codehaus/groovy/vmplugin/v7/Java7.java +++ b/src/main/org/codehaus/groovy/vmplugin/v7/Java7.java @@ -24,7 +24,6 @@ import org.codehaus.groovy.vmplugin.v6.Java6; import java.lang.invoke.MethodHandle; import java.lang.invoke.MethodHandles; import java.lang.reflect.Constructor; -import java.lang.reflect.InvocationTargetException; import java.lang.reflect.Method; import java.security.AccessController; import java.security.PrivilegedAction; @@ -37,7 +36,7 @@ import java.security.PrivilegedAction; * @author Jochen Theodorou */ public class Java7 extends Java6 { - private final static Constructor<MethodHandles.Lookup> LOOKUP_Constructor; + private static final Constructor<MethodHandles.Lookup> LOOKUP_Constructor; static { Constructor<MethodHandles.Lookup> con = null; try {