Re: RFR: 8309867: redundant class field RSAPadding.md

2023-06-12 Thread Valerie Peng
On Mon, 12 Jun 2023 16:39:33 GMT, Xue-Lei Andrew Fan wrote: > Hi, > > May I get this simple update reviewed? > > The class field RSAPadding.md can be converted to a local variable of the > constructor, and save the class footprint. > > Thanks, > Xuelei Marked as reviewed by valeriep (Reviewe

Re: RFR: 8309867: redundant class field RSAPadding.md

2023-06-12 Thread Xue-Lei Andrew Fan
On Mon, 12 Jun 2023 17:53:42 GMT, Hai-May Chao wrote: >> Hi, >> >> May I get this simple update reviewed? >> >> The class field RSAPadding.md can be converted to a local variable of the >> constructor, and save the class footprint. >> >> Thanks, >> Xuelei > > LGTM Thank you @haimaychao and @

Re: RFR: 8309867: redundant class field RSAPadding.md

2023-06-12 Thread Sean Mullan
On Mon, 12 Jun 2023 16:39:33 GMT, Xue-Lei Andrew Fan wrote: > Hi, > > May I get this simple update reviewed? > > The class field RSAPadding.md can be converted to a local variable of the > constructor, and save the class footprint. > > Thanks, > Xuelei Looks good. I'd be more inclined to cal

Re: RFR: 8309867: redundant class field RSAPadding.md

2023-06-12 Thread Weijun Wang
On Mon, 12 Jun 2023 16:39:33 GMT, Xue-Lei Andrew Fan wrote: > Hi, > > May I get this simple update reviewed? > > The class field RSAPadding.md can be converted to a local variable of the > constructor, and save the class footprint. > > Thanks, > Xuelei Marked as reviewed by weijun (Reviewer)

Re: RFR: 8309867: redundant class field RSAPadding.md

2023-06-12 Thread Hai-May Chao
On Mon, 12 Jun 2023 16:39:33 GMT, Xue-Lei Andrew Fan wrote: > Hi, > > May I get this simple update reviewed? > > The class field RSAPadding.md can be converted to a local variable of the > constructor, and save the class footprint. > > Thanks, > Xuelei Marked as reviewed by hchao (Committer)