> Hi,
> 
>  The constant pool currently has a lot of methods specific to extracting 
> parts of the operands array. What this array actually is, is a sequence of 
> bootstrap method attribute entries, where each entry has the following 
> components:
> 
> ```c++
> struct BSMAE {
>   u2 bootstrap_method_index;
>   u2 argument_count;
>   u2 arguments[argument_count];
> }
> 
> 
> We can removes some of these operands array specific methods, and instead 
> allows you to extract BSMAttributeEntrys which you can then use to extract 
> its piece wise components. This makes for a nicer interface, and a bit easier 
> to come into as a reader of the code, as it more closely mirrors the JVMS. 
> 
> Please consider!
> 
> Testing: Currently GHA, running tier1-tier3

Johan Sjölen has updated the pull request incrementally with one additional 
commit since the last revision:

  Lois's comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25298/files
  - new: https://git.openjdk.org/jdk/pull/25298/files/e10d91a5..13e27259

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25298&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25298&range=00-01

  Stats: 10 lines in 1 file changed: 0 ins; 10 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/25298.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25298/head:pull/25298

PR: https://git.openjdk.org/jdk/pull/25298

Reply via email to