RFR: 8278863: Add method ClassDesc::ofInternalName [v3]
Adam Sotona
asotona at openjdk.org
Mon Aug 22 07:27:25 UTC 2022
On Tue, 26 Jul 2022 09:25:48 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Adam Sotona has updated the pull request incrementally with one additional commit since the last revision:
>>
>> updated ClassDesc::ofInternal javadoc with JVMS link and fixed indentation
>
> test/jdk/java/lang/constant/ClassDescTest.java line 268:
>
>> 266: }
>> 267:
>> 268: List<String> badInternalNames = List.of("I;", "[]",
>
> is `[]` a good test for arrays? Wouldn't something more realistic like `[Ljava/lang/String` be better? I note that `[]` is also used in another test. Problem with testing `[]` is that if the checking logic has a bug, or a regression is introduced so that it only detects `]` instead of `[` the test would not detect that (but I guess that's a remote possibility).
Thanks for pointing it out, I'll add a valid array descriptor into the list of badInternalNames.
-------------
PR: https://git.openjdk.org/jdk/pull/9201
More information about the core-libs-dev
mailing list