[GitHub] incubator-groovy pull request: GROOVY-7680: Bump gradle to version...

2015-11-18 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/192 GROOVY-7680: Bump gradle to version 2.9 (closes #192) You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert/incubator

[GitHub] incubator-groovy pull request: GROOVY-7672: TupleConstructor shoul...

2015-11-11 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/187 GROOVY-7672: TupleConstructor should have only SOURCE retention policy You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk

[GitHub] incubator-groovy pull request: GROOVY-7650: CLONE - collectEntries...

2015-10-28 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/166 GROOVY-7650: CLONE - collectEntries throws cryptic error when used wi… …th split(delimiter, limit) (closes #166) You can merge this pull request into a Git repository by running

[GitHub] incubator-groovy pull request:

2015-10-22 Thread paulk-asert
Github user paulk-asert commented on the pull request: https://github.com/apache/incubator-groovy/commit/643815aa1deec53cf0f775d0eb2e317c85da2526#commitcomment-13937078 Darn, Intellij showed the file as changed after doing 'gradlew wrapper' so I didn't check the contents. I'll fix

[GitHub] incubator-groovy pull request:

2015-10-12 Thread paulk-asert
Github user paulk-asert commented on the pull request: https://github.com/apache/incubator-groovy/commit/8ed5d29ed6cd7384cc34a1e9f3f28e7262b3e3ad#commitcomment-13713804 All good. Love your work! :-) On Mon, Oct 12, 2015 at 4:55 PM, Pascal Schumacher <notificati...@github.

[GitHub] incubator-groovy pull request:

2015-10-11 Thread paulk-asert
Github user paulk-asert commented on the pull request: https://github.com/apache/incubator-groovy/commit/57719291957e34c409552412304f4c6268ff1735#commitcomment-13707349 Yes, this was removed as part of the MacroGroovy PR (which isn't part of GROOVY_2_4_X). I had to put it back

[GitHub] incubator-groovy pull request: GROOVY-7548: @Grab should provide a...

2015-08-30 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/83 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: GROOVY-7551: groovy-all has a depen...

2015-08-20 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/91 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: GROOVY-7548: @Grab should provide a...

2015-08-15 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/83 GROOVY-7548: @Grab should provide a way to set system properties You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert

[GitHub] incubator-groovy pull request: GROOVY-7540: Add a method to String...

2015-08-07 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/80 GROOVY-7540: Add a method to StringGroovyMethods for replacing String… … pairs supplied as a Map You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-groovy pull request: GROOVY-7529: Fix up transforms (apa...

2015-08-05 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/77 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request:

2015-08-02 Thread paulk-asert
Github user paulk-asert commented on the pull request: https://github.com/apache/incubator-groovy/commit/971cee027b34b9e5d92ed6c8cce3c541af0974f6#commitcomment-12485046 In src/test/groovy/bugs/Groovy7520Bug.groovy: In src/test/groovy/bugs/Groovy7520Bug.groovy on line 1: test

[GitHub] incubator-groovy pull request: GROOVY-7529: Fix up transforms (apa...

2015-07-31 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/incubator-groovy/pull/77#discussion_r36029568 --- Diff: src/spec/doc/core-metaprogramming.adoc --- @@ -966,6 +966,25 @@ include::{projectdir}/src/spec/test

[GitHub] incubator-groovy pull request: GROOVY-7529: Fix up transforms (apa...

2015-07-30 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/77 GROOVY-7529: Fix up transforms (apart from TupleConstructor) which ar… …e affected by empty includes default You can merge this pull request into a Git repository by running

[GitHub] incubator-groovy pull request: GROOVY-7522: TupleConstructor overw...

2015-07-28 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/75 GROOVY-7522: TupleConstructor overwrites empty default constructors You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert

[GitHub] incubator-groovy pull request: GROOVY-7523: TupleConstructor with ...

2015-07-26 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/72 GROOVY-7523: TupleConstructor with empty includes includes all You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert

[GitHub] incubator-groovy pull request: GROOVY-7518: hashCode() throws NPE ...

2015-07-23 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/70 GROOVY-7518: hashCode() throws NPE when using @CompileStatic with @Eq… …ualsAndHashCode You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-groovy pull request: GROOVY-7486: In groovy.util.Node.de...

2015-07-14 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/59 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: Groovy7488

2015-07-14 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/61 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request:

2015-07-13 Thread paulk-asert
Github user paulk-asert commented on the pull request: https://github.com/apache/incubator-groovy/commit/ad43ad456b3c9828fc538c95998d9e6509700222#commitcomment-12132192 In src/test/org/codehaus/groovy/runtime/memoize/CacheCleanupCollectedSoftReferencesTest.groovy: In src/test

[GitHub] incubator-groovy pull request: GROOVY-7486: In groovy.util.Node.de...

2015-07-13 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/incubator-groovy/pull/59#discussion_r34463093 --- Diff: src/test/groovy/tree/NavigationTest.groovy --- @@ -57,11 +65,25 @@ class NavigationTest extends GroovyTestCase

[GitHub] incubator-groovy pull request: GROOVY-7486: In groovy.util.Node.de...

2015-07-13 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/incubator-groovy/pull/59#discussion_r34474352 --- Diff: src/main/groovy/util/Node.java --- @@ -537,26 +539,40 @@ private NodeList getByName(String name

[GitHub] incubator-groovy pull request: GROOVY-7495: Diamond inheritance of...

2015-07-09 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/58 GROOVY-7495: Diamond inheritance of interfaces makes method return ty… …pe incompatible You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-groovy pull request: GROOVY-7495: Diamond inheritance of...

2015-07-09 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/58 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: GROOVY-7492: Groovy should allow Co...

2015-07-05 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/54 GROOVY-7492: Groovy should allow CompileStatic classes to not impleme… …nt GroovyObject You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-groovy pull request: GROOVY-7491: Upgrade Ant to 1.9.6

2015-07-03 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/52 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: GROOVY-7416: Problem With CompileSt...

2015-07-03 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/23 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: GROOVY-7479: Groovy should provide ...

2015-07-03 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/48 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: GROOVY-7485: BUG! exception in phas...

2015-07-03 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/50 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: GROOVY-7491: Upgrade Ant to 1.9.6

2015-07-03 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/52 GROOVY-7491: Upgrade Ant to 1.9.6 You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert/incubator-groovy groovy7491

[GitHub] incubator-groovy pull request: GROOVY-7485: BUG! exception in phas...

2015-07-02 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/50 GROOVY-7485: BUG! exception in phase 'semantic analysis'... when tryi… …ng to call instance method as default for a constructor parameter You can merge this pull request into a Git

[GitHub] incubator-groovy pull request: GROOVY-7018 GroovyClassLoader addCl...

2015-06-30 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/incubator-groovy/pull/43#discussion_r33632554 --- Diff: src/main/groovy/ui/GroovyMain.java --- @@ -391,17 +391,16 @@ public String getText(String uriOrFilename) throws IOException

[GitHub] incubator-groovy pull request: GROOVY-7479: Groovy should provide ...

2015-06-29 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/48 GROOVY-7479: Groovy should provide inits and tails DGM methods fo… …r lists You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-groovy pull request: GROOVY-7477: CompileStatic-related ...

2015-06-28 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/47 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request: GROOVY-7477: CompileStatic-related ...

2015-06-26 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/47 GROOVY-7477: CompileStatic-related BUG! exception in phase 'instruct… …ion selection' You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-groovy pull request: GROOVY-5185: Cast operator preceden...

2015-06-22 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/42 GROOVY-5185: Cast operator precedence is incorrect I think this rolls back the prod code changes from GROOVY-2605 but we may have done other changes in the meantime since the tests

[GitHub] incubator-groovy pull request: GROOVY-5185: Cast operator preceden...

2015-06-22 Thread paulk-asert
Github user paulk-asert closed the pull request at: https://github.com/apache/incubator-groovy/pull/42 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] incubator-groovy pull request:

2015-06-19 Thread paulk-asert
Github user paulk-asert commented on the pull request: https://github.com/apache/incubator-groovy/commit/daac89dd587aadf6e21fb7b95bbab06e07976fbf#commitcomment-11779003 Please see: https://issues.apache.org/jira/browse/GROOVY-7470 It would be great if you could action. Thanks

[GitHub] incubator-groovy pull request: GROOVY-4811: exception while compil...

2015-06-18 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/41 GROOVY-4811: exception while compiling annotation type You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert/incubator

[GitHub] incubator-groovy pull request:

2015-06-03 Thread paulk-asert
Github user paulk-asert commented on the pull request: https://github.com/apache/incubator-groovy/commit/37a8cbde92d183e67b8b0a23de615576a5e47f46#commitcomment-11494315 And he's in! :-) --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-groovy pull request: GROOVY-6212: SpreadExpression BUG! ...

2015-05-31 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/30 GROOVY-6212: SpreadExpression BUG! when slicing a List You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert/incubator

[GitHub] incubator-groovy pull request: GROOVY-7434: Groovy should support ...

2015-05-25 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/24 GROOVY-7434: Groovy should support resolving ambiguous signatures whe… …n using ClosureParams You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-groovy pull request: GROOVY-7227 AST transformations rel...

2015-05-22 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/22 GROOVY-7227 AST transformations relating to properties/fields should validate their includes/excludes lists Ported (copied and reworked for recent changes) from the old repo

[GitHub] incubator-groovy pull request: GROOVY-6319: Canonical annotation s...

2015-05-20 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/21 GROOVY-6319: Canonical annotation should allow property names in toSt… …ring You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-groovy pull request: GROOVY-7353: Groovy should provide ...

2015-05-19 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/20 GROOVY-7353: Groovy should provide a MapConstructor AST transform You can merge this pull request into a Git repository by running: $ git pull https://github.com/paulk-asert

[GitHub] incubator-groovy pull request: GROOVY-7428: Change precedence prio...

2015-05-18 Thread paulk-asert
GitHub user paulk-asert opened a pull request: https://github.com/apache/incubator-groovy/pull/18 GROOVY-7428: Change precedence priority of ** compared to pre/post in… …c/dec and unary plus/minus You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-groovy pull request: GROOVY-7425: Adding 'getBytes(Map p...

2015-05-15 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/incubator-groovy/pull/16#discussion_r30414852 --- Diff: src/test/org/codehaus/groovy/runtime/URLGetBytesTest.groovy --- @@ -0,0 +1,114 @@ +/* + * Copyright 2003-2015 the original

[GitHub] incubator-groovy pull request: GROOVY-7425: Adding 'getBytes(Map p...

2015-05-15 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/incubator-groovy/pull/16#discussion_r30414741 --- Diff: src/main/org/codehaus/groovy/runtime/ResourceGroovyMethods.java --- @@ -694,6 +694,29 @@ public static String getText(URL url, Map

[GitHub] incubator-groovy pull request: GROOVY-7425: Adding 'getBytes(Map p...

2015-05-15 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/incubator-groovy/pull/16#discussion_r30414765 --- Diff: src/test/org/codehaus/groovy/runtime/URLGetBytesTest.groovy --- @@ -0,0 +1,114 @@ +/* + * Copyright 2003-2015 the original

[GitHub] incubator-groovy pull request: Access to Method parameter names at...

2015-05-13 Thread paulk-asert
Github user paulk-asert commented on a diff in the pull request: https://github.com/apache/incubator-groovy/pull/15#discussion_r30281069 --- Diff: src/main/groovy/ui/GroovyMain.java --- @@ -305,6 +310,7 @@ private static boolean process(CommandLine line) throws ParseException