On Thu, 25 Apr 2024 13:38:56 GMT, Viktor Klang <vkl...@openjdk.org> wrote:
>> Claes Redestad has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> comma-separated
>
> src/java.base/share/classes/java/lang/constant/MethodTypeDesc.java line 208:
>
>> 206: default String displayDescriptor() {
>> 207: int count = parameterCount();
>> 208: StringBuilder sb = new StringBuilder(24).append('(');
>
> 24 is chosen by fair dice-roll? :)
More or less: default capacity is 16, 24 is 1.5x that - a pretty typical growth
factor. It also happens to be enough for very common descriptors such as
`()Ljava/lang/Object;` while not too large to cause a regression on very small
(and also very common) descriptors such as `()V`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18945#discussion_r1579533391
