On 15/05/2019 20:11, Valerie Peng wrote:
I updated the webrev to switch to ConcurrentHashMap. The javadoc spec
of computeIfAbsent method cautioned that the mapping func should be
short and simple and must not attempt to update other mappings of this
map. Provider verification code does not quite fit the above criteria
for the mapping. So, I did not use computeIfAbsent method and just
made minor update to webrev.01 with Xuelei's suggestion of re-checking
the cache again inside the synchronized block.
http://cr.openjdk.java.net/~valeriep/7107615/webrev.03/
Comments?
Does it need to synchronize on JceSecurity.class? I'm mostly wondering
if it can use computeIfAbsent.
-Alan
