groovy-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pa...@apache.org
Subject groovy git commit: GROOVY-8871: Long Constants Defined in Groovy 2.5.3 Under OpenJDK 11 Cause an IllegalAccessError
Date Thu, 08 Nov 2018 10:32:38 GMT
Repository: groovy
Updated Branches:
  refs/heads/master da64d1b7f -> 7a41f0efd


GROOVY-8871: Long Constants Defined in Groovy 2.5.3 Under OpenJDK 11 Cause an IllegalAccessError


Project: http://git-wip-us.apache.org/repos/asf/groovy/repo
Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/7a41f0ef
Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/7a41f0ef
Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/7a41f0ef

Branch: refs/heads/master
Commit: 7a41f0efd6426bf71386c7c89a1a094f2ff517fb
Parents: da64d1b
Author: Paul King <paulk@asert.com.au>
Authored: Thu Nov 8 20:32:28 2018 +1000
Committer: Paul King <paulk@asert.com.au>
Committed: Thu Nov 8 20:32:28 2018 +1000

----------------------------------------------------------------------
 src/main/java/org/codehaus/groovy/control/OptimizerVisitor.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/groovy/blob/7a41f0ef/src/main/java/org/codehaus/groovy/control/OptimizerVisitor.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/codehaus/groovy/control/OptimizerVisitor.java b/src/main/java/org/codehaus/groovy/control/OptimizerVisitor.java
index 4c7230e..c9313dd 100644
--- a/src/main/java/org/codehaus/groovy/control/OptimizerVisitor.java
+++ b/src/main/java/org/codehaus/groovy/control/OptimizerVisitor.java
@@ -113,8 +113,9 @@ public class OptimizerVisitor extends ClassCodeExpressionTransformer {
         do {
             name = "$const$" + index++;
         } while (currentClass.getDeclaredField(name) != null);
+        // TODO consider moving initcode to <clinit> and remaking field final
         field = new FieldNode(name,
-                Opcodes.ACC_PRIVATE | Opcodes.ACC_STATIC | Opcodes.ACC_SYNTHETIC | Opcodes.ACC_FINAL,
+                Opcodes.ACC_PRIVATE | Opcodes.ACC_STATIC | Opcodes.ACC_SYNTHETIC,
                 constantExpression.getType(),
                 currentClass,
                 constantExpression);


Mime
View raw message