[jdk17u-dev] RFR: 8304976: Optimize DateTimeFormatterBuilder.ZoneTextPrinterParser.getTree()

[Aleksey Shipilev]

On Thu, 1 Jun 2023 11:06:06 GMT, Oli Gillespie <ogillespie at openjdk.org> wrote:

> Yes that's true, but it doesn't seem particularly valuable, and discussion on the details might easily outweigh any benefits. There wasn't much evidence provided for the original change, anyway.
> 
> For example, it should be `2` to match upstream behaviour due to the load factor. 

Right. It looks like `new HashMap<>(2)` is good then?

For backports, I would rather match the behavior to other releases as much as possible. At very least this allows to be more bug-to-bug compatible with other releases. If we match, a later bug/improvement report against this change in 17u have more chances to apply to other releases, and vice versa.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1414#issuecomment-1571853787