RFR: 8257037: No javac warning when calling deprecated constructor with diamond [v2]
Guoxiong Li
github.com+13688759+lgxbslgx at openjdk.java.net
Fri Dec 4 19:14:17 UTC 2020
On Wed, 2 Dec 2020 12:15:08 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>>> Is the code `newConstr.setAttributes(sym);` necessary in `Resolve.findDiamond`?
>>
>> Sorry, that was a leftover from a previous experiment. That's not necessary. We'll run some more tests on our side and I'll let you know.
>
>> @mcimadamore Should I add you as a contributor of this patch? Does it need another reviewer to review if I add you as a contributor?
>
> As you prefer - I don't mind either way. I don't think adding me as a contributor should prevent the integration, or me being able to sponsor, but I'm not 100% sure.
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
-------------
PR: https://git.openjdk.java.net/jdk/pull/1490
More information about the compiler-dev
mailing list