RFR: 8254996: make jdk.net.UnixDomainPrincipal a record class
Patrick Concannon
pconcannon at openjdk.java.net
Mon Dec 7 15:37:20 UTC 2020
Hi,
Could someone please review my code for JDK-8254996: 'make jdk.net.UnixDomainPrincipal a record class'?
`jdk.net.UnixDomainPrincipal` is a simple immutable data class that requires boilerplate methods for access. However, these methods and fields are susceptible to trivial mistakes and add to the verbosity of the class.
This fix replaces the data class with a record class, a new type in JDK 16, making the code more concise and easier to maintain. This code is binary compatible with the original. For the generated API doc change, please refer to the specdiff attached to the CSR.
CSR : https://bugs.openjdk.java.net/browse/JDK-8257823
Kind regards,
Patrick
-------------
Commit messages:
- 8254996: make jdk.net.UnixDomainPrincipal a record class
Changes: https://git.openjdk.java.net/jdk/pull/1668/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1668&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8254996
Stats: 52 lines in 1 file changed: 3 ins; 45 del; 4 mod
Patch: https://git.openjdk.java.net/jdk/pull/1668.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1668/head:pull/1668
PR: https://git.openjdk.java.net/jdk/pull/1668
More information about the net-dev
mailing list