Re: RFR: 8341664: ReferenceClassDescImpl cache internalName [v6]

2024-10-18 Thread Chen Liang
On Fri, 18 Oct 2024 01:09:09 GMT, Shaojin Wen wrote: >> ReferenceClassDescImpl caches internalName to avoid creating a new String >> object for each call, which can also simplify the implementation of >> classfile. > > Shaojin Wen has updated the pull request with a new target base due to a >

Re: RFR: 8341664: ReferenceClassDescImpl cache internalName [v6]

2024-10-17 Thread Shaojin Wen
> ReferenceClassDescImpl caches internalName to avoid creating a new String > object for each call, which can also simplify the implementation of classfile. Shaojin Wen has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated c

Re: RFR: 8341664: ReferenceClassDescImpl cache internalName [v5]

2024-10-10 Thread Shaojin Wen
> ReferenceClassDescImpl caches internalName to avoid creating a new String > object for each call, which can also simplify the implementation of classfile. Shaojin Wen has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated c

Re: RFR: 8341664: ReferenceClassDescImpl cache internalName [v4]

2024-10-10 Thread Shaojin Wen
> ReferenceClassDescImpl caches internalName to avoid creating a new String > object for each call, which can also simplify the implementation of classfile. Shaojin Wen has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated c

Re: RFR: 8341664: ReferenceClassDescImpl cache internalName [v3]

2024-10-10 Thread Shaojin Wen
> ReferenceClassDescImpl caches internalName to avoid creating a new String > object for each call, which can also simplify the implementation of classfile. Shaojin Wen has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated c

Re: RFR: 8341664: ReferenceClassDescImpl cache internalName [v2]

2024-10-09 Thread Shaojin Wen
> ReferenceClassDescImpl caches internalName to avoid creating a new String > object for each call, which can also simplify the implementation of classfile. Shaojin Wen has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated c

RFR: 8341664: ReferenceClassDescImpl cache internalName

2024-10-07 Thread Shaojin Wen
ReferenceClassDescImpl caches internalName to avoid creating a new String object for each call, which can also simplify the implementation of classfile. - Commit messages: - suggestion from @liach - stable - ReferenceClassDescImpl cache internalName Changes: https://git.openjdk.o

Re: RFR: 8341664: ReferenceClassDescImpl cache internalName

2024-10-07 Thread Shaojin Wen
On Sun, 6 Oct 2024 01:22:33 GMT, Shaojin Wen wrote: > ReferenceClassDescImpl caches internalName to avoid creating a new String > object for each call, which can also simplify the implementation of classfile. In many scenarios where classfile is used, ClassDesc is cached by statically declarin

Re: RFR: 8341664: ReferenceClassDescImpl cache internalName

2024-10-07 Thread Chen Liang
On Sun, 6 Oct 2024 01:22:33 GMT, Shaojin Wen wrote: > ReferenceClassDescImpl caches internalName to avoid creating a new String > object for each call, which can also simplify the implementation of classfile. Changes requested by liach (Reviewer). The caching of class entry forms in ReferenceC