From commits-return-9632-archive-asf-public=cust-asf.ponee.io@groovy.apache.org Sun Nov 3 15:42:17 2019 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id 21DD4180607 for ; Sun, 3 Nov 2019 16:42:17 +0100 (CET) Received: (qmail 88473 invoked by uid 500); 3 Nov 2019 15:42:16 -0000 Mailing-List: contact commits-help@groovy.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@groovy.apache.org Delivered-To: mailing list commits@groovy.apache.org Received: (qmail 88453 invoked by uid 99); 3 Nov 2019 15:42:16 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 03 Nov 2019 15:42:16 +0000 Received: by gitbox.apache.org (ASF Mail Server at gitbox.apache.org, from userid 33) id 553B3805E6; Sun, 3 Nov 2019 15:42:16 +0000 (UTC) Date: Sun, 03 Nov 2019 15:42:16 +0000 To: "commits@groovy.apache.org" Subject: [groovy] branch GROOVY_3_0_X updated (34296c9 -> 10096e6) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Message-ID: <157279573613.25471.9638114992416249601@gitbox.apache.org> From: sunlan@apache.org X-Git-Host: gitbox.apache.org X-Git-Repo: groovy X-Git-Refname: refs/heads/GROOVY_3_0_X X-Git-Reftype: branch X-Git-Oldrev: 34296c97f520ff633edaa0b81c932389b39eec01 X-Git-Newrev: 10096e64a1278e1520fbc9a49db441ade0e21876 X-Git-NotificationType: ref_changed X-Git-Multimail-Version: 1.5.dev Auto-Submitted: auto-generated This is an automated email from the ASF dual-hosted git repository. sunlan pushed a change to branch GROOVY_3_0_X in repository https://gitbox.apache.org/repos/asf/groovy.git. from 34296c9 Refactor code new 4dc126f Trivial refactoring: cleanup `CachedClass` new 80e9aa5 Minor refactoring: Anonymous type can be replaced with lambda expression new fb57610 Minor refactoring: Anonymous type can be replaced with method reference new 0a7e3f4 Minor refactoring: Lambda can be replaced with method reference new 3acf040 Trivial refactoring: Can be replaced with single 'Map.replaceAll' method call new c63e282 Trivial refactoring: Can be replaced with single 'Map.computeIfAbsent' method call new 7b0c8bb Trivial refactoring: remove unused imports new dd8badc Fix javadoc errors new 7a70cbd Fix "java.lang.RuntimeException: No suitable ClassLoader found for grab" caused by refactoring new 1528aea Minor refactoring: 'for' loop can be replaced with 'foreach' new 05431cf Minor refactoring: 'while' loop can be replaced with 'foreach' new 8504c00 Trivial refactoring: Anonymous type has shorter lambda alternative new 15a7e9e Trivial refactoring: `Collections.sort()` can be replaced with `List.sort()` new aa966b3 Trivial refactoring: Comparator combinator can be used new be4e98f Trivial refactoring: replaced with empty array new aa81c0e Trivial refactoring: Can be replaced with `size() != 1` new d339c88 Trivial refactoring: remove unnecessary check and cast new 10096e6 Trivial refactoring: avoid redundant operations on list The 18 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: src/main/java/groovy/inspect/Inspector.java | 3 +- .../java/groovy/lang/BenchmarkInterceptor.java | 6 +- src/main/java/groovy/lang/ExpandoMetaClass.java | 140 ++++++++++----------- src/main/java/groovy/lang/GroovyClassLoader.java | 71 +++++------ src/main/java/groovy/lang/GroovyCodeSource.java | 28 ++--- src/main/java/groovy/lang/GroovyShell.java | 36 +----- src/main/java/groovy/lang/MetaClassImpl.java | 34 ++--- src/main/java/groovy/lang/ProxyMetaClass.java | 24 +--- .../java/groovy/util/FactoryBuilderSupport.java | 15 +-- src/main/java/groovy/util/GroovyScriptEngine.java | 40 +++--- src/main/java/groovy/util/NodeList.java | 2 +- src/main/java/groovy/util/ObjectGraphBuilder.java | 70 ++++------- src/main/java/groovy/util/ProxyGenerator.java | 14 +-- .../internal/util/ReevaluatingReference.java | 13 +- .../util/concurrentlinkedhashmap/Weighers.java | 8 +- src/main/java/org/codehaus/groovy/antlr/Main.java | 31 ++--- .../groovy/antlr/java/Java2GroovyProcessor.java | 5 +- .../groovy/antlr/treewalker/CompositeVisitor.java | 20 +-- .../antlr/treewalker/FlatNodeListTraversal.java | 8 +- .../java/org/codehaus/groovy/ast/CompileUnit.java | 2 +- .../groovy/ast/decompiled/AsmDecompiler.java | 5 +- .../groovy/ast/expr/ArgumentListExpression.java | 3 +- .../codehaus/groovy/ast/tools/GenericsUtils.java | 4 +- .../groovy/ast/tools/WideningCategories.java | 32 +++-- .../groovy/classgen/AsmClassGenerator.java | 18 ++- .../codehaus/groovy/classgen/BytecodeSequence.java | 7 +- .../groovy/classgen/DummyClassGenerator.java | 9 +- .../org/codehaus/groovy/classgen/ReturnAdder.java | 11 +- .../groovy/classgen/VariableScopeVisitor.java | 7 +- .../codehaus/groovy/classgen/asm/CompileStack.java | 9 +- .../groovy/classgen/asm/InvocationWriter.java | 32 +++-- .../codehaus/groovy/classgen/asm/MethodCaller.java | 3 +- .../groovy/classgen/asm/StatementWriter.java | 20 ++- .../asm/sc/StaticCompilationMopWriter.java | 7 +- .../groovy/control/CompilerConfiguration.java | 2 +- .../codehaus/groovy/control/ErrorCollector.java | 12 +- .../java/org/codehaus/groovy/control/Janitor.java | 6 +- .../org/codehaus/groovy/control/LabelVerifier.java | 1 - .../codehaus/groovy/control/ProcessingUnit.java | 10 +- .../org/codehaus/groovy/control/SourceUnit.java | 6 +- .../codehaus/groovy/reflection/CachedClass.java | 60 ++++----- .../groovy/reflection/CachedConstructor.java | 3 +- .../codehaus/groovy/reflection/SunClassLoader.java | 12 +- .../reflection/stdclasses/CachedSAMClass.java | 3 +- .../groovy/runtime/DefaultGroovyMethods.java | 19 ++- .../groovy/runtime/EncodingGroovyMethods.java | 4 +- .../groovy/runtime/ProcessGroovyMethods.java | 52 ++++---- .../groovy/runtime/ProxyGeneratorAdapter.java | 6 +- .../groovy/runtime/ScriptBytecodeAdapter.java | 4 +- .../groovy/runtime/SocketGroovyMethods.java | 6 +- .../runtime/callsite/GroovySunClassLoader.java | 16 ++- .../runtime/metaclass/ConcurrentReaderHashMap.java | 108 ++++++++-------- .../runtime/metaclass/MetaClassRegistryImpl.java | 62 ++++----- .../groovy/runtime/metaclass/MetaMethodIndex.java | 4 +- .../runtime/powerassert/AssertionRenderer.java | 6 +- .../java/org/codehaus/groovy/syntax/CSTNode.java | 14 +-- .../codehaus/groovy/tools/FileSystemCompiler.java | 4 +- .../org/codehaus/groovy/tools/GroovyStarter.java | 6 +- .../groovy/tools/javac/JavaStubGenerator.java | 6 +- .../groovy/tools/shell/util/MessageSource.java | 11 +- .../transform/AnnotationCollectorTransform.java | 3 +- .../transform/MapConstructorASTTransformation.java | 1 - .../transform/SortableASTTransformation.java | 9 +- .../transform/ToStringASTTransformation.java | 9 +- .../TupleConstructorASTTransformation.java | 9 +- .../stc/AbstractExtensionMethodCache.java | 4 +- .../stc/GroovyTypeCheckingExtensionSupport.java | 6 +- .../transform/stc/StaticTypeCheckingSupport.java | 5 +- .../transform/stc/StaticTypeCheckingVisitor.java | 12 +- .../groovy/util/AbstractConcurrentMap.java | 12 +- .../groovy/util/AbstractConcurrentMapBase.java | 100 +++++++-------- .../groovy/util/ManagedConcurrentLinkedQueue.java | 4 +- .../org/codehaus/groovy/util/ReferenceManager.java | 38 +++--- .../codehaus/groovy/vmplugin/v5/JUnit4Utils.java | 6 +- .../v7/IndyGuardsFiltersAndSignatures.java | 5 +- .../codehaus/groovy/vmplugin/v7/IndyInterface.java | 8 +- .../org/codehaus/groovy/vmplugin/v7/Java7.java | 18 +-- .../org/codehaus/groovy/vmplugin/v9/Java9.java | 6 +- 78 files changed, 568 insertions(+), 867 deletions(-)