RFR: 8281631: HashMap copy constructor and putAll can over-allocate table [v21]

[XenoAmess]

On Sun, 20 Feb 2022 19:19:16 GMT, liach <duke at openjdk.java.net> wrote:

>> Imo you should just remove the `if (expectedSize == 0)` check than using this somewhat ugly trick to avoid calling `size()` twice (the second call is only used for this relatively useless fast-path, especially for the concurrent collections you refer to)
>
> In fact, if we do worry about the performance of adding from maps, calling `map.forEach(this::put);` would be a better alternative both in concurrency (as the concurrent map itself takes charage) and object allocation-wise (no allocation of immutable entry objects), but that belongs to another issue.

@liach Hi. please have a look at the latest commit.
do you think it be better now?

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

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