On Wed, 3 Nov 2021 14:20:52 GMT, Hannes Wallnöfer <hann...@openjdk.org> wrote:

>> Jonathan Gibbons has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   update to show fields of serializable records in the serializable form
>
> test/langtools/jdk/javadoc/doclet/testRecordTypes/TestRecordTypes.java line 
> 573:
> 
>> 571:                      */
>> 572:                     public record Point(int x, int y) implements 
>> java.io.Serializable { }""");
>> 573: 
> 
> I just noted the static files in the `examples` directory already contain a 
> `public record SerializablePoint(int x, int y) implements Serializable { }`, 
> but it seems the output is never checked. Not an error, just a bit of tiny 
> bit of wasted resources.

Yeah. IIRC, the examples were specifically taken from public documentation so 
have a separate role.  I think the test is clearer as is.

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

PR: https://git.openjdk.java.net/jdk/pull/6071

Reply via email to