You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by jw...@apache.org on 2016/08/23 02:15:56 UTC
[1/5] groovy git commit: findbugs: un-callable method in anonymous
class (closes #389)
Repository: groovy
Updated Branches:
refs/heads/master f175e34b5 -> b914a7447
findbugs: un-callable method in anonymous class (closes #389)
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/454158d7
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/454158d7
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/454158d7
Branch: refs/heads/master
Commit: 454158d7c0008900985dc3f2644e6d346a4491b4
Parents: 4f328a1
Author: John Wagenleitner <jw...@apache.org>
Authored: Sun Aug 21 16:14:55 2016 -0700
Committer: John Wagenleitner <jw...@apache.org>
Committed: Mon Aug 22 19:11:09 2016 -0700
----------------------------------------------------------------------
src/main/groovy/lang/MetaClassImpl.java | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/454158d7/src/main/groovy/lang/MetaClassImpl.java
----------------------------------------------------------------------
diff --git a/src/main/groovy/lang/MetaClassImpl.java b/src/main/groovy/lang/MetaClassImpl.java
index e5a0f92..b32763b 100644
--- a/src/main/groovy/lang/MetaClassImpl.java
+++ b/src/main/groovy/lang/MetaClassImpl.java
@@ -494,10 +494,13 @@ public class MetaClassImpl implements MetaClass, MutableMetaClass {
class MOPIter extends MethodIndexAction {
boolean useThis;
- public boolean skipClass(CachedClass clazz) {
- return !useThis && clazz == theCachedClass;
+
+ @Override
+ public boolean skipClass(Class clazz) {
+ return !useThis && clazz == theClass;
}
+ @Override
public void methodNameAction(Class clazz, MetaMethodIndex.Entry e) {
if (useThis) {
if (e.methods == null)
[5/5] groovy git commit: Removed uncessery Integer Object creation
Posted by jw...@apache.org.
Removed uncessery Integer Object creation
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/b914a744
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/b914a744
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/b914a744
Branch: refs/heads/master
Commit: b914a744705b7fc1145a7967fb8660fbb5af97a9
Parents: c2da84b
Author: Anand upadhyay <an...@nielsen.com>
Authored: Fri Aug 19 21:38:35 2016 +0530
Committer: John Wagenleitner <jw...@apache.org>
Committed: Mon Aug 22 19:11:51 2016 -0700
----------------------------------------------------------------------
src/main/groovy/lang/IntRange.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/b914a744/src/main/groovy/lang/IntRange.java
----------------------------------------------------------------------
diff --git a/src/main/groovy/lang/IntRange.java b/src/main/groovy/lang/IntRange.java
index 8493d1f..46e9b4a 100644
--- a/src/main/groovy/lang/IntRange.java
+++ b/src/main/groovy/lang/IntRange.java
@@ -422,7 +422,7 @@ public class IntRange extends AbstractList<Integer> implements Range<Integer> {
final int from = this.getFrom();
final int to = this.getTo();
- hashCode = new Integer(((from+to+1)*(from+to))/2).intValue()+to;
+ hashCode = ((from+to+1)*(from+to))/2+to;
return hashCode;
}
}
[3/5] groovy git commit: findbugs: impossible instanceof
Posted by jw...@apache.org.
findbugs: impossible instanceof
loop.getBooleanExpression() can't return a ConstantExpression and the if branch never executed.
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/bf398af7
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/bf398af7
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/bf398af7
Branch: refs/heads/master
Commit: bf398af7a0a7cf8dfd8d3a6fedac61ff00ff766f
Parents: f175e34
Author: John Wagenleitner <jw...@apache.org>
Authored: Sun Aug 21 14:46:58 2016 -0700
Committer: John Wagenleitner <jw...@apache.org>
Committed: Mon Aug 22 19:11:09 2016 -0700
----------------------------------------------------------------------
.../org/codehaus/groovy/classgen/asm/StatementWriter.java | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/bf398af7/src/main/org/codehaus/groovy/classgen/asm/StatementWriter.java
----------------------------------------------------------------------
diff --git a/src/main/org/codehaus/groovy/classgen/asm/StatementWriter.java b/src/main/org/codehaus/groovy/classgen/asm/StatementWriter.java
index a91ac4c..5c14e73 100644
--- a/src/main/org/codehaus/groovy/classgen/asm/StatementWriter.java
+++ b/src/main/org/codehaus/groovy/classgen/asm/StatementWriter.java
@@ -25,6 +25,7 @@ import org.codehaus.groovy.ast.ClassHelper;
import org.codehaus.groovy.ast.ClassNode;
import org.codehaus.groovy.ast.Parameter;
import org.codehaus.groovy.ast.expr.ArgumentListExpression;
+import org.codehaus.groovy.ast.expr.BooleanExpression;
import org.codehaus.groovy.ast.expr.ClosureListExpression;
import org.codehaus.groovy.ast.expr.ConstantExpression;
import org.codehaus.groovy.ast.expr.EmptyExpression;
@@ -228,10 +229,10 @@ public class StatementWriter {
Label breakLabel = controller.getCompileStack().getBreakLabel();
mv.visitLabel(continueLabel);
- Expression bool = loop.getBooleanExpression();
+ BooleanExpression bool = loop.getBooleanExpression();
boolean boolHandled = false;
- if (bool instanceof ConstantExpression) {
- ConstantExpression constant = (ConstantExpression) bool;
+ if (bool.getExpression() instanceof ConstantExpression) {
+ ConstantExpression constant = (ConstantExpression) bool.getExpression();
if (constant.getValue()==Boolean.TRUE) {
boolHandled = true;
// do nothing
[2/5] groovy git commit: findbugs: impossible cast
Posted by jw...@apache.org.
findbugs: impossible cast
removed cast that would always fail and added some basic tests.
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/4f328a19
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/4f328a19
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/4f328a19
Branch: refs/heads/master
Commit: 4f328a19ae8e827cbe3f11a0b97911ba6875716a
Parents: bf398af
Author: John Wagenleitner <jw...@apache.org>
Authored: Sun Aug 21 15:25:41 2016 -0700
Committer: John Wagenleitner <jw...@apache.org>
Committed: Mon Aug 22 19:11:09 2016 -0700
----------------------------------------------------------------------
.../runtime/typehandling/ShortTypeHandling.java | 1 -
.../typehandling/ShortTypeHandlingTest.groovy | 64 ++++++++++++++++++++
2 files changed, 64 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/4f328a19/src/main/org/codehaus/groovy/runtime/typehandling/ShortTypeHandling.java
----------------------------------------------------------------------
diff --git a/src/main/org/codehaus/groovy/runtime/typehandling/ShortTypeHandling.java b/src/main/org/codehaus/groovy/runtime/typehandling/ShortTypeHandling.java
index 44c0276..f45ca0c 100644
--- a/src/main/org/codehaus/groovy/runtime/typehandling/ShortTypeHandling.java
+++ b/src/main/org/codehaus/groovy/runtime/typehandling/ShortTypeHandling.java
@@ -41,7 +41,6 @@ public class ShortTypeHandling {
public static String castToString(Object object) {
if (object==null) return null;
- if (object instanceof Class) return (String) object;
return object.toString();
}
http://git-wip-us.apache.org/repos/asf/groovy/blob/4f328a19/src/test/org/codehaus/groovy/runtime/typehandling/ShortTypeHandlingTest.groovy
----------------------------------------------------------------------
diff --git a/src/test/org/codehaus/groovy/runtime/typehandling/ShortTypeHandlingTest.groovy b/src/test/org/codehaus/groovy/runtime/typehandling/ShortTypeHandlingTest.groovy
new file mode 100644
index 0000000..6eba4fa
--- /dev/null
+++ b/src/test/org/codehaus/groovy/runtime/typehandling/ShortTypeHandlingTest.groovy
@@ -0,0 +1,64 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.codehaus.groovy.runtime.typehandling
+
+import static org.codehaus.groovy.runtime.typehandling.ShortTypeHandling.castToClass
+import static org.codehaus.groovy.runtime.typehandling.ShortTypeHandling.castToString
+import static org.codehaus.groovy.runtime.typehandling.ShortTypeHandling.castToChar
+import static org.codehaus.groovy.runtime.typehandling.ShortTypeHandling.castToEnum
+
+class ShortTypeHandlingTest extends GroovyTestCase {
+
+ void testCastToClass() {
+ assert castToClass(null) == null
+ assert castToClass(Integer.class) == Integer.class
+ assert castToClass('java.lang.String') == String.class
+ shouldFail(GroovyCastException) {
+ castToClass(Collections.emptyList())
+ }
+ }
+
+ void testCastToString() {
+ assert castToString(null) == null
+ assert castToString(String.class) == 'class java.lang.String'
+ assert castToString(List.class) == 'interface java.util.List'
+ }
+
+ void testCastToCharacter() {
+ assert castToChar(null) == null
+ char c = (char)'c'
+ assert castToChar((Object)c) == c
+ assert castToChar(Integer.valueOf(99)) == c
+ assert castToChar("${c}") == c
+ assert castToChar('c') == c
+ shouldFail(GroovyCastException) {
+ castToChar(new Date())
+ }
+ }
+
+ void testCastToEnum() {
+ assert castToEnum(null, TestStages) == null
+ assert castToEnum((Object)TestStages.AFTER_CLASS, TestStages) == TestStages.AFTER_CLASS
+ assert castToEnum("BEFORE_TEST", TestStages) == TestStages.BEFORE_TEST
+ }
+
+ enum TestStages {
+ BEFORE_CLASS, BEFORE_TEST, TEST, AFTER_TEST, AFTER_CLASS
+ }
+}
[4/5] groovy git commit: GROOVY-7834: Calculating hashcode of
IntRange by pairing function (closes #388)
Posted by jw...@apache.org.
GROOVY-7834: Calculating hashcode of IntRange by pairing function (closes #388)
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/c2da84b2
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/c2da84b2
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/c2da84b2
Branch: refs/heads/master
Commit: c2da84b2ac05b478fee41639848489a94d8db179
Parents: 454158d
Author: Anand upadhyay <an...@nielsen.com>
Authored: Fri Aug 19 17:13:08 2016 +0530
Committer: John Wagenleitner <jw...@apache.org>
Committed: Mon Aug 22 19:11:19 2016 -0700
----------------------------------------------------------------------
src/main/groovy/lang/IntRange.java | 10 ++++++++++
src/test/groovy/lang/IntRangeTest.groovy | 11 +++++++++++
2 files changed, 21 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/c2da84b2/src/main/groovy/lang/IntRange.java
----------------------------------------------------------------------
diff --git a/src/main/groovy/lang/IntRange.java b/src/main/groovy/lang/IntRange.java
index 3855f43..8493d1f 100644
--- a/src/main/groovy/lang/IntRange.java
+++ b/src/main/groovy/lang/IntRange.java
@@ -415,4 +415,14 @@ public class IntRange extends AbstractList<Integer> implements Range<Integer> {
step(step, adapter);
return adapter.asList();
}
+
+ @Override
+ public int hashCode(){
+ int hashCode;
+ final int from = this.getFrom();
+ final int to = this.getTo();
+
+ hashCode = new Integer(((from+to+1)*(from+to))/2).intValue()+to;
+ return hashCode;
+ }
}
http://git-wip-us.apache.org/repos/asf/groovy/blob/c2da84b2/src/test/groovy/lang/IntRangeTest.groovy
----------------------------------------------------------------------
diff --git a/src/test/groovy/lang/IntRangeTest.groovy b/src/test/groovy/lang/IntRangeTest.groovy
index 73c5a5e..c1afa76 100644
--- a/src/test/groovy/lang/IntRangeTest.groovy
+++ b/src/test/groovy/lang/IntRangeTest.groovy
@@ -138,4 +138,15 @@ class IntRangeTest extends GroovyTestCase {
assert bs[-1..<-7].toString() == '{0, 5}'
assert bs[20..<-8].toString() == '{2, 3}'
}
+
+ void testHashCode(){
+ def maxRange = new IntRange(1,Integer.MAX_VALUE)
+ def rangeWithName = [:]
+ rangeWithName.put(maxRange, "maxRange")
+ def dupRange = new IntRange(1,Integer.MAX_VALUE)
+ assertEquals(rangeWithName.get(dupRange), "maxRange")
+ }
+
+
+
}