[groovy] branch master updated: test already covered by groovyTestCase cases

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
 new 438d5e6  test already covered by groovyTestCase cases
438d5e6 is described below

commit 438d5e605d02c7e836161ef6b905d70efe034efb
Author: Paul King 
AuthorDate: Tue Jan 1 19:21:07 2019 +1000

test already covered by groovyTestCase cases
---
 src/test/groovy/ImmutableModificationTest.groovy | 43 
 1 file changed, 43 deletions(-)

diff --git a/src/test/groovy/ImmutableModificationTest.groovy 
b/src/test/groovy/ImmutableModificationTest.groovy
deleted file mode 100644
index 3647057..000
--- a/src/test/groovy/ImmutableModificationTest.groovy
+++ /dev/null
@@ -1,43 +0,0 @@
-/*
- *  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 groovy
-
-/**
- * check that the new asImmutable() method works
- * as specified in GROOVY-623
- *
- * @author mailto:jeremy.ray...@bigfoot.com";>Jeremy Rayner
- */
-
-class ImmutableModificationTest extends GroovyTestCase {
-void testCollectionAsImmutable() {
-def challenger = ["Telson", "Sharna", "Darv", "Astra"]
-def hopefullyImmutable = challenger.asImmutable()
-try {
-challenger.add("Angel One")
-challenger << "Angel Two"
-
-// @todo fail("'challenger' is supposed to be an immutable 
collection.")
-
-} catch (UnsupportedOperationException e) {
-// success if this exception is thrown
-assert 4 == challenger.size()
-}
-}
-}
\ No newline at end of file

[groovy] branch master updated: cleanup/remove @author tags as per Apache recommended practices cont'd

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
 new 81ac94d  cleanup/remove @author tags as per Apache recommended 
practices cont'd
81ac94d is described below

commit 81ac94d9592c81c0ce0826e7cdb7b00911c40a8f
Author: Paul King 
AuthorDate: Tue Jan 1 19:42:01 2019 +1000

cleanup/remove @author tags as per Apache recommended practices cont'd
---
 gradle/pomconfigurer.gradle|   6 +-
 src/test/groovy/FileTest.groovy|   3 -
 src/test/groovy/FilterLineTest.groovy  |   2 -
 src/test/groovy/Foo.groovy |   2 -
 src/test/groovy/GroovyClosureMethodsTest.groovy|   2 -
 src/test/groovy/HexTest.groovy |   2 -
 src/test/groovy/IdentityClosureTest.groovy |   3 -
 .../groovy/InvokeNormalMethodsFirstTest.groovy |   2 -
 src/test/groovy/LiteralTypesTest.groovy| 190 +++---
 src/test/groovy/MapConstructionTest.groovy |   2 -
 src/test/groovy/MapPropertyTest.groovy |  19 +-
 .../MethodParameterAccessWithinClosureTest.groovy  |  51 +-
 src/test/groovy/MinMaxTest.groovy  |  38 +-
 src/test/groovy/MultiCatchTest.groovy  |   4 +-
 src/test/groovy/MultiDimArraysTest.groovy  |   6 +-
 src/test/groovy/NullPropertyTest.groovy|  11 +-
 src/test/groovy/OverridePropertyGetterTest.groovy  |  23 +-
 ...ateVariableAccessFromAnotherInstanceTest.groovy |  14 +-
 src/test/groovy/ProcessTest.groovy |  48 +-
 src/test/groovy/Property2Test.groovy   |   3 -
 src/test/groovy/PropertyTest.groovy|  55 +-
 src/test/groovy/ReadLineTest.groovy|  43 +-
 src/test/groovy/RegularExpressionsTest.groovy  | 130 ++--
 src/test/groovy/ReturnTest.groovy  |  26 +-
 src/test/groovy/SocketTest.groovy  |   4 +-
 src/test/groovy/SortTest.groovy|  38 +-
 src/test/groovy/SpreadDotTest.groovy   |  99 +--
 .../groovy/SwitchWithDifferentTypesTest.groovy |  48 +-
 src/test/groovy/TextPropertyTest.groovy|  14 +-
 .../UniqueOnCollectionWithClosureTest.groovy   |  15 +-
 .../UniqueOnCollectionWithComparatorTest.groovy| 153 ++---
 src/test/groovy/VarargsMethodTest.groovy   |  73 +-
 src/test/groovy/annotations/MyIntegerAnno.groovy   |  69 +-
 src/test/groovy/beans/BindableTest.groovy  | 745 ++---
 34 files changed, 947 insertions(+), 996 deletions(-)

diff --git a/gradle/pomconfigurer.gradle b/gradle/pomconfigurer.gradle
index 6a5ce05..61aa7bb 100644
--- a/gradle/pomconfigurer.gradle
+++ b/gradle/pomconfigurer.gradle
@@ -588,9 +588,6 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 name 'Chris K Wensel'
 }
 contributor {
-name 'Joachim Baumann'
-}
-contributor {
 name 'David Sutherland'
 }
 contributor {
@@ -684,6 +681,9 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 name 'Thomas Heller'
 }
 contributor {
+name 'John Stump'
+}
+contributor {
 name 'Ivan Ganza'
 }
 contributor {
diff --git a/src/test/groovy/FileTest.groovy b/src/test/groovy/FileTest.groovy
index 613a05d..9e9fe11 100644
--- a/src/test/groovy/FileTest.groovy
+++ b/src/test/groovy/FileTest.groovy
@@ -23,9 +23,6 @@ import static groovy.io.FileVisitResult.*
 
 /**
  * Unit test for File GDK methods
- *
- * @author Marc Guillemot
- * @author Paul King
  */
 class FileTest extends GroovyTestCase {
 
diff --git a/src/test/groovy/FilterLineTest.groovy 
b/src/test/groovy/FilterLineTest.groovy
index a091ad9..1611e40 100644
--- a/src/test/groovy/FilterLineTest.groovy
+++ b/src/test/groovy/FilterLineTest.groovy
@@ -22,8 +22,6 @@ package groovy
  * check that the new filterLine() method on InputStream is ok
  * (and indirectly test newReader() method on InputStream)
  * as specified in GROOVY-624 and GROOVY-625
- *
- * @author mailto:jeremy.ray...@bigfoot.com";>Jeremy Rayner
  */
 
 class FilterLineTest extends GroovyTestCase {
diff --git a/src/test/groovy/Foo.groovy b/src/test/groovy/Foo.groovy
index ce183e2..889dd53 100644
--- a/src/test/groovy/Foo.groovy
+++ b/src/test/groovy/Foo.groovy
@@ -20,8 +20,6 @@ package groovy
 
 /** 
  * A dummy bean for testing the use of properties in Groovy
- * 
- * @author mailto:ja...@coredevelopers.net";>James Strachan
  */
 class Foo implements Serializable {
 
diff --git a/src/test/groovy/GroovyClosureMethodsTest.groovy 
b/src/test/groovy/GroovyClosureMethodsTest.groovy
index 256d1f6..f43a7b5 100644
--- a/src/test/groovy/GroovyClosureMethodsTest.groovy
+++ b/src/test/groovy

[groovy] branch master updated (81ac94d -> 10391f5)

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git.


from 81ac94d  cleanup/remove @author tags as per Apache recommended 
practices cont'd
 new 5e263d8  whitespace/formatting plus cleanup/remove @author tags as per 
Apache recommended practices cont'd
 new 10391f5  whitespace/formatting plus cleanup/remove @author tags as per 
Apache recommended practices cont'd

The 2 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:
 gradle/pomconfigurer.gradle|   6 +
 src/main/groovy/groovy/transform/ASTTest.java  |   1 -
 .../groovy/transform/AnnotationCollector.java  |   1 -
 src/main/groovy/groovy/transform/AutoClone.java|   1 -
 .../groovy/groovy/transform/AutoCloneStyle.java|   1 -
 src/main/groovy/groovy/transform/BaseScript.java   |   4 -
 .../groovy/transform/CompilationUnitAware.java |   2 -
 .../groovy/groovy/transform/CompileDynamic.groovy  |   3 +-
 .../groovy/groovy/transform/CompileStatic.java |   3 -
 .../groovy/transform/ConditionalInterrupt.groovy   |  74 ---
 .../groovy/groovy/transform/EqualsAndHashCode.java |   1 -
 .../groovy/transform/ExternalizeMethods.java   |   1 -
 .../groovy/groovy/transform/IndexedProperty.java   |   1 -
 .../groovy/transform/InheritConstructors.java  |   1 -
 src/main/groovy/groovy/transform/Memoized.java |   2 -
 src/main/groovy/groovy/transform/PackageScope.java |   1 -
 .../groovy/transform/PackageScopeTarget.java   |   1 -
 src/main/groovy/groovy/transform/SelfType.java |   1 -
 src/main/groovy/groovy/transform/SourceURI.java|   1 -
 src/main/groovy/groovy/transform/Synchronized.java |   1 -
 .../groovy/groovy/transform/TailRecursive.groovy   |   3 +-
 .../groovy/groovy/transform/ThreadInterrupt.groovy |  69 +++
 .../groovy/groovy/transform/TimedInterrupt.groovy  |   8 +-
 src/main/groovy/groovy/transform/ToString.java |   2 -
 src/main/groovy/groovy/transform/TypeChecked.java  |   1 -
 .../groovy/groovy/transform/TypeCheckingMode.java  |   2 -
 src/main/groovy/groovy/transform/WithReadLock.java |   1 -
 .../groovy/groovy/transform/WithWriteLock.java |   1 -
 src/test/groovy/io/LineColumnReaderTest.groovy |   3 -
 .../groovy/lang/BenchmarkInterceptorTest.groovy|  29 +--
 src/test/groovy/lang/ClosureResolvingTest.groovy   |   3 +-
 .../groovy/lang/DelegatingMetaClassTest.groovy |   8 +-
 .../lang/ExpandoMetaClassCreationHandleTest.groovy |  41 ++--
 src/test/groovy/lang/ExpandoMetaClassTest.groovy   | 126 ++--
 src/test/groovy/lang/GetMethodsTest.groovy |  13 +-
 src/test/groovy/lang/GroovyCodeSourceTest.groovy   |   7 +-
 src/test/groovy/lang/GroovySystemTest.groovy   |  10 +-
 src/test/groovy/lang/InterceptorTest.groovy|   4 +-
 src/test/groovy/lang/MapOfClosureTest.groovy   | 229 +++--
 src/test/groovy/lang/MetaClassRegistryTest.groovy  |   9 +-
 src/test/groovy/lang/PropertyMissingTest.groovy|  22 +-
 .../groovy/lang/ReferenceSerializationTest.groovy  |   2 -
 src/test/groovy/lang/RespondsToTest.groovy |  12 +-
 .../lang/ScriptSourcePositionInAstTest.groovy  |   4 +-
 src/test/groovy/mock/interceptor/Caller.groovy |  15 +-
 .../groovy/mock/interceptor/Collaborator.groovy|  11 +-
 .../mock/interceptor/MockSingleCallTest.groovy |  38 ++--
 .../mock/interceptor/StubCallSequenceTest.groovy   |   2 -
 .../mock/interceptor/StubSingleCallTest.groovy |   6 +-
 .../groovy/operator/BitwiseOperatorsTest.groovy|  32 ++-
 src/test/groovy/operator/NegateListsTest.groovy|   6 +-
 src/test/groovy/operator/PowerOperatorsTest.groovy |  21 +-
 .../groovy/operator/SpreadListOperatorTest.groovy  |   8 +-
 .../groovy/operator/SpreadMapOperatorTest.groovy   |  64 +++---
 src/test/groovy/time/TimeCategoryTest.groovy   |  10 +-
 src/test/groovy/transform/LazyTest.groovy  |  18 +-
 .../groovy/transform/ThreadInterruptTest.groovy|   2 -
 57 files changed, 434 insertions(+), 515 deletions(-)

[groovy] 02/02: whitespace/formatting plus cleanup/remove @author tags as per Apache recommended practices cont'd

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit 10391f59811907eb693f88cf99189e5b68b350c4
Author: Paul King 
AuthorDate: Wed Jan 2 08:45:39 2019 +1000

whitespace/formatting plus cleanup/remove @author tags as per Apache 
recommended practices cont'd
---
 gradle/pomconfigurer.gradle|  3 +
 src/main/groovy/groovy/transform/ASTTest.java  |  1 -
 .../groovy/transform/AnnotationCollector.java  |  1 -
 src/main/groovy/groovy/transform/AutoClone.java|  1 -
 .../groovy/groovy/transform/AutoCloneStyle.java|  1 -
 src/main/groovy/groovy/transform/BaseScript.java   |  4 --
 .../groovy/transform/CompilationUnitAware.java |  2 -
 .../groovy/groovy/transform/CompileStatic.java |  3 -
 .../groovy/groovy/transform/EqualsAndHashCode.java |  1 -
 .../groovy/transform/ExternalizeMethods.java   |  1 -
 .../groovy/groovy/transform/IndexedProperty.java   |  1 -
 .../groovy/transform/InheritConstructors.java  |  1 -
 src/main/groovy/groovy/transform/Memoized.java |  2 -
 src/main/groovy/groovy/transform/PackageScope.java |  1 -
 .../groovy/transform/PackageScopeTarget.java   |  1 -
 src/main/groovy/groovy/transform/SelfType.java |  1 -
 src/main/groovy/groovy/transform/SourceURI.java|  1 -
 src/main/groovy/groovy/transform/Synchronized.java |  1 -
 .../groovy/groovy/transform/ThreadInterrupt.groovy | 66 --
 src/main/groovy/groovy/transform/ToString.java |  2 -
 src/main/groovy/groovy/transform/TypeChecked.java  |  1 -
 .../groovy/groovy/transform/TypeCheckingMode.java  |  2 -
 src/main/groovy/groovy/transform/WithReadLock.java |  1 -
 .../groovy/groovy/transform/WithWriteLock.java |  1 -
 24 files changed, 52 insertions(+), 48 deletions(-)

diff --git a/gradle/pomconfigurer.gradle b/gradle/pomconfigurer.gradle
index e9bd98b..49e447e 100644
--- a/gradle/pomconfigurer.gradle
+++ b/gradle/pomconfigurer.gradle
@@ -720,6 +720,9 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 name 'Vladimir Vivien'
 }
 contributor {
+name 'Vladimir Orany'
+}
+contributor {
 name 'Joe Wolf'
 }
 contributor {
diff --git a/src/main/groovy/groovy/transform/ASTTest.java 
b/src/main/groovy/groovy/transform/ASTTest.java
index 1cdb480..f72759f 100644
--- a/src/main/groovy/groovy/transform/ASTTest.java
+++ b/src/main/groovy/groovy/transform/ASTTest.java
@@ -49,7 +49,6 @@ import java.lang.annotation.Target;
  * The node variable refers to the AST node where the AST test 
annotation is put. In the previous example,
  * it means that node refers to the declaration node (int x).
  *
- * @author Cedric Champeau
  * @since 2.0.0
  *
  */
diff --git a/src/main/groovy/groovy/transform/AnnotationCollector.java 
b/src/main/groovy/groovy/transform/AnnotationCollector.java
index d0a5653..b2476ea 100644
--- a/src/main/groovy/groovy/transform/AnnotationCollector.java
+++ b/src/main/groovy/groovy/transform/AnnotationCollector.java
@@ -170,7 +170,6 @@ import java.lang.annotation.Target;
  * assert user.toString() == 'User(mrhaki)'
  * 
  * 
- * @author mailto:blackd...@gmx.org";>Jochen "blackdrag" Theodorou
  * @see org.codehaus.groovy.transform.AnnotationCollectorTransform
  */
 @java.lang.annotation.Documented
diff --git a/src/main/groovy/groovy/transform/AutoClone.java 
b/src/main/groovy/groovy/transform/AutoClone.java
index 995a187..f165cab 100644
--- a/src/main/groovy/groovy/transform/AutoClone.java
+++ b/src/main/groovy/groovy/transform/AutoClone.java
@@ -230,7 +230,6 @@ import java.lang.annotation.Target;
  * http://www.agiledeveloper.com/articles/cloning072002.htm";>http://www.agiledeveloper.com/articles/cloning072002.htm
  * 
  *
- * @author Paul King
  * @see AutoCloneStyle
  * @see ExternalizeMethods
  * @since 1.8.0
diff --git a/src/main/groovy/groovy/transform/AutoCloneStyle.java 
b/src/main/groovy/groovy/transform/AutoCloneStyle.java
index 81b0ebd..d2b4521 100644
--- a/src/main/groovy/groovy/transform/AutoCloneStyle.java
+++ b/src/main/groovy/groovy/transform/AutoCloneStyle.java
@@ -21,7 +21,6 @@ package groovy.transform;
 /**
  * Intended style to use for cloning when using the {@code @}AutoClone 
annotation.
  *
- * @author Paul King
  * @since 1.8.0
  * @see groovy.transform.AutoClone
  */
diff --git a/src/main/groovy/groovy/transform/BaseScript.java 
b/src/main/groovy/groovy/transform/BaseScript.java
index 2720b79..bc058cc 100644
--- a/src/main/groovy/groovy/transform/BaseScript.java
+++ b/src/main/groovy/groovy/transform/BaseScript.java
@@ -139,10 +139,6 @@ import java.lang.annotation.Target;
  * assert car.distance == 20
  * assert car.state == 'stopped'
  * 
- * 
- * @author Paul King
- * @author Vladimir Orany
- * @author Jim White
  * @since 2.2.0
  */
 @java.lang.annotation.Documented
diff --git a/src

[groovy] 01/02: whitespace/formatting plus cleanup/remove @author tags as per Apache recommended practices cont'd

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit 5e263d87c0871143117c7d2ab8579d02456ae029
Author: Paul King 
AuthorDate: Wed Jan 2 08:35:36 2019 +1000

whitespace/formatting plus cleanup/remove @author tags as per Apache 
recommended practices cont'd
---
 gradle/pomconfigurer.gradle|   3 +
 .../groovy/groovy/transform/CompileDynamic.groovy  |   3 +-
 .../groovy/transform/ConditionalInterrupt.groovy   |  74 ---
 .../groovy/groovy/transform/TailRecursive.groovy   |   3 +-
 .../groovy/groovy/transform/ThreadInterrupt.groovy | 127 +---
 .../groovy/groovy/transform/TimedInterrupt.groovy  |   8 +-
 src/test/groovy/io/LineColumnReaderTest.groovy |   3 -
 .../groovy/lang/BenchmarkInterceptorTest.groovy|  29 +--
 src/test/groovy/lang/ClosureResolvingTest.groovy   |   3 +-
 .../groovy/lang/DelegatingMetaClassTest.groovy |   8 +-
 .../lang/ExpandoMetaClassCreationHandleTest.groovy |  41 ++--
 src/test/groovy/lang/ExpandoMetaClassTest.groovy   | 126 ++--
 src/test/groovy/lang/GetMethodsTest.groovy |  13 +-
 src/test/groovy/lang/GroovyCodeSourceTest.groovy   |   7 +-
 src/test/groovy/lang/GroovySystemTest.groovy   |  10 +-
 src/test/groovy/lang/InterceptorTest.groovy|   4 +-
 src/test/groovy/lang/MapOfClosureTest.groovy   | 229 +++--
 src/test/groovy/lang/MetaClassRegistryTest.groovy  |   9 +-
 src/test/groovy/lang/PropertyMissingTest.groovy|  22 +-
 .../groovy/lang/ReferenceSerializationTest.groovy  |   2 -
 src/test/groovy/lang/RespondsToTest.groovy |  12 +-
 .../lang/ScriptSourcePositionInAstTest.groovy  |   4 +-
 src/test/groovy/mock/interceptor/Caller.groovy |  15 +-
 .../groovy/mock/interceptor/Collaborator.groovy|  11 +-
 .../mock/interceptor/MockSingleCallTest.groovy |  38 ++--
 .../mock/interceptor/StubCallSequenceTest.groovy   |   2 -
 .../mock/interceptor/StubSingleCallTest.groovy |   6 +-
 .../groovy/operator/BitwiseOperatorsTest.groovy|  32 ++-
 src/test/groovy/operator/NegateListsTest.groovy|   6 +-
 src/test/groovy/operator/PowerOperatorsTest.groovy |  21 +-
 .../groovy/operator/SpreadListOperatorTest.groovy  |   8 +-
 .../groovy/operator/SpreadMapOperatorTest.groovy   |  64 +++---
 src/test/groovy/time/TimeCategoryTest.groovy   |  10 +-
 src/test/groovy/transform/LazyTest.groovy  |  18 +-
 .../groovy/transform/ThreadInterruptTest.groovy|   2 -
 35 files changed, 444 insertions(+), 529 deletions(-)

diff --git a/gradle/pomconfigurer.gradle b/gradle/pomconfigurer.gradle
index 61aa7bb..e9bd98b 100644
--- a/gradle/pomconfigurer.gradle
+++ b/gradle/pomconfigurer.gradle
@@ -755,6 +755,9 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 contributor {
 name 'Peter Gromov'
 }
+contributor {
+name 'Johannes Link'
+}
 }
 mailingLists {
 mailingList {
diff --git a/src/main/groovy/groovy/transform/CompileDynamic.groovy 
b/src/main/groovy/groovy/transform/CompileDynamic.groovy
index c5247d4..11cd23b 100644
--- a/src/main/groovy/groovy/transform/CompileDynamic.groovy
+++ b/src/main/groovy/groovy/transform/CompileDynamic.groovy
@@ -25,10 +25,9 @@ import java.lang.annotation.Documented
  * This can be used for example if you statically compile a full class but you 
want to skip
  * some methods without having to use the full annotation.
  *
- * @author Cedric Champeau
  * @since 2.1.0
  */
 @Documented
 @AnnotationCollector(processor = 
"org.codehaus.groovy.transform.CompileDynamicProcessor")
-public @interface CompileDynamic {
+@interface CompileDynamic {
 }
\ No newline at end of file
diff --git a/src/main/groovy/groovy/transform/ConditionalInterrupt.groovy 
b/src/main/groovy/groovy/transform/ConditionalInterrupt.groovy
index 66aa64c..085 100644
--- a/src/main/groovy/groovy/transform/ConditionalInterrupt.groovy
+++ b/src/main/groovy/groovy/transform/ConditionalInterrupt.groovy
@@ -102,9 +102,6 @@ import java.lang.annotation.Target
  *
  * @see TimedInterrupt
  * @see ThreadInterrupt
- * @author Cedric Champeau
- * @author Hamlet D'Arcy
- * @author Paul King
  * @since 1.8.0
  */
 @java.lang.annotation.Documented
@@ -112,44 +109,43 @@ import java.lang.annotation.Target
 @Target([ElementType.PACKAGE, ElementType.METHOD, ElementType.FIELD, 
ElementType.TYPE, ElementType.LOCAL_VARIABLE])
 
@GroovyASTTransformationClass(["org.codehaus.groovy.transform.ConditionalInterruptibleASTTransformation"])
 @interface ConditionalInterrupt {
-  /**
-   * Set this to false if you have multiple classes within one source file and 
only
-   * want a conditional check on some of the classes. Place annotations on the 
classes
-   * you want enhanced. Set to true (the default) for blanket coverage of 
conditional
-   * checks on all methods, loops an

[groovy] branch GROOVY_2_5_X updated (dd36f04 -> b01db1b)

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a change to branch GROOVY_2_5_X
in repository https://gitbox.apache.org/repos/asf/groovy.git.


from dd36f04  cleanup/remove @author tags as per Apache recommended 
practices for groovy-bsf, groovy-jmx, groovy-json, groovy-jsr223
 new 1630fc1  cleanup/remove @author tags as per Apache recommended 
practices cont'd
 new adc693b  test already covered by groovyTestCase cases
 new 0b00c8a  cleanup/remove @author tags as per Apache recommended 
practices cont'd
 new f8e8ad1  whitespace/formatting plus cleanup/remove @author tags as per 
Apache recommended practices cont'd
 new b01db1b  whitespace/formatting plus cleanup/remove @author tags as per 
Apache recommended practices cont'd

The 5 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:
 gradle/pomconfigurer.gradle|  18 +-
 src/main/groovy/groovy/transform/ASTTest.java  |   1 -
 .../groovy/transform/AnnotationCollector.java  |   1 -
 src/main/groovy/groovy/transform/AutoClone.java|   1 -
 .../groovy/groovy/transform/AutoCloneStyle.java|   1 -
 src/main/groovy/groovy/transform/BaseScript.java   |   4 -
 .../groovy/transform/CompilationUnitAware.java |   2 -
 .../groovy/groovy/transform/CompileDynamic.groovy  |   3 +-
 .../groovy/groovy/transform/CompileStatic.java |   3 -
 .../groovy/transform/ConditionalInterrupt.groovy   |  74 +-
 .../groovy/groovy/transform/EqualsAndHashCode.java |   1 -
 .../groovy/transform/ExternalizeMethods.java   |   1 -
 .../groovy/groovy/transform/IndexedProperty.java   |   1 -
 .../groovy/transform/InheritConstructors.java  |   1 -
 src/main/groovy/groovy/transform/Memoized.java |   2 -
 src/main/groovy/groovy/transform/PackageScope.java |   1 -
 .../groovy/transform/PackageScopeTarget.java   |   1 -
 src/main/groovy/groovy/transform/SelfType.java |   1 -
 src/main/groovy/groovy/transform/SourceURI.java|   1 -
 src/main/groovy/groovy/transform/Synchronized.java |   1 -
 .../groovy/groovy/transform/TailRecursive.groovy   |   3 +-
 .../groovy/groovy/transform/ThreadInterrupt.groovy |  69 +-
 .../groovy/groovy/transform/TimedInterrupt.groovy  |   8 +-
 src/main/groovy/groovy/transform/ToString.java |   2 -
 src/main/groovy/groovy/transform/TypeChecked.java  |   1 -
 .../groovy/groovy/transform/TypeCheckingMode.java  |   2 -
 src/main/groovy/groovy/transform/WithReadLock.java |   1 -
 .../groovy/groovy/transform/WithWriteLock.java |   1 -
 .../test/builder/ObjectGraphBuilderTest.groovy |   2 -
 src/test/gls/enums/EnumTest.groovy |   3 -
 src/test/gls/innerClass/InnerInterfaceTest.groovy  |   2 -
 src/test/gls/syntax/BinaryLiteralTest.groovy   |   4 -
 src/test/gls/syntax/Gep3OrderDslTest.groovy|   3 -
 src/test/gls/syntax/UnderscoreInNumbersTest.groovy |   2 -
 src/test/groovy/AmbiguousInvocationTest.groovy |   4 +-
 src/test/groovy/BinaryStreamsTest.groovy   |   2 -
 src/test/groovy/BreakContinueLabelTest.groovy  |   2 -
 src/test/groovy/CallInnerClassCtorTest.groovy  |   1 -
 src/test/groovy/ClassExpressionTest.groovy |   2 -
 src/test/groovy/ClosureAsParamTest.groovy  |   3 -
 src/test/groovy/ClosureCloneTest.groovy|   3 -
 src/test/groovy/ClosureComparatorTest.groovy   |   8 +-
 src/test/groovy/ClosureComposeTest.groovy  |   2 +-
 src/test/groovy/ClosureDefaultParameterTest.groovy |   3 -
 src/test/groovy/ClosureInStaticMethodTest.groovy   |  35 +-
 src/test/groovy/ClosureMethodCallTest.groovy   |  41 +-
 src/test/groovy/ClosureMethodTest.groovy   | 140 ++--
 src/test/groovy/ClosureMethodsOnFileTest.groovy|   8 +-
 src/test/groovy/ClosureMissingMethodTest.groovy|  93 ++-
 src/test/groovy/ClosureReturnTest.groovy   |  12 +-
 src/test/groovy/ClosureTest.groovy |  54 +-
 .../groovy/ClosureUsingOuterVariablesTest.groovy   |  47 +-
 src/test/groovy/ClosureWithDefaultParamTest.groovy |  72 +-
 .../groovy/ClosureWithEmptyParametersTest.groovy   |   5 +-
 src/test/groovy/DummyMethodsGroovy.groovy  |   3 -
 src/test/groovy/ExceptionInClosureTest.groovy  |  11 +-
 src/test/groovy/FileTest.groovy|   3 -
 src/test/groovy/FilterLineTest.groovy  |   2 -
 src/test/groovy/Foo.groovy |   2 -
 src/test/groovy/GroovyClosureMethodsTest.groovy|   2 -
 src/test/groovy/HexTest.groovy |   2 -
 src/test/groovy/IdentityClosureTest.groovy |   3 -
 src/test/groovy/ImmutableModificationTest.groovy   |  43 --
 .../groovy/InvokeNormalMethodsFirstTest.groovy |   2 -
 src/test/groovy/LiteralTypesTest.groovy| 190 +++---
 src/test/groovy/MapCons

[groovy] 01/05: cleanup/remove @author tags as per Apache recommended practices cont'd

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch GROOVY_2_5_X
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit 1630fc11f6f52270911e799811902e65185dc0eb
Author: Paul King 
AuthorDate: Tue Jan 1 16:18:31 2019 +1000

cleanup/remove @author tags as per Apache recommended practices cont'd
---
 gradle/pomconfigurer.gradle|   6 +
 .../test/builder/ObjectGraphBuilderTest.groovy |   2 -
 src/test/gls/enums/EnumTest.groovy |   3 -
 src/test/gls/innerClass/InnerInterfaceTest.groovy  |   2 -
 src/test/gls/syntax/BinaryLiteralTest.groovy   |   4 -
 src/test/gls/syntax/Gep3OrderDslTest.groovy|   3 -
 src/test/gls/syntax/UnderscoreInNumbersTest.groovy |   2 -
 src/test/groovy/AmbiguousInvocationTest.groovy |   4 +-
 src/test/groovy/BinaryStreamsTest.groovy   |   2 -
 src/test/groovy/BreakContinueLabelTest.groovy  |   2 -
 src/test/groovy/CallInnerClassCtorTest.groovy  |   1 -
 src/test/groovy/ClassExpressionTest.groovy |   2 -
 src/test/groovy/ClosureAsParamTest.groovy  |   3 -
 src/test/groovy/ClosureCloneTest.groovy|   3 -
 src/test/groovy/ClosureComparatorTest.groovy   |   8 +-
 src/test/groovy/ClosureComposeTest.groovy  |   2 +-
 src/test/groovy/ClosureDefaultParameterTest.groovy |   3 -
 src/test/groovy/ClosureInStaticMethodTest.groovy   |  35 +++---
 src/test/groovy/ClosureMethodCallTest.groovy   |  41 +++---
 src/test/groovy/ClosureMethodTest.groovy   | 140 ++---
 src/test/groovy/ClosureMethodsOnFileTest.groovy|   8 +-
 src/test/groovy/ClosureMissingMethodTest.groovy|  93 +++---
 src/test/groovy/ClosureReturnTest.groovy   |  12 +-
 src/test/groovy/ClosureTest.groovy |  54 
 .../groovy/ClosureUsingOuterVariablesTest.groovy   |  47 ---
 src/test/groovy/ClosureWithDefaultParamTest.groovy |  72 ++-
 .../groovy/ClosureWithEmptyParametersTest.groovy   |   5 +-
 src/test/groovy/DummyMethodsGroovy.groovy  |   3 -
 src/test/groovy/ExceptionInClosureTest.groovy  |  11 +-
 29 files changed, 256 insertions(+), 317 deletions(-)

diff --git a/gradle/pomconfigurer.gradle b/gradle/pomconfigurer.gradle
index 828d408..6a5ce05 100644
--- a/gradle/pomconfigurer.gradle
+++ b/gradle/pomconfigurer.gradle
@@ -699,6 +699,12 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 name 'Martin Stockhammer'
 }
 contributor {
+name 'Martin C. Martin'
+}
+contributor {
+name 'Alexey Verkhovsky'
+}
+contributor {
 name 'Alberto Mijares'
 }
 contributor {
diff --git a/src/spec/test/builder/ObjectGraphBuilderTest.groovy 
b/src/spec/test/builder/ObjectGraphBuilderTest.groovy
index f3e487f..3f3d987 100644
--- a/src/spec/test/builder/ObjectGraphBuilderTest.groovy
+++ b/src/spec/test/builder/ObjectGraphBuilderTest.groovy
@@ -24,8 +24,6 @@ import asciidoctor.Utils
 * Tests for ObjectGraphBuilder. The tests directly in this file are specific
 * to ObjectGraphBuilder. Functionality in common with other Builders
 * is tested in the parent class.
-*
-* @author Groovy Documentation Community
 */
 class ObjectGraphBuilderTest  extends GroovyTestCase {
 
diff --git a/src/test/gls/enums/EnumTest.groovy 
b/src/test/gls/enums/EnumTest.groovy
index e883ed1..684f55e 100644
--- a/src/test/gls/enums/EnumTest.groovy
+++ b/src/test/gls/enums/EnumTest.groovy
@@ -22,9 +22,6 @@ import gls.CompilableTestSupport
 
 /**
  * Tests various properties of enums.
- *
- * @author Paul King
- * @author Roshan Dawrani
  */
 class EnumTest extends CompilableTestSupport {
 
diff --git a/src/test/gls/innerClass/InnerInterfaceTest.groovy 
b/src/test/gls/innerClass/InnerInterfaceTest.groovy
index 8f66866..41d407a 100644
--- a/src/test/gls/innerClass/InnerInterfaceTest.groovy
+++ b/src/test/gls/innerClass/InnerInterfaceTest.groovy
@@ -20,8 +20,6 @@ package gls.innerClass
 
 /**
  * Tests on inner interface usage
- *
- * @author Roshan Dawrani
  */
 class InnerInterfaceTest extends GroovyTestCase {
 
diff --git a/src/test/gls/syntax/BinaryLiteralTest.groovy 
b/src/test/gls/syntax/BinaryLiteralTest.groovy
index c9b16a9..8cdea34 100644
--- a/src/test/gls/syntax/BinaryLiteralTest.groovy
+++ b/src/test/gls/syntax/BinaryLiteralTest.groovy
@@ -20,10 +20,6 @@ package gls.syntax
 
 import gls.CompilableTestSupport
 
-/**
- * 
- * @author Guillaume Laforge
- */
 class BinaryLiteralTest extends CompilableTestSupport {
 
 void testSomeBinaryLiteralNumbers() {
diff --git a/src/test/gls/syntax/Gep3OrderDslTest.groovy 
b/src/test/gls/syntax/Gep3OrderDslTest.groovy
index f906b5c..640beaa 100644
--- a/src/test/gls/syntax/Gep3OrderDslTest.groovy
+++ b/src/test/gls/syntax/Gep3OrderDslTest.groovy
@@ -20,9 +20,6 @@ package gls.syntax
 
 import org.codehaus.groovy

[groovy] 03/05: cleanup/remove @author tags as per Apache recommended practices cont'd

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch GROOVY_2_5_X
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit 0b00c8ad62469c547466d863d3f60f35c29c0adc
Author: Paul King 
AuthorDate: Tue Jan 1 19:42:01 2019 +1000

cleanup/remove @author tags as per Apache recommended practices cont'd
---
 gradle/pomconfigurer.gradle|   6 +-
 src/test/groovy/FileTest.groovy|   3 -
 src/test/groovy/FilterLineTest.groovy  |   2 -
 src/test/groovy/Foo.groovy |   2 -
 src/test/groovy/GroovyClosureMethodsTest.groovy|   2 -
 src/test/groovy/HexTest.groovy |   2 -
 src/test/groovy/IdentityClosureTest.groovy |   3 -
 .../groovy/InvokeNormalMethodsFirstTest.groovy |   2 -
 src/test/groovy/LiteralTypesTest.groovy| 190 +++---
 src/test/groovy/MapConstructionTest.groovy |   2 -
 src/test/groovy/MapPropertyTest.groovy |  19 +-
 .../MethodParameterAccessWithinClosureTest.groovy  |  51 +-
 src/test/groovy/MinMaxTest.groovy  |  38 +-
 src/test/groovy/MultiCatchTest.groovy  |   4 +-
 src/test/groovy/MultiDimArraysTest.groovy  |   6 +-
 src/test/groovy/NullPropertyTest.groovy|  11 +-
 src/test/groovy/OverridePropertyGetterTest.groovy  |  23 +-
 ...ateVariableAccessFromAnotherInstanceTest.groovy |  14 +-
 src/test/groovy/ProcessTest.groovy |  48 +-
 src/test/groovy/Property2Test.groovy   |   3 -
 src/test/groovy/PropertyTest.groovy|  55 +-
 src/test/groovy/ReadLineTest.groovy|  43 +-
 src/test/groovy/RegularExpressionsTest.groovy  | 130 ++--
 src/test/groovy/ReturnTest.groovy  |  26 +-
 src/test/groovy/SocketTest.groovy  |   4 +-
 src/test/groovy/SortTest.groovy|  38 +-
 src/test/groovy/SpreadDotTest.groovy   |  99 +--
 .../groovy/SwitchWithDifferentTypesTest.groovy |  48 +-
 src/test/groovy/TextPropertyTest.groovy|  14 +-
 .../UniqueOnCollectionWithClosureTest.groovy   |  15 +-
 .../UniqueOnCollectionWithComparatorTest.groovy| 153 ++---
 src/test/groovy/VarargsMethodTest.groovy   |  73 +-
 src/test/groovy/annotations/MyIntegerAnno.groovy   |  69 +-
 src/test/groovy/beans/BindableTest.groovy  | 745 ++---
 34 files changed, 947 insertions(+), 996 deletions(-)

diff --git a/gradle/pomconfigurer.gradle b/gradle/pomconfigurer.gradle
index 6a5ce05..61aa7bb 100644
--- a/gradle/pomconfigurer.gradle
+++ b/gradle/pomconfigurer.gradle
@@ -588,9 +588,6 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 name 'Chris K Wensel'
 }
 contributor {
-name 'Joachim Baumann'
-}
-contributor {
 name 'David Sutherland'
 }
 contributor {
@@ -684,6 +681,9 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 name 'Thomas Heller'
 }
 contributor {
+name 'John Stump'
+}
+contributor {
 name 'Ivan Ganza'
 }
 contributor {
diff --git a/src/test/groovy/FileTest.groovy b/src/test/groovy/FileTest.groovy
index 613a05d..9e9fe11 100644
--- a/src/test/groovy/FileTest.groovy
+++ b/src/test/groovy/FileTest.groovy
@@ -23,9 +23,6 @@ import static groovy.io.FileVisitResult.*
 
 /**
  * Unit test for File GDK methods
- *
- * @author Marc Guillemot
- * @author Paul King
  */
 class FileTest extends GroovyTestCase {
 
diff --git a/src/test/groovy/FilterLineTest.groovy 
b/src/test/groovy/FilterLineTest.groovy
index a091ad9..1611e40 100644
--- a/src/test/groovy/FilterLineTest.groovy
+++ b/src/test/groovy/FilterLineTest.groovy
@@ -22,8 +22,6 @@ package groovy
  * check that the new filterLine() method on InputStream is ok
  * (and indirectly test newReader() method on InputStream)
  * as specified in GROOVY-624 and GROOVY-625
- *
- * @author mailto:jeremy.ray...@bigfoot.com";>Jeremy Rayner
  */
 
 class FilterLineTest extends GroovyTestCase {
diff --git a/src/test/groovy/Foo.groovy b/src/test/groovy/Foo.groovy
index ce183e2..889dd53 100644
--- a/src/test/groovy/Foo.groovy
+++ b/src/test/groovy/Foo.groovy
@@ -20,8 +20,6 @@ package groovy
 
 /** 
  * A dummy bean for testing the use of properties in Groovy
- * 
- * @author mailto:ja...@coredevelopers.net";>James Strachan
  */
 class Foo implements Serializable {
 
diff --git a/src/test/groovy/GroovyClosureMethodsTest.groovy 
b/src/test/groovy/GroovyClosureMethodsTest.groovy
index 256d1f6..f43a7b5 100644
--- a/src/test/groovy/GroovyClosureMethodsTest.groovy
+++ b/src/test/groovy/GroovyClosureMethodsTest.groovy
@@ -25,8 +25,6 @@ import java.util.concurrent.TimeUnit
  * Test case for the eachObject method on a file containing
  * zero, one or more objects (obj

[groovy] 04/05: whitespace/formatting plus cleanup/remove @author tags as per Apache recommended practices cont'd

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch GROOVY_2_5_X
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit f8e8ad1d58fc3b879cfbb793b123eb25f9e047eb
Author: Paul King 
AuthorDate: Wed Jan 2 08:35:36 2019 +1000

whitespace/formatting plus cleanup/remove @author tags as per Apache 
recommended practices cont'd
---
 gradle/pomconfigurer.gradle|   3 +
 .../groovy/groovy/transform/CompileDynamic.groovy  |   3 +-
 .../groovy/transform/ConditionalInterrupt.groovy   |  74 ---
 .../groovy/groovy/transform/TailRecursive.groovy   |   3 +-
 .../groovy/groovy/transform/ThreadInterrupt.groovy | 127 +---
 .../groovy/groovy/transform/TimedInterrupt.groovy  |   8 +-
 src/test/groovy/io/LineColumnReaderTest.groovy |   3 -
 .../groovy/lang/BenchmarkInterceptorTest.groovy|  29 +--
 src/test/groovy/lang/ClosureResolvingTest.groovy   |   3 +-
 .../groovy/lang/DelegatingMetaClassTest.groovy |   8 +-
 .../lang/ExpandoMetaClassCreationHandleTest.groovy |  41 ++--
 src/test/groovy/lang/ExpandoMetaClassTest.groovy   | 126 ++--
 src/test/groovy/lang/GetMethodsTest.groovy |  13 +-
 src/test/groovy/lang/GroovyCodeSourceTest.groovy   |   7 +-
 src/test/groovy/lang/GroovySystemTest.groovy   |  10 +-
 src/test/groovy/lang/InterceptorTest.groovy|   4 +-
 src/test/groovy/lang/MapOfClosureTest.groovy   | 229 +++--
 src/test/groovy/lang/MetaClassRegistryTest.groovy  |   9 +-
 src/test/groovy/lang/PropertyMissingTest.groovy|  22 +-
 .../groovy/lang/ReferenceSerializationTest.groovy  |   2 -
 src/test/groovy/lang/RespondsToTest.groovy |  12 +-
 .../lang/ScriptSourcePositionInAstTest.groovy  |   4 +-
 src/test/groovy/mock/interceptor/Caller.groovy |  15 +-
 .../groovy/mock/interceptor/Collaborator.groovy|  11 +-
 .../mock/interceptor/MockSingleCallTest.groovy |  38 ++--
 .../mock/interceptor/StubCallSequenceTest.groovy   |   2 -
 .../mock/interceptor/StubSingleCallTest.groovy |   6 +-
 .../groovy/operator/BitwiseOperatorsTest.groovy|  32 ++-
 src/test/groovy/operator/NegateListsTest.groovy|   6 +-
 src/test/groovy/operator/PowerOperatorsTest.groovy |  21 +-
 .../groovy/operator/SpreadListOperatorTest.groovy  |   8 +-
 .../groovy/operator/SpreadMapOperatorTest.groovy   |  64 +++---
 src/test/groovy/time/TimeCategoryTest.groovy   |  10 +-
 src/test/groovy/transform/LazyTest.groovy  |  18 +-
 .../groovy/transform/ThreadInterruptTest.groovy|   2 -
 35 files changed, 444 insertions(+), 529 deletions(-)

diff --git a/gradle/pomconfigurer.gradle b/gradle/pomconfigurer.gradle
index 61aa7bb..e9bd98b 100644
--- a/gradle/pomconfigurer.gradle
+++ b/gradle/pomconfigurer.gradle
@@ -755,6 +755,9 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 contributor {
 name 'Peter Gromov'
 }
+contributor {
+name 'Johannes Link'
+}
 }
 mailingLists {
 mailingList {
diff --git a/src/main/groovy/groovy/transform/CompileDynamic.groovy 
b/src/main/groovy/groovy/transform/CompileDynamic.groovy
index c5247d4..11cd23b 100644
--- a/src/main/groovy/groovy/transform/CompileDynamic.groovy
+++ b/src/main/groovy/groovy/transform/CompileDynamic.groovy
@@ -25,10 +25,9 @@ import java.lang.annotation.Documented
  * This can be used for example if you statically compile a full class but you 
want to skip
  * some methods without having to use the full annotation.
  *
- * @author Cedric Champeau
  * @since 2.1.0
  */
 @Documented
 @AnnotationCollector(processor = 
"org.codehaus.groovy.transform.CompileDynamicProcessor")
-public @interface CompileDynamic {
+@interface CompileDynamic {
 }
\ No newline at end of file
diff --git a/src/main/groovy/groovy/transform/ConditionalInterrupt.groovy 
b/src/main/groovy/groovy/transform/ConditionalInterrupt.groovy
index 66aa64c..085 100644
--- a/src/main/groovy/groovy/transform/ConditionalInterrupt.groovy
+++ b/src/main/groovy/groovy/transform/ConditionalInterrupt.groovy
@@ -102,9 +102,6 @@ import java.lang.annotation.Target
  *
  * @see TimedInterrupt
  * @see ThreadInterrupt
- * @author Cedric Champeau
- * @author Hamlet D'Arcy
- * @author Paul King
  * @since 1.8.0
  */
 @java.lang.annotation.Documented
@@ -112,44 +109,43 @@ import java.lang.annotation.Target
 @Target([ElementType.PACKAGE, ElementType.METHOD, ElementType.FIELD, 
ElementType.TYPE, ElementType.LOCAL_VARIABLE])
 
@GroovyASTTransformationClass(["org.codehaus.groovy.transform.ConditionalInterruptibleASTTransformation"])
 @interface ConditionalInterrupt {
-  /**
-   * Set this to false if you have multiple classes within one source file and 
only
-   * want a conditional check on some of the classes. Place annotations on the 
classes
-   * you want enhanced. Set to true (the default) for blanket coverage of 
conditional
-   * checks on all methods, lo

[groovy] 02/05: test already covered by groovyTestCase cases

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch GROOVY_2_5_X
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit adc693b6e20d652cae338f0c9451bc6130c0c2b5
Author: Paul King 
AuthorDate: Tue Jan 1 19:21:07 2019 +1000

test already covered by groovyTestCase cases
---
 src/test/groovy/ImmutableModificationTest.groovy | 43 
 1 file changed, 43 deletions(-)

diff --git a/src/test/groovy/ImmutableModificationTest.groovy 
b/src/test/groovy/ImmutableModificationTest.groovy
deleted file mode 100644
index 3647057..000
--- a/src/test/groovy/ImmutableModificationTest.groovy
+++ /dev/null
@@ -1,43 +0,0 @@
-/*
- *  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 groovy
-
-/**
- * check that the new asImmutable() method works
- * as specified in GROOVY-623
- *
- * @author mailto:jeremy.ray...@bigfoot.com";>Jeremy Rayner
- */
-
-class ImmutableModificationTest extends GroovyTestCase {
-void testCollectionAsImmutable() {
-def challenger = ["Telson", "Sharna", "Darv", "Astra"]
-def hopefullyImmutable = challenger.asImmutable()
-try {
-challenger.add("Angel One")
-challenger << "Angel Two"
-
-// @todo fail("'challenger' is supposed to be an immutable 
collection.")
-
-} catch (UnsupportedOperationException e) {
-// success if this exception is thrown
-assert 4 == challenger.size()
-}
-}
-}
\ No newline at end of file

[groovy] 05/05: whitespace/formatting plus cleanup/remove @author tags as per Apache recommended practices cont'd

2019-01-01 Thread paulk 
This is an automated email from the ASF dual-hosted git repository.

paulk pushed a commit to branch GROOVY_2_5_X
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit b01db1b0c04711ce3072601d13ce7a802e242e53
Author: Paul King 
AuthorDate: Wed Jan 2 08:45:39 2019 +1000

whitespace/formatting plus cleanup/remove @author tags as per Apache 
recommended practices cont'd
---
 gradle/pomconfigurer.gradle|  3 +
 src/main/groovy/groovy/transform/ASTTest.java  |  1 -
 .../groovy/transform/AnnotationCollector.java  |  1 -
 src/main/groovy/groovy/transform/AutoClone.java|  1 -
 .../groovy/groovy/transform/AutoCloneStyle.java|  1 -
 src/main/groovy/groovy/transform/BaseScript.java   |  4 --
 .../groovy/transform/CompilationUnitAware.java |  2 -
 .../groovy/groovy/transform/CompileStatic.java |  3 -
 .../groovy/groovy/transform/EqualsAndHashCode.java |  1 -
 .../groovy/transform/ExternalizeMethods.java   |  1 -
 .../groovy/groovy/transform/IndexedProperty.java   |  1 -
 .../groovy/transform/InheritConstructors.java  |  1 -
 src/main/groovy/groovy/transform/Memoized.java |  2 -
 src/main/groovy/groovy/transform/PackageScope.java |  1 -
 .../groovy/transform/PackageScopeTarget.java   |  1 -
 src/main/groovy/groovy/transform/SelfType.java |  1 -
 src/main/groovy/groovy/transform/SourceURI.java|  1 -
 src/main/groovy/groovy/transform/Synchronized.java |  1 -
 .../groovy/groovy/transform/ThreadInterrupt.groovy | 66 --
 src/main/groovy/groovy/transform/ToString.java |  2 -
 src/main/groovy/groovy/transform/TypeChecked.java  |  1 -
 .../groovy/groovy/transform/TypeCheckingMode.java  |  2 -
 src/main/groovy/groovy/transform/WithReadLock.java |  1 -
 .../groovy/groovy/transform/WithWriteLock.java |  1 -
 24 files changed, 52 insertions(+), 48 deletions(-)

diff --git a/gradle/pomconfigurer.gradle b/gradle/pomconfigurer.gradle
index e9bd98b..49e447e 100644
--- a/gradle/pomconfigurer.gradle
+++ b/gradle/pomconfigurer.gradle
@@ -720,6 +720,9 @@ project.ext.pomConfigureClosureWithoutTweaks = {
 name 'Vladimir Vivien'
 }
 contributor {
+name 'Vladimir Orany'
+}
+contributor {
 name 'Joe Wolf'
 }
 contributor {
diff --git a/src/main/groovy/groovy/transform/ASTTest.java 
b/src/main/groovy/groovy/transform/ASTTest.java
index 1cdb480..f72759f 100644
--- a/src/main/groovy/groovy/transform/ASTTest.java
+++ b/src/main/groovy/groovy/transform/ASTTest.java
@@ -49,7 +49,6 @@ import java.lang.annotation.Target;
  * The node variable refers to the AST node where the AST test 
annotation is put. In the previous example,
  * it means that node refers to the declaration node (int x).
  *
- * @author Cedric Champeau
  * @since 2.0.0
  *
  */
diff --git a/src/main/groovy/groovy/transform/AnnotationCollector.java 
b/src/main/groovy/groovy/transform/AnnotationCollector.java
index d0a5653..b2476ea 100644
--- a/src/main/groovy/groovy/transform/AnnotationCollector.java
+++ b/src/main/groovy/groovy/transform/AnnotationCollector.java
@@ -170,7 +170,6 @@ import java.lang.annotation.Target;
  * assert user.toString() == 'User(mrhaki)'
  * 
  * 
- * @author mailto:blackd...@gmx.org";>Jochen "blackdrag" Theodorou
  * @see org.codehaus.groovy.transform.AnnotationCollectorTransform
  */
 @java.lang.annotation.Documented
diff --git a/src/main/groovy/groovy/transform/AutoClone.java 
b/src/main/groovy/groovy/transform/AutoClone.java
index 995a187..f165cab 100644
--- a/src/main/groovy/groovy/transform/AutoClone.java
+++ b/src/main/groovy/groovy/transform/AutoClone.java
@@ -230,7 +230,6 @@ import java.lang.annotation.Target;
  * http://www.agiledeveloper.com/articles/cloning072002.htm";>http://www.agiledeveloper.com/articles/cloning072002.htm
  * 
  *
- * @author Paul King
  * @see AutoCloneStyle
  * @see ExternalizeMethods
  * @since 1.8.0
diff --git a/src/main/groovy/groovy/transform/AutoCloneStyle.java 
b/src/main/groovy/groovy/transform/AutoCloneStyle.java
index 81b0ebd..d2b4521 100644
--- a/src/main/groovy/groovy/transform/AutoCloneStyle.java
+++ b/src/main/groovy/groovy/transform/AutoCloneStyle.java
@@ -21,7 +21,6 @@ package groovy.transform;
 /**
  * Intended style to use for cloning when using the {@code @}AutoClone 
annotation.
  *
- * @author Paul King
  * @since 1.8.0
  * @see groovy.transform.AutoClone
  */
diff --git a/src/main/groovy/groovy/transform/BaseScript.java 
b/src/main/groovy/groovy/transform/BaseScript.java
index 2720b79..bc058cc 100644
--- a/src/main/groovy/groovy/transform/BaseScript.java
+++ b/src/main/groovy/groovy/transform/BaseScript.java
@@ -139,10 +139,6 @@ import java.lang.annotation.Target;
  * assert car.distance == 20
  * assert car.state == 'stopped'
  * 
- * 
- * @author Paul King
- * @author Vladimir Orany
- * @author Jim White
  * @since 2.2.0
  */
 @java.lang.annotation.Documented
diff --git