RFR: 8311077: Fix -Wconversion warnings in jvmti code [v4]
    Coleen Phillimore 
    coleenp at openjdk.org
       
    Wed Jul  5 12:34:05 UTC 2023
    
    
  
On Fri, 30 Jun 2023 13:10:06 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> Please review change for mostly fixing return types in the constant pool and metadata to fix -Wconversion warnings in JVMTI code.  The order of preference for changes are: 1. change the types to more distinct types (fields in the constant pool are u2 because that's their size in the classfile), 2. add direct int casts if the value has been checked in asserts above, and 3. checked_cast<> if not verified, and 4. added some pointer_delta_as_ints where needed.
>> Tested with tier1-4.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   David's suggestions.
Thanks David for the review.  I do change the types as low in the call stack as practical with these changes.  I think your comment above disappeared because the last change reverted that code.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14710#issuecomment-1621658684
    
    
More information about the hotspot-dev
mailing list