RFR: 8257037: No javac warning when calling deprecated constructor with diamond [v2]

Guoxiong Li github.com+13688759+lgxbslgx at openjdk.java.net
Mon Dec 7 12:00:29 UTC 2020


On Mon, 7 Dec 2020 10:40:05 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> I revise the test code based on the comment[1] written by Jonathan Gibbons in JDK-8231622[2].
>> 
>>> For tests that use /ref=file -XDrawDiagnostics the conventions are:
>>>
>>>   1. Omit the complete legal header, including the copyright and license
>>>   2. After `@test` add the text `/nodynamiccopyright/`
>>>
>>> #1 protects the file against any future changes in the length of the legal header, that might affect line numbers, and #2 is for use by automated scripts that may check for the presence of the legal header.
>> 
>> [1] https://mail.openjdk.java.net/pipermail/compiler-dev/2020-December/015542.html
>> [2] https://github.com/openjdk/jdk/pull/1626
>
> The extensive testing revealed at least 10 cases where the compiler is issuing a new, resolution-related error when it wasn't doing so before - (the errors seem to have to do with `abstract` methods). As a result I think it's best to go with your first conservative approach, to avoid troubles. I'll try to investigate further and come up with a reduced test case for the failing condition - but that doesn't have to hold up this fix. Thanks!

It is a good news that the trouble is found before the integration instead of after the release of JDK16.
I revise the code to previous version.
Thank your for the review.

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

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


More information about the compiler-dev mailing list