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