Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/b02d2f57c09ba6ba46553ed77aff191ec9ab274e#commitcomment-22219164
In
subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/util/StringUtils.java:
In
subprojects/parser
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/b02d2f57c09ba6ba46553ed77aff191ec9ab274e#commitcomment-22219142
In
subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/util/StringUtils.java:
In
subprojects/parser
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/b02d2f57c09ba6ba46553ed77aff191ec9ab274e#commitcomment-22219131
In
subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/util/StringUtils.java:
In
subprojects/par
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/b02d2f57c09ba6ba46553ed77aff191ec9ab274e#commitcomment-22219095
In src/main/org/apache/groovy/util/Maps.java:
In src/main/org/apache/groovy/util/Maps.java on line 11:
Thanks for rev
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/542
---
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, or if the feature is enab
Github user jwagenleitner commented on the pull request:
https://github.com/apache/groovy/commit/b02d2f57c09ba6ba46553ed77aff191ec9ab274e#commitcomment-22218912
In src/main/org/apache/groovy/util/Maps.java:
In src/main/org/apache/groovy/util/Maps.java on line 11:
JDK9 added `Ma
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/543
---
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, or if the feature is enab
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117609738
--- Diff: src/main/groovy/lang/MetaClassImpl.java ---
@@ -1832,6 +1832,9 @@ public Object getProperty(Class sender, Object
object, String name, boolean use
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/543
GROOVY-8166: Repeated operations in AnnotationCollectorTransform and â¦
â¦Traits
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jwagenle
GitHub user ArsenBabakhanyan opened a pull request:
https://github.com/apache/groovy/pull/542
GROOVY-8193: setting sourcePosition on Property Expression
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ArsenBabakhanyan/groovy GROO
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117188820
--- Diff: src/main/groovy/lang/MetaClassImpl.java ---
@@ -1832,6 +1832,9 @@ public Object getProperty(Class sender, Object
object, String name, boolean use
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117188688
--- Diff: src/main/groovy/lang/MetaClassImpl.java ---
@@ -1832,6 +1832,9 @@ public Object getProperty(Class sender, Object
object, String name, boolean use
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117188226
--- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java ---
@@ -90,6 +92,12 @@ public CachedClass getDeclaringClass() {
public fi
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117188013
--- Diff: src/main/groovy/lang/MetaClassImpl.java ---
@@ -1832,6 +1832,9 @@ public Object getProperty(Class sender, Object
object, String name, boolean use
Github user blackdrag commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117177738
--- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java ---
@@ -90,6 +92,12 @@ public CachedClass getDeclaringClass() {
public fi
Github user blackdrag commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117177601
--- Diff: src/main/groovy/lang/MetaClassImpl.java ---
@@ -1832,6 +1832,9 @@ public Object getProperty(Class sender, Object
object, String name, boolean use
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117087531
--- Diff: src/main/groovy/lang/MetaClassImpl.java ---
@@ -1832,6 +1832,9 @@ public Object getProperty(Class sender, Object
object, String name, boolean use
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r117078128
--- Diff: src/main/groovy/lang/MetaClassImpl.java ---
@@ -1832,6 +1832,9 @@ public Object getProperty(Class sender, Object
object, String name, boolean
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/296
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/533
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/541
---
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, or if the feature is enab
GitHub user mnonnenmacher opened a pull request:
https://github.com/apache/groovy/pull/541
Improve naming and documentation for String.tokenize
Calling the delimiter argument token is confusing as according to the
StringTokenizer documentation delimiters are NOT treated as tokens
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/533#discussion_r116533820
--- Diff: src/main/org/apache/groovy/ast/tools/MethodNodeUtils.java ---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
Github user paulk-asert commented on a diff in the pull request:
https://github.com/apache/groovy/pull/533#discussion_r116460943
--- Diff: src/main/org/apache/groovy/ast/tools/MethodNodeUtils.java ---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user paulk-asert commented on a diff in the pull request:
https://github.com/apache/groovy/pull/533#discussion_r116460996
--- Diff: src/main/org/apache/groovy/ast/tools/MethodNodeUtils.java ---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377499
--- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java ---
@@ -124,6 +131,12 @@ public String getSignature() {
}
public
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377497
--- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java ---
@@ -90,6 +91,12 @@ public CachedClass getDeclaringClass() {
public fi
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377501
--- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java ---
@@ -324,6 +337,12 @@ else if (o2 instanceof CachedMethod)
}
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377495
--- Diff: src/main/org/codehaus/groovy/reflection/CachedField.java ---
@@ -65,6 +72,12 @@ public Object getProperty(final Object object) {
* @thro
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377482
--- Diff: src/main/org/codehaus/groovy/reflection/CachedField.java ---
@@ -51,6 +52,12 @@ public int getModifiers() {
*/
public Object ge
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377432
--- Diff:
src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Founda
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377435
--- Diff:
src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Founda
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377418
--- Diff:
src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Founda
Github user dpolivaev commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116377367
--- Diff:
src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Founda
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/540
GROOVY-7535: Groovy category throwing MissingMethodException and Missâ¦
â¦ingPropertyException when using multiple threads
You can merge this pull request into a Git repository by running:
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365638
--- Diff:
src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Fo
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365514
--- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java ---
@@ -324,6 +337,12 @@ else if (o2 instanceof CachedMethod)
}
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365223
--- Diff: src/main/org/codehaus/groovy/reflection/CachedField.java ---
@@ -65,6 +72,12 @@ public Object getProperty(final Object object) {
* @
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365151
--- Diff:
src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Fo
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365503
--- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java ---
@@ -124,6 +131,12 @@ public String getSignature() {
}
pu
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365354
--- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java ---
@@ -90,6 +91,12 @@ public CachedClass getDeclaringClass() {
publi
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365210
--- Diff: src/main/org/codehaus/groovy/reflection/CachedField.java ---
@@ -51,6 +52,12 @@ public int getModifiers() {
*/
public Objec
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365112
--- Diff:
src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Fo
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/532#discussion_r116365147
--- Diff:
src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java ---
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Fo
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/533#discussion_r116363909
--- Diff: src/main/org/apache/groovy/ast/tools/MethodNodeUtils.java ---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/533#discussion_r116363829
--- Diff: src/main/org/apache/groovy/ast/tools/MethodNodeUtils.java ---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache Software Foundation (AS
Github user jwagenleitner commented on a diff in the pull request:
https://github.com/apache/groovy/pull/533#discussion_r116363767
--- Diff: src/main/org/apache/groovy/ast/tools/ClassNodeUtils.java ---
@@ -0,0 +1,48 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user shils commented on a diff in the pull request:
https://github.com/apache/groovy/pull/539#discussion_r116149015
--- Diff:
src/test/org/codehaus/groovy/classgen/asm/sc/ArraysAndCollectionsStaticCompileTest.groovy
---
@@ -128,6 +128,18 @@ class ArraysAndCollectionsStaticC
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/537
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/539
---
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, or if the feature is enab
Github user paulk-asert commented on a diff in the pull request:
https://github.com/apache/groovy/pull/539#discussion_r115862393
--- Diff:
src/test/org/codehaus/groovy/classgen/asm/sc/ArraysAndCollectionsStaticCompileTest.groovy
---
@@ -128,6 +128,18 @@ class ArraysAndCollectionsS
GitHub user shils opened a pull request:
https://github.com/apache/groovy/pull/539
GROOVY-8074: Statically compiled dot property accesses instance propeâ¦
â¦rty instead of Map property
You can merge this pull request into a Git repository by running:
$ git pull https://github
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/536
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/535
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/530
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/538
---
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, or if the feature is enab
GitHub user armsargis opened a pull request:
https://github.com/apache/groovy/pull/538
GROOVY-8135: SecureASTCustomizer whitelist does not work
For arrays we should get componentType instead of type
You can merge this pull request into a Git repository by running:
$ git pull ht
GitHub user shils opened a pull request:
https://github.com/apache/groovy/pull/537
GROOVY-8157: Flow typing doesn't work with assignment to a parameter
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/shils/groovy GROOVY-8157
Alt
GitHub user shils opened a pull request:
https://github.com/apache/groovy/pull/536
Improve type inference/STC compatibility of Sql methods with closure
parameters
This PR adds @ClosureParams to closure parameters of the following methods
in Sql and DataSet:
* asList
* eachR
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/485
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/534
---
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, or if the feature is enab
GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/535
GROOVY-8176: tap - exception in phase 'instruction selection'
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/paulk-asert/groovy groovy8176
GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/534
GROOVY-8174: Groovy.Sql+Oracle parameter substitution problem
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/paulk-asert/groovy groovy8174
GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/533
GROOVY-7840: Verifier#makeDescriptorWithoutReturnType uses ClassNode#â¦
â¦toString with generics
You can merge this pull request into a Git repository by running:
$ git pull https://githu
GitHub user dpolivaev opened a pull request:
https://github.com/apache/groovy/pull/532
Prevent CachedField and CachedMethod from leaking access permissions â¦
â¦to scripts
https://issues.apache.org/jira/browse/GROOVY-8163
You can merge this pull request into a Git reposit
Github user jwagenleitner commented on the pull request:
https://github.com/apache/groovy/commit/0fb89906aa587920d11fae063bba1d1f8fe26254#commitcomment-21969276
I created [GROOVY-8171](https://issues.apache.org/jira/browse/GROOVY-8171)
because there is a slight difference in behavior
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/0fb89906aa587920d11fae063bba1d1f8fe26254#commitcomment-21966548
I run the test locally, parrot can handle it properly.
---
If your project is set up for it, you can reply to this email an
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/0fb89906aa587920d11fae063bba1d1f8fe26254#commitcomment-21966439
@danielsun1106 So is it a bug in the new groovy parser?
---
If your project is set up for it, you can reply to this emai
Github user jwagenleitner commented on the pull request:
https://github.com/apache/groovy/commit/0fb89906aa587920d11fae063bba1d1f8fe26254#commitcomment-21963417
Thanks, I had forgot to test with antlr4 enabled.
---
If your project is set up for it, you can reply to this email and hav
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/531
---
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, or if the feature is enab
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/531
fix spec test
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jwagenleitner/groovy dollarslashy
Alternatively you can review and apply th
Github user PascalSchumacher commented on the pull request:
https://github.com/apache/groovy/commit/0fb89906aa587920d11fae063bba1d1f8fe26254#commitcomment-21962912
test fails on travis, see:
https://travis-ci.org/apache/groovy/builds/227402483?utm_source=github_status&utm_medium=notif
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/530
Clarify documentation around indy
From a recent [user mailing list
question](http://mail-archives.apache.org/mod_mbox/groovy-users/201704.mbox/%3c44315a6eb40a3769a12a5bfe465da...@posteo.de%3E)
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/527
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/526
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/529
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/528
---
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, or if the feature is enab
GitHub user paulk-asert opened a pull request:
https://github.com/apache/groovy/pull/529
GROOVY-8127: Access to Trait$Trait$Helper#$self is forbidden
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/paulk-asert/groovy groovy8127
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/528
GROOVY-7579: Improve docs for invokeMethod
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jwagenleitner/groovy groovy7579
Alternatively
GitHub user PascalSchumacher opened a pull request:
https://github.com/apache/groovy/pull/527
remove skipping of tests for builds.apache.org as we no longer use it
even groovy decides to use builds.apache.org again, these workarounds may
not be necessary today
You can merge this pu
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/517
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/525
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/512
---
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, or if the feature is enab
GitHub user PascalSchumacher opened a pull request:
https://github.com/apache/groovy/pull/526
cleanup now that jdk7 is baseline
for master and the GROOVY_2_5_X branch
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/PascalSchumache
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/521
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/524
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/523
---
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, or if the feature is enab
GitHub user chetankm86 opened a pull request:
https://github.com/apache/groovy/pull/525
Groovy 2 4 x
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/groovy GROOVY_2_4_X
Alternatively you can review and apply these changes
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/524
GROOVY-8156: Compile error when ListenerList annotation exists
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jwagenleitner/groovy groovy
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/518
---
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, or if the feature is enab
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/522
---
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, or if the feature is enab
Github user blackdrag commented on a diff in the pull request:
https://github.com/apache/groovy/pull/523#discussion_r111070966
--- Diff: src/main/org/codehaus/groovy/vmplugin/VMPluginFactory.java ---
@@ -26,39 +28,10 @@
*/
public class VMPluginFactory {
-pri
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/515
---
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, or if the feature is enab
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/523
cleanup now that jdk7 is baseline
Avoids reflective checks/constructions where it is no longer needed since 7
is now baseline in 2_5_X and master.
You can merge this pull request into a Git r
Github user danielsun1106 commented on the pull request:
https://github.com/apache/groovy/commit/ce12609009c6572247d8d194047aac6812753b92#commitcomment-21710945
In src/main/groovy/lang/Groovydoc.java:
In src/main/groovy/lang/Groovydoc.java on line 1:
OK. I will add it later :)
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/522
GROOVY-8144: Invoking a public method declared in a non-public class â¦
â¦result in a IllegalAccessError
Commit 1a4c9918a4f12e64 introduced the DecompiledClassNode as part of
ena
Github user paulk-asert commented on the pull request:
https://github.com/apache/groovy/commit/ce12609009c6572247d8d194047aac6812753b92#commitcomment-21709694
In src/main/groovy/lang/Groovydoc.java:
In src/main/groovy/lang/Groovydoc.java on line 1:
Needs the normal Apache heade
GitHub user dkhamitov opened a pull request:
https://github.com/apache/groovy/pull/521
GROOVY-7879 Groovy calls wrong method if there is a static method on â¦
Hi guys, can we consider a small kind of tactical fix (imho) to GROOVY-7879
or is it a bad idea/solution? It fixes the situ
Github user asfgit closed the pull request at:
https://github.com/apache/groovy/pull/520
---
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, or if the feature is enab
GitHub user jwagenleitner opened a pull request:
https://github.com/apache/groovy/pull/520
GROOVY-8140: Invoke method not returning MOP super method if isCallToâ¦
â¦Super
See [dev mailing list
discussion](http://mail-archives.apache.org/mod_mbox/groovy-dev/201703.mbox/%3C
901 - 1000 of 2069 matches
Mail list logo
