Leonard Brünings created GROOVY-9699:
----------------------------------------
Summary: @CompileStatic does not use bridge methods for keyed map
access
Key: GROOVY-9699
URL: https://issues.apache.org/jira/browse/GROOVY-9699
Project: Groovy
Issue Type: Bug
Components: Static compilation
Reporter: Leonard Brünings
Fix For: 2.5.13
Extracted from GROOVY-9695:
{code:groovy}
import java.util.regex.Pattern
import org.junit.Test
import groovy.transform.CompileStatic
class Example {
@Test
void testBaseListClass() {
new Base().checkList()
}
@Test
void testChildListClass() {
new Child().checkList()
}
@Test
void testBaseMapClass() {
new Base().checkMap()
}
@Test
void testChildMapClass() {
new Child().checkMap()
}
}
@CompileStatic
class Base {
private static final Pattern PATTERNS = ~/.*Test/
def checkList() {
List failures = []
def closure = {
failures << PATTERNS.pattern()
}
closure()
}
def checkMap() {
Map failures = [:]
def closure = {
failures[PATTERNS.pattern()] = 1
}
closure()
}
}
class Child extends Base {
}
{code}
For the list it works fine (Procyon Decompiler)
{code:java}
public Object doCall(final Object it) {
return
DefaultGroovyMethods.leftShift((List<String>)ScriptBytecodeAdapter.castToType(this.failures.get(),
List.class), Base.pfaccess$0((Base)null).pattern());
}
{code}
For the map it doesn't use the bridge method (Procyon Decompiler)
{code:java}
public Object doCall(final Object it) {
final int n = 1;
DefaultGroovyMethods.putAt((Map<String,
Integer>)ScriptBytecodeAdapter.castToType(this.failures.get(), Map.class),
((Pattern)this.getProperty("PATTERNS")).pattern(), n);
return n;
}
{code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
