Re: RFR: 8296612: CertAttrSet is useless [v2]

2022-11-10 Thread Sean Mullan
On Thu, 10 Nov 2022 20:51:40 GMT, Weijun Wang wrote: >> - All `CertAttrSet` child classes now implement `DerEncoder` >> - `DerEncoder::derEncode` is renamed to `DerEncoder::encode`, and duplicate >> methods are removed >> - `s.s.x.Extension` now implements `DerEncoder`, so its child classes no

Re: RFR: 8296612: CertAttrSet is useless [v2]

2022-11-10 Thread Weijun Wang
> - All `CertAttrSet` child classes now implement `DerEncoder` > - `DerEncoder::derEncode` is renamed to `DerEncoder::encode`, and duplicate > methods are removed > - `s.s.x.Extension` now implements `DerEncoder`, so its child classes no need > to implement it > - `X509CertImpl::encode(OutputStre

Re: RFR: 8296612: CertAttrSet is useless

2022-11-10 Thread Weijun Wang
On Thu, 10 Nov 2022 20:24:31 GMT, Sean Mullan wrote: >> - All `CertAttrSet` child classes now implement `DerEncoder` >> - `DerEncoder::derEncode` is renamed to `DerEncoder::encode`, and duplicate >> methods are removed >> - `s.s.x.Extension` now implements `DerEncoder`, so its child classes no

Re: RFR: 8296612: CertAttrSet is useless

2022-11-10 Thread Sean Mullan
On Tue, 8 Nov 2022 22:42:05 GMT, Weijun Wang wrote: > - All `CertAttrSet` child classes now implement `DerEncoder` > - `DerEncoder::derEncode` is renamed to `DerEncoder::encode`, and duplicate > methods are removed > - `s.s.x.Extension` now implements `DerEncoder`, so its child classes no need

RFR: 8296612: CertAttrSet is useless

2022-11-08 Thread Weijun Wang
- All `CertAttrSet` child classes now implement `DerEncoder` - `DerEncoder::derEncode` is renamed to `DerEncoder::encode`, and duplicate methods are removed - `s.s.x.Extension` now implements `DerEncoder`, so its child classes no need to implement it - `X509CertImpl::encode(OutputStream out)` rem