Hello Xuelei, I have renamed inputBuffer to recordBody. Also, as you suggested, recordBody is not removed but used for multiple records. So, it should be better for performance.
JDK15 webrev: http://cr.openjdk.java.net/~dcherepanov/8239788/webrev.v5/ Regards Alexey > On 4 Mar 2020, at 21:23, Xuelei Fan <xuelei....@oracle.com> wrote: > > > http://cr.openjdk.java.net/~bae/8239788/webrev.v4/ > > SSLSocketInputRecord: > 54 // Cache for incomplete input record. > 55 private ByteBuffer inputBuffer = null; > This variable is used for record body, I may use a instinctive name, for > example recordBody. > > Otherwise, looks good to me. > > I think, for performance, it may be possible to reuse this buffer for > multiple records. I'd appreciate if you want to make an improvement in this > update as well. > > Thanks, > Xuelei >
signature.asc
Description: Message signed with OpenPGP
