RFR: 8252000: remove usage of PropertyResolvingWrapper in vmTestbase/nsk/jdb [v2]
    Igor Ignatyev 
    iignatyev at openjdk.java.net
       
    Fri Sep 25 15:18:48 UTC 2020
    
    
  
On Fri, 25 Sep 2020 05:38:08 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Igor Ignatyev has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - realign table in locals002
>>  - updated copyright year
>>  - move var declarations closer to the 1st use
>>    sprinkle some `var`
>
> The changes look good, but I think the practice of doing a massive cleanup in "remove usage of PropertyResolvingWrapper
> in vmTestbase/nsk/jdb" PR is misleading. Better to split this in two issues (at least next time)?
@shipilev , the next iteration should have all your comments addressed.
-------------
PR: https://git.openjdk.java.net/jdk/pull/350
    
    
More information about the serviceability-dev
mailing list