RFR: JEP 359: Records (Preview) (full code)

Chris Hegarty chris.hegarty at oracle.com
Tue Dec 3 12:38:05 UTC 2019


Hi Joe,

> On 3 Dec 2019, at 05:36, Joe Darcy <joe.darcy at oracle.com> wrote:
> 
> ...
> 
> As a non-essential cleanup, in src/java.base/share/classes/java/io/ObjectOutputStream.java:
> 
> 1444             final boolean isRecord = isRecord(obj.getClass()) ? true : false;
> 1445             if (isRecord) {
> 1446                 writeRecordData(obj,desc);
> 1447             } else if (desc.isExternalizable() && !desc.isProxy()) {
> 
> would be clearer as
> 
> 1445             if (isRecord(obj.getClass())) {
> 1446                 writeRecordData(obj,desc);
> 1447             } else if (desc.isExternalizable() && !desc.isProxy()) {

Remi brought up similar on amber-dev ( and the implement has been updated ).

https://mail.openjdk.java.net/pipermail/amber-dev/2019-December/005334.html

-Chris.


More information about the compiler-dev mailing list