You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2018/04/04 00:12:03 UTC
[1/7] groovy git commit: Avoid call to 'toArray()' with pre-sized
array argument
Repository: groovy
Updated Branches:
refs/heads/master 5c9c72413 -> 02e0f2aac
Avoid call to 'toArray()' with pre-sized array argument
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/fe4224a6
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/fe4224a6
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/fe4224a6
Branch: refs/heads/master
Commit: fe4224a66872052ccefffcc724c45ad70a41f6eb
Parents: 5c9c724
Author: sunlan <su...@apache.org>
Authored: Wed Apr 4 07:59:07 2018 +0800
Committer: sunlan <su...@apache.org>
Committed: Wed Apr 4 08:03:50 2018 +0800
----------------------------------------------------------------------
.../codehaus/groovy/ant/FileSystemCompilerFacade.java | 2 +-
.../main/java/org/codehaus/groovy/ant/Groovyc.java | 2 +-
.../tools/groovydoc/ExternalGroovyClassDoc.java | 2 +-
.../groovy/tools/groovydoc/SimpleGroovyClassDoc.java | 14 +++++++-------
.../groovy/tools/groovydoc/SimpleGroovyDoc.java | 2 +-
.../groovydoc/SimpleGroovyExecutableMemberDoc.java | 2 +-
.../tools/groovydoc/SimpleGroovyPackageDoc.java | 14 +++++++-------
.../groovy/tools/groovydoc/SimpleGroovyParameter.java | 2 +-
.../groovydoc/SimpleGroovyProgramElementDoc.java | 2 +-
.../groovy/tools/groovydoc/SimpleGroovyRootDoc.java | 4 ++--
.../groovy-sql/src/main/java/groovy/sql/Sql.java | 2 +-
.../groovy/binding/ClosureTriggerBinding.java | 4 ++--
12 files changed, 26 insertions(+), 26 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/FileSystemCompilerFacade.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/FileSystemCompilerFacade.java b/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/FileSystemCompilerFacade.java
index e9b3327..3235c23 100644
--- a/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/FileSystemCompilerFacade.java
+++ b/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/FileSystemCompilerFacade.java
@@ -37,7 +37,7 @@ public class FileSystemCompilerFacade {
if (forceLookupUnnamedFiles) {
argList.remove("--forceLookupUnnamedFiles");
}
- String[] newArgs = forceLookupUnnamedFiles ? argList.toArray(new String[argList.size()]) : args;
+ String[] newArgs = forceLookupUnnamedFiles ? argList.toArray(new String[0]) : args;
FileSystemCompiler.commandLineCompileWithErrorHandling(newArgs, forceLookupUnnamedFiles);
}
}
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/Groovyc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/Groovyc.java b/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/Groovyc.java
index 8af272b..5543a32 100644
--- a/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/Groovyc.java
+++ b/subprojects/groovy-ant/src/main/java/org/codehaus/groovy/ant/Groovyc.java
@@ -1154,7 +1154,7 @@ public class Groovyc extends MatchingTask {
private String[] makeCommandLine(List<String> commandLineList) {
log.verbose("Compilation arguments:\n" + DefaultGroovyMethods.join((Iterable)commandLineList, "\n"));
- return commandLineList.toArray(new String[commandLineList.size()]);
+ return commandLineList.toArray(new String[0]);
}
private void runForked(String[] commandLine) {
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/ExternalGroovyClassDoc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/ExternalGroovyClassDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/ExternalGroovyClassDoc.java
index d4611f7..c9fecb7 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/ExternalGroovyClassDoc.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/ExternalGroovyClassDoc.java
@@ -46,7 +46,7 @@ public class ExternalGroovyClassDoc implements GroovyClassDoc {
}
public GroovyAnnotationRef[] annotations() {
- return annotationRefs.toArray(new GroovyAnnotationRef[annotationRefs.size()]);
+ return annotationRefs.toArray(new GroovyAnnotationRef[0]);
}
public String qualifiedTypeName() {
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
index e3d4637..ace2b45 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
@@ -116,7 +116,7 @@ public class SimpleGroovyClassDoc extends SimpleGroovyAbstractableElementDoc imp
*/
public GroovyConstructorDoc[] constructors() {
Collections.sort(constructors);
- return constructors.toArray(new GroovyConstructorDoc[constructors.size()]);
+ return constructors.toArray(new GroovyConstructorDoc[0]);
}
public boolean add(GroovyConstructorDoc constructor) {
@@ -145,7 +145,7 @@ public class SimpleGroovyClassDoc extends SimpleGroovyAbstractableElementDoc imp
*/
public GroovyClassDoc[] innerClasses() {
Collections.sort(nested);
- return nested.toArray(new GroovyClassDoc[nested.size()]);
+ return nested.toArray(new GroovyClassDoc[0]);
}
public boolean addNested(GroovyClassDoc nestedClass) {
@@ -157,7 +157,7 @@ public class SimpleGroovyClassDoc extends SimpleGroovyAbstractableElementDoc imp
*/
public GroovyFieldDoc[] fields() {
Collections.sort(fields);
- return fields.toArray(new GroovyFieldDoc[fields.size()]);
+ return fields.toArray(new GroovyFieldDoc[0]);
}
public boolean add(GroovyFieldDoc field) {
@@ -169,7 +169,7 @@ public class SimpleGroovyClassDoc extends SimpleGroovyAbstractableElementDoc imp
*/
public GroovyFieldDoc[] properties() {
Collections.sort(properties);
- return properties.toArray(new GroovyFieldDoc[properties.size()]);
+ return properties.toArray(new GroovyFieldDoc[0]);
}
public boolean addProperty(GroovyFieldDoc property) {
@@ -181,7 +181,7 @@ public class SimpleGroovyClassDoc extends SimpleGroovyAbstractableElementDoc imp
*/
public GroovyFieldDoc[] enumConstants() {
Collections.sort(enumConstants);
- return enumConstants.toArray(new GroovyFieldDoc[enumConstants.size()]);
+ return enumConstants.toArray(new GroovyFieldDoc[0]);
}
public boolean addEnumConstant(GroovyFieldDoc field) {
@@ -193,7 +193,7 @@ public class SimpleGroovyClassDoc extends SimpleGroovyAbstractableElementDoc imp
*/
public GroovyMethodDoc[] methods() {
Collections.sort(methods);
- return methods.toArray(new GroovyMethodDoc[methods.size()]);
+ return methods.toArray(new GroovyMethodDoc[0]);
}
public boolean add(GroovyMethodDoc method) {
@@ -732,7 +732,7 @@ public class SimpleGroovyClassDoc extends SimpleGroovyAbstractableElementDoc imp
public GroovyClassDoc[] interfaces() {
Collections.sort(interfaceClasses);
- return interfaceClasses.toArray(new GroovyClassDoc[interfaceClasses.size()]);
+ return interfaceClasses.toArray(new GroovyClassDoc[0]);
}
public GroovyType[] interfaceTypes() {/*todo*/
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyDoc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyDoc.java
index f87c7cd..8dd1d9c 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyDoc.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyDoc.java
@@ -108,7 +108,7 @@ public class SimpleGroovyDoc implements GroovyDoc, GroovyTokenTypes {
setDeprecated(true);
}
}
- tags = result.toArray(new GroovyTag[result.size()]);
+ tags = result.toArray(new GroovyTag[0]);
}
public static String calculateFirstSentence(String raw) {
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyExecutableMemberDoc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyExecutableMemberDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyExecutableMemberDoc.java
index 53219c1..5316ce4 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyExecutableMemberDoc.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyExecutableMemberDoc.java
@@ -35,7 +35,7 @@ public class SimpleGroovyExecutableMemberDoc extends SimpleGroovyMemberDoc imple
}
public GroovyParameter[] parameters() {
- return (GroovyParameter[]) parameters.toArray(new GroovyParameter[parameters.size()]);
+ return (GroovyParameter[]) parameters.toArray(new GroovyParameter[0]);
}
public void add(GroovyParameter parameter) {
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java
index b703554..36090fb 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyPackageDoc.java
@@ -39,7 +39,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac
}
public GroovyClassDoc[] allClasses() {
- return classDocs.values().toArray(new GroovyClassDoc[classDocs.values().size()]);
+ return classDocs.values().toArray(new GroovyClassDoc[0]);
}
public void setDescription(String description) {
@@ -68,7 +68,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac
public GroovyClassDoc[] allClasses(boolean arg0) {
List<GroovyClassDoc> classDocValues = new ArrayList<GroovyClassDoc>(classDocs.values());
- return classDocValues.toArray(new GroovyClassDoc[classDocValues.size()]);
+ return classDocValues.toArray(new GroovyClassDoc[0]);
}
public GroovyClassDoc[] enums() {
@@ -78,7 +78,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac
result.add(doc);
}
}
- return result.toArray(new GroovyClassDoc[result.size()]);
+ return result.toArray(new GroovyClassDoc[0]);
}
public GroovyClassDoc[] errors() {
@@ -88,7 +88,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac
result.add(doc);
}
}
- return result.toArray(new GroovyClassDoc[result.size()]);
+ return result.toArray(new GroovyClassDoc[0]);
}
public GroovyClassDoc[] exceptions() {
@@ -98,7 +98,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac
result.add(doc);
}
}
- return result.toArray(new GroovyClassDoc[result.size()]);
+ return result.toArray(new GroovyClassDoc[0]);
}
public GroovyClassDoc findClass(String arg0) {/*todo*/
@@ -112,7 +112,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac
result.add(doc);
}
}
- return result.toArray(new GroovyClassDoc[result.size()]);
+ return result.toArray(new GroovyClassDoc[0]);
}
public GroovyClassDoc[] ordinaryClasses() {
@@ -122,7 +122,7 @@ public class SimpleGroovyPackageDoc extends SimpleGroovyDoc implements GroovyPac
result.add(doc);
}
}
- return result.toArray(new GroovyClassDoc[result.size()]);
+ return result.toArray(new GroovyClassDoc[0]);
}
public String description() {
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyParameter.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyParameter.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyParameter.java
index 947d48c..27cf037 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyParameter.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyParameter.java
@@ -62,7 +62,7 @@ public class SimpleGroovyParameter implements GroovyParameter {
}
public GroovyAnnotationRef[] annotations() {
- return annotationRefs.toArray(new GroovyAnnotationRef[annotationRefs.size()]);
+ return annotationRefs.toArray(new GroovyAnnotationRef[0]);
}
public void addAnnotationRef(GroovyAnnotationRef ref) {
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyProgramElementDoc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyProgramElementDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyProgramElementDoc.java
index 7b86ea0..8a2339d 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyProgramElementDoc.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyProgramElementDoc.java
@@ -98,7 +98,7 @@ public class SimpleGroovyProgramElementDoc extends SimpleGroovyDoc implements Gr
}
public GroovyAnnotationRef[] annotations() {
- return annotationRefs.toArray(new GroovyAnnotationRef[annotationRefs.size()]);
+ return annotationRefs.toArray(new GroovyAnnotationRef[0]);
}
public void addAnnotationRef(GroovyAnnotationRef ref) {
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java
index cc0dd1b..eeffd16 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyRootDoc.java
@@ -87,7 +87,7 @@ public class SimpleGroovyRootDoc extends SimpleGroovyDoc implements GroovyRootDo
classDocValues = new ArrayList<GroovyClassDoc>(classDocs.values());
Collections.sort(classDocValues);
}
- return classDocValues.toArray(new GroovyClassDoc[classDocValues.size()]);
+ return classDocValues.toArray(new GroovyClassDoc[0]);
}
public String[][] options() {/*todo*/
@@ -117,7 +117,7 @@ public class SimpleGroovyRootDoc extends SimpleGroovyDoc implements GroovyRootDo
packageDocValues = new ArrayList<GroovyPackageDoc>(packageDocs.values());
Collections.sort(packageDocValues);
}
- return packageDocValues.toArray(new GroovyPackageDoc[packageDocValues.size()]);
+ return packageDocValues.toArray(new GroovyPackageDoc[0]);
}
public Map<String, GroovyClassDoc> getVisibleClasses(List importedClassesAndPackages) {
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java b/subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java
index a554644..da61e9f 100644
--- a/subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java
+++ b/subprojects/groovy-sql/src/main/java/groovy/sql/Sql.java
@@ -4589,7 +4589,7 @@ public class Sql {
@Override
protected PreparedStatement execute(Connection connection, String sql) throws SQLException {
if (returnGeneratedKeys == USE_COLUMN_NAMES && keyColumnNames != null) {
- return connection.prepareStatement(sql, keyColumnNames.toArray(new String[keyColumnNames.size()]));
+ return connection.prepareStatement(sql, keyColumnNames.toArray(new String[0]));
}
if (returnGeneratedKeys != 0) {
return connection.prepareStatement(sql, returnGeneratedKeys);
http://git-wip-us.apache.org/repos/asf/groovy/blob/fe4224a6/subprojects/groovy-swing/src/main/java/org/codehaus/groovy/binding/ClosureTriggerBinding.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-swing/src/main/java/org/codehaus/groovy/binding/ClosureTriggerBinding.java b/subprojects/groovy-swing/src/main/java/org/codehaus/groovy/binding/ClosureTriggerBinding.java
index 8a0fecb..6c41a9e 100644
--- a/subprojects/groovy-swing/src/main/java/org/codehaus/groovy/binding/ClosureTriggerBinding.java
+++ b/subprojects/groovy-swing/src/main/java/org/codehaus/groovy/binding/ClosureTriggerBinding.java
@@ -55,7 +55,7 @@ public class ClosureTriggerBinding implements TriggerBinding, SourceBinding {
for (Map.Entry<String, BindPathSnooper> entry : snooper.fields.entrySet()) {
childPaths.add(createBindPath(entry.getKey(), entry.getValue()));
}
- bp.children = childPaths.toArray(new BindPath[childPaths.size()]);
+ bp.children = childPaths.toArray(new BindPath[0]);
return bp;
}
@@ -123,7 +123,7 @@ public class ClosureTriggerBinding implements TriggerBinding, SourceBinding {
PropertyPathFullBinding fb = new PropertyPathFullBinding();
fb.setSourceBinding(new ClosureSourceBinding(closure));
fb.setTargetBinding(target);
- fb.bindPaths = rootPaths.toArray(new BindPath[rootPaths.size()]);
+ fb.bindPaths = rootPaths.toArray(new BindPath[0]);
return fb;
}
[7/7] groovy git commit: Optimize imports
Posted by su...@apache.org.
Optimize imports
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/02e0f2aa
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/02e0f2aa
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/02e0f2aa
Branch: refs/heads/master
Commit: 02e0f2aac6491b24cdbffee65f3171db6c6fa20b
Parents: 9755409
Author: sunlan <su...@apache.org>
Authored: Wed Apr 4 08:09:36 2018 +0800
Committer: sunlan <su...@apache.org>
Committed: Wed Apr 4 08:09:36 2018 +0800
----------------------------------------------------------------------
.../main/java/org/apache/groovy/json/internal/MapItemValue.java | 2 --
.../src/main/java/groovy/xml/streamingmarkupsupport/Builder.java | 1 -
2 files changed, 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/02e0f2aa/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/MapItemValue.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/MapItemValue.java b/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/MapItemValue.java
index d57568e..325214f 100644
--- a/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/MapItemValue.java
+++ b/subprojects/groovy-json/src/main/java/org/apache/groovy/json/internal/MapItemValue.java
@@ -21,8 +21,6 @@ package org.apache.groovy.json.internal;
import java.util.Map;
import java.util.concurrent.ConcurrentHashMap;
-import static org.apache.groovy.json.internal.Exceptions.die;
-
/**
* This holds a mapping from value key to value value to maximize laziness.
*
http://git-wip-us.apache.org/repos/asf/groovy/blob/02e0f2aa/subprojects/groovy-xml/src/main/java/groovy/xml/streamingmarkupsupport/Builder.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-xml/src/main/java/groovy/xml/streamingmarkupsupport/Builder.java b/subprojects/groovy-xml/src/main/java/groovy/xml/streamingmarkupsupport/Builder.java
index a48ac7a..6b38498 100644
--- a/subprojects/groovy-xml/src/main/java/groovy/xml/streamingmarkupsupport/Builder.java
+++ b/subprojects/groovy-xml/src/main/java/groovy/xml/streamingmarkupsupport/Builder.java
@@ -22,7 +22,6 @@ import groovy.lang.Closure;
import groovy.lang.GroovyObjectSupport;
import java.util.HashMap;
-import java.util.Iterator;
import java.util.List;
import java.util.Map;
[6/7] groovy git commit: Unnecessary 'contains' check
Posted by su...@apache.org.
Unnecessary 'contains' check
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/97554095
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/97554095
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/97554095
Branch: refs/heads/master
Commit: 97554095dc1883afeda0b69ff85f68e8cb6733ff
Parents: 6d0e2f6
Author: sunlan <su...@apache.org>
Authored: Wed Apr 4 08:08:06 2018 +0800
Committer: sunlan <su...@apache.org>
Committed: Wed Apr 4 08:08:06 2018 +0800
----------------------------------------------------------------------
.../groovy-json/src/main/java/groovy/json/JsonGenerator.java | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/97554095/subprojects/groovy-json/src/main/java/groovy/json/JsonGenerator.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-json/src/main/java/groovy/json/JsonGenerator.java b/subprojects/groovy-json/src/main/java/groovy/json/JsonGenerator.java
index f6b27f1..be61d67 100644
--- a/subprojects/groovy-json/src/main/java/groovy/json/JsonGenerator.java
+++ b/subprojects/groovy-json/src/main/java/groovy/json/JsonGenerator.java
@@ -255,9 +255,7 @@ public interface JsonGenerator {
Closure<?> closure)
{
Converter converter = new DefaultJsonGenerator.ClosureConverter(type, closure);
- if (converters.contains(converter)) {
- converters.remove(converter);
- }
+ converters.remove(converter);
return addConverter(converter);
}
[2/7] groovy git commit: Avoid manual array to collection copy
Posted by su...@apache.org.
Avoid manual array to collection copy
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/668ec9f5
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/668ec9f5
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/668ec9f5
Branch: refs/heads/master
Commit: 668ec9f5916d67069a258a72e2b9f1c30d4cfd77
Parents: fe4224a
Author: sunlan <su...@apache.org>
Authored: Wed Apr 4 08:04:39 2018 +0800
Committer: sunlan <su...@apache.org>
Committed: Wed Apr 4 08:04:39 2018 +0800
----------------------------------------------------------------------
.../groovy-json/src/main/java/groovy/json/JsonBuilder.java | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/668ec9f5/subprojects/groovy-json/src/main/java/groovy/json/JsonBuilder.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-json/src/main/java/groovy/json/JsonBuilder.java b/subprojects/groovy-json/src/main/java/groovy/json/JsonBuilder.java
index 939570e..89cee15 100644
--- a/subprojects/groovy-json/src/main/java/groovy/json/JsonBuilder.java
+++ b/subprojects/groovy-json/src/main/java/groovy/json/JsonBuilder.java
@@ -27,6 +27,7 @@ import java.io.Writer;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
+import java.util.Collections;
import java.util.HashMap;
import java.util.LinkedHashMap;
import java.util.List;
@@ -176,9 +177,7 @@ public class JsonBuilder extends GroovyObjectSupport implements Writable {
*/
public Object call(Object... args) {
List<Object> listContent = new ArrayList<Object>();
- for (Object it : args) {
- listContent.add(it);
- }
+ Collections.addAll(listContent, args);
content = listContent;
return content;
[3/7] groovy git commit: 'addAll()' call can be replaced with
parametrized constructor call
Posted by su...@apache.org.
'addAll()' call can be replaced with parametrized constructor call
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/7e7efffa
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/7e7efffa
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/7e7efffa
Branch: refs/heads/master
Commit: 7e7efffa9026711e85317f3b5bae6ed3586b25d9
Parents: 668ec9f
Author: sunlan <su...@apache.org>
Authored: Wed Apr 4 08:05:07 2018 +0800
Committer: sunlan <su...@apache.org>
Committed: Wed Apr 4 08:05:07 2018 +0800
----------------------------------------------------------------------
.../org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/7e7efffa/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
index ace2b45..b5be4d7 100644
--- a/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
+++ b/subprojects/groovy-groovydoc/src/main/java/org/codehaus/groovy/tools/groovydoc/SimpleGroovyClassDoc.java
@@ -268,8 +268,7 @@ public class SimpleGroovyClassDoc extends SimpleGroovyAbstractableElementDoc imp
public Set<GroovyClassDoc> getParentInterfaces() {
Set<GroovyClassDoc> result = new LinkedHashSet<GroovyClassDoc>();
result.add(this);
- Set<GroovyClassDoc> next = new LinkedHashSet<GroovyClassDoc>();
- next.addAll(Arrays.asList(this.interfaces()));
+ Set<GroovyClassDoc> next = new LinkedHashSet<GroovyClassDoc>(Arrays.asList(this.interfaces()));
while (!next.isEmpty()) {
Set<GroovyClassDoc> temp = next;
next = new LinkedHashSet<GroovyClassDoc>();
[5/7] groovy git commit: Avoid redundant array creation
Posted by su...@apache.org.
Avoid redundant array creation
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/6d0e2f65
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/6d0e2f65
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/6d0e2f65
Branch: refs/heads/master
Commit: 6d0e2f657528a1b012533a3f7282fa7bd1446179
Parents: b07369a
Author: sunlan <su...@apache.org>
Authored: Wed Apr 4 08:07:34 2018 +0800
Committer: sunlan <su...@apache.org>
Committed: Wed Apr 4 08:07:34 2018 +0800
----------------------------------------------------------------------
.../groovy-sql/src/main/java/groovy/sql/ExtractIndexAndSql.java | 2 +-
.../groovy-swing/src/main/java/groovy/model/ClosureModel.java | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/6d0e2f65/subprojects/groovy-sql/src/main/java/groovy/sql/ExtractIndexAndSql.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-sql/src/main/java/groovy/sql/ExtractIndexAndSql.java b/subprojects/groovy-sql/src/main/java/groovy/sql/ExtractIndexAndSql.java
index 59d9c7d..c8b67f7 100644
--- a/subprojects/groovy-sql/src/main/java/groovy/sql/ExtractIndexAndSql.java
+++ b/subprojects/groovy-sql/src/main/java/groovy/sql/ExtractIndexAndSql.java
@@ -197,7 +197,7 @@ class ExtractIndexAndSql {
int index = (indexStr == null || indexStr.length() == 0 || ":".equals(indexStr)) ? 0 : Integer.parseInt(indexStr) - 1;
String prop = matcher.group(2);
if (prop == null) prop = matcher.group(4);
- indexPropList.add(new Tuple(new Object[]{index, prop == null || prop.length() == 0 ? "<this>" : prop}));
+ indexPropList.add(new Tuple(index, prop == null || prop.length() == 0 ? "<this>" : prop));
txtIndex = matcher.end();
}
newSql.append(sql.substring(txtIndex)); // append ending SQL after last param.
http://git-wip-us.apache.org/repos/asf/groovy/blob/6d0e2f65/subprojects/groovy-swing/src/main/java/groovy/model/ClosureModel.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-swing/src/main/java/groovy/model/ClosureModel.java b/subprojects/groovy-swing/src/main/java/groovy/model/ClosureModel.java
index d25c6e7..82c1471 100644
--- a/subprojects/groovy-swing/src/main/java/groovy/model/ClosureModel.java
+++ b/subprojects/groovy-swing/src/main/java/groovy/model/ClosureModel.java
@@ -65,7 +65,7 @@ public class ClosureModel implements ValueModel, NestedValueModel {
if (writeClosure != null) {
Object source = sourceModel.getValue();
if (source != null) {
- writeClosure.call(new Object[] { source, value });
+ writeClosure.call(source, value);
}
}
}
[4/7] groovy git commit: 'new String()' is redundant
Posted by su...@apache.org.
'new String()' is redundant
Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/b07369a5
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/b07369a5
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/b07369a5
Branch: refs/heads/master
Commit: b07369a506c1792695fcfaf6d6337ceef7836d4f
Parents: 7e7efff
Author: sunlan <su...@apache.org>
Authored: Wed Apr 4 08:05:38 2018 +0800
Committer: sunlan <su...@apache.org>
Committed: Wed Apr 4 08:05:38 2018 +0800
----------------------------------------------------------------------
.../java/org/apache/groovy/jsondirect/DirectFastStringService.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/groovy/blob/b07369a5/subprojects/groovy-json-direct/src/main/java/org/apache/groovy/jsondirect/DirectFastStringService.java
----------------------------------------------------------------------
diff --git a/subprojects/groovy-json-direct/src/main/java/org/apache/groovy/jsondirect/DirectFastStringService.java b/subprojects/groovy-json-direct/src/main/java/org/apache/groovy/jsondirect/DirectFastStringService.java
index 0d7d4ad..a2ed84a 100644
--- a/subprojects/groovy-json-direct/src/main/java/org/apache/groovy/jsondirect/DirectFastStringService.java
+++ b/subprojects/groovy-json-direct/src/main/java/org/apache/groovy/jsondirect/DirectFastStringService.java
@@ -37,7 +37,7 @@ public class DirectFastStringService implements FastStringService {
@Override
public String noCopyStringFromChars(char[] chars) {
if (WRITE_TO_FINAL_FIELDS) {
- String string = new String();
+ String string = "";
UNSAFE.putObject(string, STRING_VALUE_FIELD_OFFSET, chars);
return string;
} else {