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