RFR: JDK-8281169: Expand discussion of elements and types [v3]
Pavel Rappo
prappo at openjdk.org
Wed Aug 16 09:07:12 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
Changes requested by prappo (Reviewer).
src/java.compiler/share/classes/javax/lang/model/element/package-info.java line 113:
> 111: *
> 112: * @see javax.lang.model.util.Elements
> 113: * @see javax.lang.model##elementsAndTypes Element and Types
While it's a nice way to emphasize that one element maps to many types, I think it's a typo: the heading is "Elements and Types".
Suggestion:
* @see javax.lang.model##elementsAndTypes Elements and Types
src/java.compiler/share/classes/javax/lang/model/type/package-info.java line 37:
> 35: *
> 36: * @see javax.lang.model.util.Types
> 37: * @see javax.lang.model##elementsAndTypes Element and Types
Suggestion:
* @see javax.lang.model##elementsAndTypes Elements and Types
-------------
PR Review: https://git.openjdk.org/jdk/pull/15097#pullrequestreview-1580099834
PR Review Comment: https://git.openjdk.org/jdk/pull/15097#discussion_r1295573445
PR Review Comment: https://git.openjdk.org/jdk/pull/15097#discussion_r1295578497
More information about the compiler-dev
mailing list