On Wed, 30 Apr 2025 18:03:29 GMT, Chen Liang <li...@openjdk.org> wrote:

>> A prerequisite to #24760; this was part of it, but was split out to simplify 
>> the review of the addition of `maskToAccessFlags(int, Location, CFFV)` by 
>> minimizing the changes in javap. That patch will only need to update 
>> `BasicWriter` as a result.
>> 
>> Testing: langtools/tools/javap, tier 1+2 underway
>
> Chen Liang has updated the pull request with a new target base due to a merge 
> or a rebase. The incremental webrev excludes the unrelated changes brought in 
> by the merge/rebase. The pull request contains three additional commits since 
> the last revision:
> 
>  - Adapt the location APIs
>  - Merge branch 'master' of https://github.com/openjdk/jdk into 
> fix/javap-cffv-access-flags
>  - 8355956: Prepare javap for class file format aware access flag parsing

Thanks for the swift review!

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

PR Comment: https://git.openjdk.org/jdk/pull/24955#issuecomment-2843414833

Reply via email to