RFR: JDK-8281169: Expand discussion of elements and types [v3]
Joe Darcy
darcy at openjdk.org
Wed Aug 16 16:38:16 UTC 2023
On Wed, 16 Aug 2023 10:27:40 GMT, Pavel Rappo <prappo at openjdk.org> wrote:
>> Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
>>
>> - Implement review feedback.
>> - Merge branch 'master' into JDK-8281169
>> - Iteration of spec update.
>> - Merge branch 'master' into JDK-8281169
>> - Checkpoint intermediate state.
>> - Merge branch 'master' into JDK-8281169
>> - JDK-8281169: Expand discussion of elements and types
>
> src/java.compiler/share/classes/javax/lang/model/package-info.java line 81:
>
>> 79: * a {@code @DefinitelyNotNull} type annotation while the type of
>> 80: * the parameter is a {@code String} annotated with a {@code
>> 81: * @MightBeNull} type annotation. In a reflective API, since the set
>
>> In a reflective API,
>
> Nit: re-reading that bit made me pause for a moment and doubt that javax.lang.model is a reflective API, which I hope it is. We could help the reader by rephrasing that more assertively as "In a reflective API, such as this one," or dropping "In a reflective API" altogether.
Will change the opening sentence to:
"Types and hierarchies of packages comprising a Java language model, a reflective API that models the declarations and types of the Java programming language."
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15097#discussion_r1296170187
More information about the compiler-dev
mailing list