RFR: 8342469: Improve API documentation for java.lang.classfile.instruction [v3]

Chen Liang liach at openjdk.org
Mon Nov 18 04:12:42 UTC 2024


On Mon, 18 Nov 2024 02:58:31 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Improve the documentation for classfile instructions. Includes links to all opcodes, usage notes for instructions, and other various fixes.
>> 
>> API Diff: https://cr.openjdk.org/~liach/apidiff/cf-instr/java.base/module-summary.html
>> Javadoc: https://cr.openjdk.org/~liach/javadoc/cf-instr/java.base/java/lang/classfile/instruction/package-summary.html
>> 
>> Please review the associated CSR as well.
>
> Chen Liang has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
> 
>  - Further facelift
>  - Merge branch 'master' of https://github.com/openjdk/jdk into docs/cf-instruction
>  - Merge branch 'master' of https://github.com/openjdk/jdk into docs/cf-instruction
>  - Typos, long lines
>  - Labels
>  - Wip instr

After offline discussions, I believe it is necessary to also include the data models of instructions in the javadoc; they will be removed from the `java.lang.classfile` top-level documentation later.

I am pleased with the current state of the documentation, and have submitted a CSR.
Please review the renderings:

API Diff: https://cr.openjdk.org/~liach/apidiff/cf-instr/java.base/module-summary.html
Javadoc: https://cr.openjdk.org/~liach/javadoc/cf-instr/java.base/java/lang/classfile/instruction/package-summary.html

(Omitted the constant rendering changes; committed locally, will push with other review fixes later)

-------------

PR Comment: https://git.openjdk.org/jdk/pull/21625#issuecomment-2481906372


More information about the core-libs-dev mailing list