RFR: 8311085: Remove implementation detail writeTo from LocalVariable(Type) [v3]
Brian Goetz
briangoetz at openjdk.org
Fri Jun 30 12:24:54 UTC 2023
On Fri, 30 Jun 2023 00:51:42 GMT, Chen Liang <liach at openjdk.org> wrote:
>> `LocalVariable` and `LocalVariableType` includes `writeTo(BufWriter)`, which should be implementation details.
>>
>> See https://mail.openjdk.org/pipermail/classfile-api-dev/2023-June/000381.html for context.
>>
>> This patch moves the implementation to `DirectCodeBuilder`'s original use sites; the old `b.canWriteDirect` branch is redundant, as `writeIndex`'s implementation already performs such an optimization.
>
> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>
> Fix line ending, update corpus test
Marked as reviewed by briangoetz (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/14705#pullrequestreview-1507092206
More information about the core-libs-dev
mailing list