Re: RFR: 8345074: java.net.InterfaceAddress constructor could be made private [v2]

2024-12-03 Thread Eirik Bjørsnøs
On Sun, 1 Dec 2024 05:50:47 GMT, Eirik Bjørsnøs wrote: >> Please review this PR which suggests to make the constructor of >> `java.net.InterfaceAddress` private. >> >> This constructor is only accessed by native code via the `NetworkInterface` >> class. Making it private would express the inte

Re: RFR: 8345074: java.net.InterfaceAddress constructor could be made private [v2]

2024-12-01 Thread Jaikiran Pai
On Sun, 1 Dec 2024 05:50:47 GMT, Eirik Bjørsnøs wrote: >> Please review this PR which suggests to make the constructor of >> `java.net.InterfaceAddress` private. >> >> This constructor is only accessed by native code via the `NetworkInterface` >> class. Making it private would express the inte

Re: RFR: 8345074: java.net.InterfaceAddress constructor could be made private [v2]

2024-12-01 Thread Jaikiran Pai
On Sun, 1 Dec 2024 05:50:47 GMT, Eirik Bjørsnøs wrote: >> Please review this PR which suggests to make the constructor of >> `java.net.InterfaceAddress` private. >> >> This constructor is only accessed by native code via the `NetworkInterface` >> class. Making it private would express the inte

Re: RFR: 8345074: java.net.InterfaceAddress constructor could be made private [v2]

2024-11-30 Thread Eirik Bjørsnøs
> Please review this PR which suggests to make the constructor of > `java.net.InterfaceAddress` private. > > This constructor is only accessed by native code via the `NetworkInterface` > class. Making it private would express the intent of this class as being > non-subclassable. > > The commen

Re: RFR: 8345074: java.net.InterfaceAddress constructor could be made private [v2]

2024-11-30 Thread Eirik Bjørsnøs
On Sun, 1 Dec 2024 01:32:31 GMT, Chen Liang wrote: > We should word like "this constructor is called via JNI in > NetworkInterface.c" so we know what care we need to take when we update this > constructor. Thanks! Yes, simpler to just say where the constructor is called from. It folllows from

Re: RFR: 8345074: java.net.InterfaceAddress constructor could be made private

2024-11-30 Thread Chen Liang
On Sat, 30 Nov 2024 18:18:29 GMT, Eirik Bjørsnøs wrote: > Please review this PR which suggests to make the constructor of > `java.net.InterfaceAddress` private. > > This constructor is only accessed by native code via the `NetworkInterface` > class. Making it private would express the intent o

Re: RFR: 8345074: java.net.InterfaceAddress constructor could be made private

2024-11-30 Thread Eirik Bjørsnøs
On Sat, 30 Nov 2024 18:18:29 GMT, Eirik Bjørsnøs wrote: > Please review this PR which suggests to make the constructor of > `java.net.InterfaceAddress` private. > > This constructor is only accessed by native code via the `NetworkInterface` > class. Making it private would express the intent o

RFR: 8345074: java.net.InterfaceAddress constructor could be made private

2024-11-30 Thread Eirik Bjørsnøs
Please review this PR which suggests to make the constructor of `java.net.InterfaceAddress` private. This constructor is only accessed by native code via the `NetworkInterface` class. Making it private would express the intent of this class as being non-subclassable. The comment of the constru