RFR: 8320575: generic type information lost on mandated parameters of record's compact constructors [v7]
Joe Darcy
darcy at openjdk.org
Thu May 23 01:01:09 UTC 2024
On Tue, 12 Dec 2023 22:21:29 GMT, Joe Darcy <darcy at openjdk.org> wrote:
>> Vicente Romero has updated the pull request incrementally with one additional commit since the last revision:
>>
>> fixing comment
>
> As the core reflection code will encounter record classes compiled before and after the javac code generation change, if the old behavior can be triggered in javac using `--release $OLD`/`--source $OLD`, that would be helpful to include as part of the testing.
> @jddarcy I have uploaded a new commit addressing your comments, thanks
Thanks @vicente-romero-oracle .
As a general comment, please update all the links to "mandated" so that the text "implicitly declared" get linked to the MANDATED enum constant. The term "mandated" may not familiar to core reflection readers, but is defined elsewhere in the platform so I don't think having the term "mandated" appears in the text in core reflection is necessarily helpful to readers. Concretely, replace javadoc like
`
implicitly ({@linkplain java.compiler/javax.lang.model.util.Elements.Origin#MANDATED mandated}) declared`
with
{@linkplain java.compiler/javax.lang.model.util.Elements.Origin#MANDATED implicitly declared}
Also, before this PR is pushed, please reflow the updated paragraphs so avoid introducing long lines of text.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17070#issuecomment-2126006488
More information about the core-libs-dev
mailing list