RFR: 8257037: No javac warning when calling deprecated constructor with diamond [v2]
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 written by Jonathan Gibbons in JDK-8231622.
> 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.
More information about the compiler-dev