RFR: JDK-8175386: Clarify exception behavior of Types utility methods [v4]
Jonathan Gibbons
jjg at openjdk.org
Fri Jan 26 21:24:26 UTC 2024
On Fri, 26 Jan 2024 21:02:53 GMT, Joe Darcy <darcy at openjdk.org> wrote:
>> Incremental improvements and clarifications in the specification of Types.
>
> 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 four additional commits since the last revision:
>
> - Merge branch 'master' into JDK-8175386
> - Implement review feedback.
> - Fix typo found in code review.
> - JDK-8175386: Clarify exception behavior of Types utility methods
Marked as reviewed by jjg (Reviewer).
src/java.compiler/share/classes/javax/lang/model/util/Types.java line 48:
> 46: * methods. One or more out of scope arguments will typically result
> 47: * in a method throwing an {@link IllegalArgumentException
> 48: * IllegalArgumentException}.
Nit: do you need the second argument? It should be the default string.
-------------
PR Review: https://git.openjdk.org/jdk/pull/17523#pullrequestreview-1846568129
PR Review Comment: https://git.openjdk.org/jdk/pull/17523#discussion_r1468158159
More information about the compiler-dev
mailing list