RFR: 8270317: Large Allocation in CipherSuite

Clive Verghese cverghese at openjdk.java.net
Thu Jul 15 17:55:12 UTC 2021


On Thu, 15 Jul 2021 17:51:36 GMT, djelinski <github.com+30433125+djelinski at openjdk.org> wrote:

>> That's what containsKey() does, returning false of the value is null.
>
> Right. But calling `containsKey` and then `get` results in two hash map lookups. If we cache the result of `get`, we only need one lookup, which should be a tiny bit faster. See https://stackoverflow.com/q/14601016

I feel that by checking for null, we can reduce one lookup at the Hashtable. Will update the pull request.

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

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



More information about the security-dev mailing list