RFR: 8294969: Convert jdk.jdeps javap to use the Classfile API [v11]
Vicente Romero
vromero at openjdk.org
Wed Sep 6 23:15:50 UTC 2023
On Thu, 20 Jul 2023 09:11:20 GMT, Adam Sotona <asotona at openjdk.org> wrote:
>> javap uses proprietary com.sun.tools.classfile library to parse class files.
>>
>> This patch converts javap to use Classfile API.
>>
>> Please review.
>>
>> Thanks,
>> Adam
>
> Adam Sotona has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 227 commits:
>
> - Merge branch 'master' into JDK-8294969-javap
> - Merge branch 'master' into JDK-8294969-javap
> - fixed code printing and ConstantPoolException reporting indoex
> - added DydnamicConstantPoolEntry::bootstrapMethodIndex
> fix of javap ConstantWriter to print DynamicConstantPoolEntry without accessing BSM attribute
> - extended ClassReader about specific entry-reading methods to avoid class cast and throw ConstantPoolException instead
> - throwing ConstantPoolException for invalid BSM entry index
> - Merge branch 'master' into JDK-8294969-javap
> - fixed JavapTask
> - Merge branch 'master' into JDK-8294969-javap
> - Merge branch 'master' into JDK-8294969-javap
>
> # Conflicts:
> # src/java.base/share/classes/jdk/internal/classfile/constantpool/ConstantPoolException.java
> - ... and 217 more: https://git.openjdk.org/jdk/compare/37c756a7...4960751b
src/jdk.jdeps/share/classes/com/sun/tools/javap/CodeWriter.java line 89:
> 87: ++n; // for 'this'
> 88: argCount = Integer.toString(n);
> 89: } catch (ConstantPoolException e) {
nice clean-up
src/jdk.jdeps/share/classes/com/sun/tools/javap/LocalVariableTableWriter.java line 46:
> 44: * deletion without notice.</b>
> 45: */
> 46: public class LocalVariableTableWriter extends InstructionDetailWriter {
nit: two spaces between `extends` and `InstructionDetailWriter`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/11411#discussion_r1317835882
PR Review Comment: https://git.openjdk.org/jdk/pull/11411#discussion_r1317915971
More information about the compiler-dev
mailing list