On Tue, 14 Mar 2023 15:10:04 GMT, Doug Simon <dnsi...@openjdk.org> wrote:

>> Access to internal modifiers is needed in 
>> `HotSpotResolvedJavaFieldTest.testEquivalenceForInternalFields()`. I moved 
>> the declaration of the method to `HotSpotResolvedJavaField`. Does this 
>> change work for you?
>
> Just use reflection to read the internal flags (like this test already does 
> for the `index` field).
> 
> I've attached 
> [review.patch](https://github.com/openjdk/jdk/files/10970245/review.patch) 
> with this change and a few other changes I think should be made for better 
> naming (plus one test cleanup).

Thank you for the patch, it will be included in the next commit.

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

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

Reply via email to