RFR: JDK-8300857: State return value for Types.asElement(NoType) explicitly [v2]
Jonathan Gibbons
jjg at openjdk.org
Wed Jan 25 23:43:22 UTC 2023
On Wed, 25 Jan 2023 03:56:30 GMT, Joe Darcy <darcy at openjdk.org> wrote:
>> Just sending out the proposed API changes for now, will add tests later.
>>
>> Please also review the corresponding CSR [JDK-8300951](https://bugs.openjdk.org/browse/JDK-8300951).
>
> Joe Darcy has updated the pull request incrementally with two additional commits since the last revision:
>
> - Applease jcheck.
> - Refine spec, add regression test.
approved with quibbles for optional update
I like the presence of the new test ;-)
test/langtools/tools/javac/processing/model/util/types/TestAsElement.java line 78:
> 76:
> 77:
> 78:
excess blank lines?
test/langtools/tools/javac/processing/model/util/types/TestAsElement.java line 87:
> 85:
> 86: private void testRoundTripCases() {
> 87: expectRoundTrip(eltUtils.getPackageElement("java.lang"));
Maybe add a type element as well as the package element and module element
-------------
Marked as reviewed by jjg (Reviewer).
PR: https://git.openjdk.org/jdk/pull/12159
More information about the compiler-dev
mailing list