> Can someone help review this fix? Changed the required-mechanism check by 
> checking if the particular mechanism is inside the list of enabled supported 
> mechanisms. This should be more reliable than calling C_GetMechanismInfo(..) 
> on the required mechanism given vendors may return various sorts of error 
> codes.
> 
> Thanks,
> Valerie

Valerie Peng has updated the pull request incrementally with one additional 
commit since the last revision:

  Address review comments and reverted back to iterating through all available
  mechanisms.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/20207/files
  - new: https://git.openjdk.org/jdk/pull/20207/files/a37ab014..8f3bdbbc

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=20207&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=20207&range=00-01

  Stats: 27 lines in 3 files changed: 7 ins; 16 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/20207.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20207/head:pull/20207

PR: https://git.openjdk.org/jdk/pull/20207

Reply via email to