On Tue, 29 Nov 2022 17:17:12 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
>> This PR proposes a variety of modernisations to the `jdk.sctp` module. >> >> During the fix of https://bugs.openjdk.org/browse/JDK-8296024, several >> improvement areas were identified including: >> >> * Replacing duplicate code segments >> * Making certain fields final >> * Using enhanced switch >> * Using records >> * Fixing typos >> * Marking fields participating in serialisation with `@Serial` >> * Modernizing toString() implementations >> * Using pattern matching >> * Using diamond operators > > src/jdk.sctp/unix/classes/sun/nio/ch/sctp/SctpMultiChannelImpl.java line 33: > >> 31: import java.io.FileDescriptor; >> 32: import java.io.IOException; >> 33: import java.util.*; > > It would be better to avoid wildcards in regular imports. Agreed. I would have to find the settings in my IDE to prevent this automatic "feature". ------------- PR: https://git.openjdk.org/jdk/pull/11418