Classfile API cleanup of labelToBci from CodeModel and CodeBuilder and removal of impl.LabelResolver

[Adam Sotona]

Hi,
I have another Classfile API change proposal to discuss.
Method labelToBci from impl.LabelResolver is exposed through CodeModel.
I propose to completely remove impl.LabelResolver and move the label resolution functionality to CodeAttribute::labelToBci.
CodeAttribute is the only API subclass of CodeModel where the bytecode index can be successfully resolved and where it makes sense to use it together with CodeAttribute::codeArray.

Also CodeBuilder exposes method labelToBci, even it does not subclass LabelResolver.
I propose to remove CodeBuilder::labelToBci as it is just another exposure of internal functionality and should not serve to users.

Pull request with the proposal is here:
https://github.com/openjdk/jdk-sandbox/pull/33

Thanks,
Adam




-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/classfile-api-dev/attachments/20220818/b9234c58/attachment.htm>