> Hi,
> 
> I need a review of this rather large change to GCM.  GCM will no longer use 
> CipherCore, and AESCrypt  to handle it's buffers and other objects.  It is 
> also a major code redesign limits the amount of data copies and make some 
> performance-based decisions.
> 
> Thanks
> 
> Tony

Anthony Scarpino has updated the pull request incrementally with one additional 
commit since the last revision:

  Remove GCTR reset() calls because GCTR is released after the operation
  some variable name consistency
  other small cleanup

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4072/files
  - new: https://git.openjdk.java.net/jdk/pull/4072/files/ccb7b357..d230e665

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4072&range=06
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4072&range=05-06

  Stats: 95 lines in 3 files changed: 19 ins; 22 del; 54 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4072.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4072/head:pull/4072

PR: https://git.openjdk.java.net/jdk/pull/4072

Reply via email to