RFR: JDK-8281169: Expand discussion of elements and types [v3]

Pavel Rappo prappo at openjdk.org
Wed Aug 16 10:31:11 UTC 2023


On Wed, 16 Aug 2023 04:59:34 GMT, Joe Darcy <darcy at openjdk.org> wrote:

>> The goal is this change is to convey to the reader an accurate "vibe" for the dichotomy between elements and type in the javax.lang.model API rather than a primer on category theory (projection-embedding pairs, etc.).
>
> 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.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15097#discussion_r1295695947


More information about the compiler-dev mailing list