RFR: 8294982: Implementation of Classfile API [v20]
Adam Sotona
asotona at openjdk.org
Mon Feb 27 10:45:51 UTC 2023
On Thu, 16 Feb 2023 10:54:00 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Adam Sotona has updated the pull request incrementally with one additional commit since the last revision:
>>
>> added 4-byte Unicode text to Utf8EntryTest
>
> src/java.base/share/classes/jdk/internal/classfile/components/ClassPrinter.java line 140:
>
>> 138:
>> 139: /**
>> 140: * A tree node holding {@link List} of nested nodes.
>
> It would perhaps be beneficial to have little examples of what these different nodes are used for. I had to look at `ClassPrinterImpl` to get some idea.
I'll link examples from package-info here.
> src/java.base/share/classes/jdk/internal/classfile/impl/ClassPrinterImpl.java line 745:
>
>> 743: list("stack", "item", Stream.of())));
>> 744: }
>> 745: var excHandlers = com.exceptionHandlers().stream().map(exc -> new ExceptionHandler(com.labelToBci(exc.tryStart()), com.labelToBci(exc.tryEnd()), com.labelToBci(exc.handler()), exc.catchType().map(ct -> ct.name().stringValue()).orElse(null))).toList();
>
> Watch out for very long lines
wil fix it, thanks.
-------------
PR: https://git.openjdk.org/jdk/pull/10982
More information about the core-libs-dev
mailing list