RFR: 8258897: wrong translation of capturing local classes inside nested lambdas [v3]

Guoxiong Li github.com+13688759+lgxbslgx at openjdk.java.net
Mon Jan 25 13:26:48 UTC 2021


On Mon, 25 Jan 2021 12:15:36 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

>> Jia Yanwei has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - fix 8258897: wrong translation of capturing local classes inside nested lambdas
>>  - rename test case directory name
>>  - revert the experimental fix in 9dec0d2
>
> test/langtools/tools/javac/lambda/8258897/CaptureVariables.java line 3:
> 
>> 1: import java.util.function.Supplier;
>> 2: 
>> 3: /**
> 
> It would be nice to tweak the test to so that:
> -the comment with `@test` is first, and imports follow
> -`@test` is augmented with `/nodynamiccopyright/`
> -`@bug` is added
> -`@summary` is added
> 
> (applies to both tests)

And the action tag `@run main CaptureVariables` should be added.

@hltj Some documents are helpful to you: 
[Regression Test Harness for the JDK: jtreg](http://openjdk.java.net/jtreg/index.html)
[The JDK Test Framework: Tag Language Specification](http://openjdk.java.net/jtreg/tag-spec.html)

-------------

PR: https://git.openjdk.java.net/jdk/pull/1479


More information about the compiler-dev mailing list