You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by pa...@apache.org on 2016/01/05 01:37:14 UTC

groovy git commit: Fixing squid:EmptyStatementUsageCheck - Empty statements should be removed

Repository: groovy
Updated Branches:
  refs/heads/GROOVY_2_4_X 4a98ac1ec -> f5cc12d6e


Fixing squid:EmptyStatementUsageCheck - Empty statements should be removed


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

Branch: refs/heads/GROOVY_2_4_X
Commit: f5cc12d6e8859dba1f59919c8fcc3fc8a9789b80
Parents: 4a98ac1
Author: Kirill Vlasov <ki...@devfactory.com>
Authored: Wed Dec 30 11:01:19 2015 +0500
Committer: paulk <pa...@asert.com.au>
Committed: Tue Jan 5 10:36:44 2016 +1000

----------------------------------------------------------------------
 .../org/codehaus/groovy/classgen/asm/indy/InvokeDynamicWriter.java | 2 +-
 src/main/org/codehaus/groovy/control/ClassNodeResolver.java        | 2 +-
 .../codehaus/groovy/transform/AnnotationCollectorTransform.java    | 2 +-
 .../groovy-json/src/main/java/groovy/json/JsonParserType.java      | 2 +-
 .../groovy-json/src/main/java/groovy/json/internal/Type.java       | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/f5cc12d6/src/main/org/codehaus/groovy/classgen/asm/indy/InvokeDynamicWriter.java
----------------------------------------------------------------------
diff --git a/src/main/org/codehaus/groovy/classgen/asm/indy/InvokeDynamicWriter.java b/src/main/org/codehaus/groovy/classgen/asm/indy/InvokeDynamicWriter.java
index 362544e..a5be513 100644
--- a/src/main/org/codehaus/groovy/classgen/asm/indy/InvokeDynamicWriter.java
+++ b/src/main/org/codehaus/groovy/classgen/asm/indy/InvokeDynamicWriter.java
@@ -228,7 +228,7 @@ public class InvokeDynamicWriter extends InvocationWriter {
 
     @Override
     public void castNonPrimitiveToBool(ClassNode sourceType) {
-        writeIndyCast(sourceType, ClassHelper.boolean_TYPE);;
+        writeIndyCast(sourceType, ClassHelper.boolean_TYPE);
     }
 
 }

http://git-wip-us.apache.org/repos/asf/groovy/blob/f5cc12d6/src/main/org/codehaus/groovy/control/ClassNodeResolver.java
----------------------------------------------------------------------
diff --git a/src/main/org/codehaus/groovy/control/ClassNodeResolver.java b/src/main/org/codehaus/groovy/control/ClassNodeResolver.java
index 45fa4c2..80f6811 100644
--- a/src/main/org/codehaus/groovy/control/ClassNodeResolver.java
+++ b/src/main/org/codehaus/groovy/control/ClassNodeResolver.java
@@ -102,7 +102,7 @@ public class ClassNodeResolver {
     protected static final ClassNode NO_CLASS = new ClassNode("NO_CLASS", Opcodes.ACC_PUBLIC,ClassHelper.OBJECT_TYPE){
         public void setRedirect(ClassNode cn) {
             throw new GroovyBugError("This is a dummy class node only! Never use it for real classes.");
-        };
+        }
     };
     
     /**

http://git-wip-us.apache.org/repos/asf/groovy/blob/f5cc12d6/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 20d93f0..6151773 100644
--- a/src/main/org/codehaus/groovy/transform/AnnotationCollectorTransform.java
+++ b/src/main/org/codehaus/groovy/transform/AnnotationCollectorTransform.java
@@ -78,7 +78,7 @@ public class AnnotationCollectorTransform {
                 if (an.getClassNode().getName().equals(AnnotationCollector.class.getName())) {
                     collector = an;
                     break;
-                };
+                }
             }
             if (collector==null) return;
             

http://git-wip-us.apache.org/repos/asf/groovy/blob/f5cc12d6/subprojects/groovy-json/src/main/java/groovy/json/JsonParserType.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-json/src/main/java/groovy/json/JsonParserType.java b/subprojects/groovy-json/src/main/java/groovy/json/JsonParserType.java
index 22104a4..65e9a41 100644
--- a/subprojects/groovy-json/src/main/java/groovy/json/JsonParserType.java
+++ b/subprojects/groovy-json/src/main/java/groovy/json/JsonParserType.java
@@ -94,5 +94,5 @@ public enum JsonParserType {
      * INDEX_OVERLAY. It should be on average the fastest known JSON parser on the JVM circa Jan 2014.
      * But not as fast as INDEX_OVERLAY.
      */
-    CHAR_BUFFER;
+    CHAR_BUFFER
 }

http://git-wip-us.apache.org/repos/asf/groovy/blob/f5cc12d6/subprojects/groovy-json/src/main/java/groovy/json/internal/Type.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-json/src/main/java/groovy/json/internal/Type.java b/subprojects/groovy-json/src/main/java/groovy/json/internal/Type.java
index 80e6b53..b37d4eb 100644
--- a/subprojects/groovy-json/src/main/java/groovy/json/internal/Type.java
+++ b/subprojects/groovy-json/src/main/java/groovy/json/internal/Type.java
@@ -23,6 +23,6 @@ package groovy.json.internal;
  */
 public enum Type {
 
-    INTEGER, STRING, DOUBLE, TRUE, FALSE, NULL, MAP, LIST;
+    INTEGER, STRING, DOUBLE, TRUE, FALSE, NULL, MAP, LIST
 
 }