Re: RFR: 8325162: Remove duplicate GCMParameters class [v2]

2024-02-15 Thread Daniel Jeliński
On Tue, 13 Feb 2024 20:58:17 GMT, Daniel Jeliński wrote: >> Currently we have 2 GCMParameters classes in java.base, one in >> sun.security.util package, the other in com.sun.crypto.provider. >> >> This PR merges the changes from the sun.security.util copy into >> com.sun.crypto.provider, and r

Re: RFR: 8325162: Remove duplicate GCMParameters class [v2]

2024-02-14 Thread Valerie Peng
On Tue, 13 Feb 2024 20:58:17 GMT, Daniel Jeliński wrote: >> Currently we have 2 GCMParameters classes in java.base, one in >> sun.security.util package, the other in com.sun.crypto.provider. >> >> This PR merges the changes from the sun.security.util copy into >> com.sun.crypto.provider, and r

Re: RFR: 8325162: Remove duplicate GCMParameters class [v2]

2024-02-14 Thread Valerie Peng
On Tue, 13 Feb 2024 20:58:17 GMT, Daniel Jeliński wrote: >> Currently we have 2 GCMParameters classes in java.base, one in >> sun.security.util package, the other in com.sun.crypto.provider. >> >> This PR merges the changes from the sun.security.util copy into >> com.sun.crypto.provider, and r

Re: RFR: 8325162: Remove duplicate GCMParameters class [v2]

2024-02-13 Thread Daniel Jeliński
On Wed, 14 Feb 2024 02:50:21 GMT, John Jiang wrote: > `com.sun.crypto.provider.GCMParameters` implies that class is a part of > `SunJCE` provider, however `sun.security.util.GCMParameters` looks not. True. Also, the only other Parameters class in `sun.security.util` is `ECParameters`, which wa

Re: RFR: 8325162: Remove duplicate GCMParameters class [v2]

2024-02-13 Thread John Jiang
On Tue, 13 Feb 2024 20:58:17 GMT, Daniel Jeliński wrote: >> Currently we have 2 GCMParameters classes in java.base, one in >> sun.security.util package, the other in com.sun.crypto.provider. >> >> This PR merges the changes from the sun.security.util copy into >> com.sun.crypto.provider, and r

Re: RFR: 8325162: Remove duplicate GCMParameters class [v2]

2024-02-13 Thread Weijun Wang
On Tue, 13 Feb 2024 20:58:17 GMT, Daniel Jeliński wrote: >> Currently we have 2 GCMParameters classes in java.base, one in >> sun.security.util package, the other in com.sun.crypto.provider. >> >> This PR merges the changes from the sun.security.util copy into >> com.sun.crypto.provider, and r

Re: RFR: 8325162: Remove duplicate GCMParameters class [v2]

2024-02-13 Thread Daniel Jeliński
> Currently we have 2 GCMParameters classes in java.base, one in > sun.security.util package, the other in com.sun.crypto.provider. > > This PR merges the changes from the sun.security.util copy into > com.sun.crypto.provider, and removes the sun.security.util class. > > As far as I could tell,

Re: RFR: 8325162: Remove duplicate GCMParameters class

2024-02-13 Thread Weijun Wang
On Mon, 12 Feb 2024 08:34:24 GMT, Daniel Jeliński wrote: > Currently we have 2 GCMParameters classes in java.base, one in > sun.security.util package, the other in com.sun.crypto.provider. > > This PR merges the changes from the sun.security.util copy into > com.sun.crypto.provider, and remove

Re: RFR: 8325162: Remove duplicate GCMParameters class

2024-02-13 Thread Daniel Jeliński
On Mon, 12 Feb 2024 08:34:24 GMT, Daniel Jeliński wrote: > Currently we have 2 GCMParameters classes in java.base, one in > sun.security.util package, the other in com.sun.crypto.provider. > > This PR merges the changes from the sun.security.util copy into > com.sun.crypto.provider, and remove

Re: RFR: 8325162: Remove duplicate GCMParameters class

2024-02-13 Thread Weijun Wang
On Mon, 12 Feb 2024 08:34:24 GMT, Daniel Jeliński wrote: > Currently we have 2 GCMParameters classes in java.base, one in > sun.security.util package, the other in com.sun.crypto.provider. > > This PR merges the changes from the sun.security.util copy into > com.sun.crypto.provider, and remove

Re: RFR: 8325162: Remove duplicate GCMParameters class

2024-02-13 Thread Daniel Jeliński
On Mon, 12 Feb 2024 08:34:24 GMT, Daniel Jeliński wrote: > Currently we have 2 GCMParameters classes in java.base, one in > sun.security.util package, the other in com.sun.crypto.provider. > > This PR merges the changes from the sun.security.util copy into > com.sun.crypto.provider, and remove

Re: RFR: 8325162: Remove duplicate GCMParameters class

2024-02-13 Thread Weijun Wang
On Mon, 12 Feb 2024 08:34:24 GMT, Daniel Jeliński wrote: > Currently we have 2 GCMParameters classes in java.base, one in > sun.security.util package, the other in com.sun.crypto.provider. > > This PR merges the changes from the sun.security.util copy into > com.sun.crypto.provider, and remove

Re: RFR: 8325162: Remove duplicate GCMParameters class

2024-02-13 Thread Daniel Jeliński
On Mon, 12 Feb 2024 08:34:24 GMT, Daniel Jeliński wrote: > Currently we have 2 GCMParameters classes in java.base, one in > sun.security.util package, the other in com.sun.crypto.provider. > > This PR merges the changes from the sun.security.util copy into > com.sun.crypto.provider, and remove

Re: RFR: 8325162: Remove duplicate GCMParameters class

2024-02-13 Thread Weijun Wang
On Mon, 12 Feb 2024 08:34:24 GMT, Daniel Jeliński wrote: > Currently we have 2 GCMParameters classes in java.base, one in > sun.security.util package, the other in com.sun.crypto.provider. > > This PR merges the changes from the sun.security.util copy into > com.sun.crypto.provider, and remove

RFR: 8325162: Remove duplicate GCMParameters class

2024-02-12 Thread Daniel Jeliński
Currently we have 2 GCMParameters classes in java.base, one in sun.security.util package, the other in com.sun.crypto.provider. This PR merges the changes from the sun.security.util copy into com.sun.crypto.provider, and removes the sun.security.util class. As far as I could tell, the sun.secur