RFR: 8252000: remove usage of PropertyResolvingWrapper in vmTestbase/nsk/jdb [v2]

Igor Ignatyev iignatyev at openjdk.java.net
Tue Sep 29 14:46:02 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 , do you have any further comments? Or are you (relatively) happy w/ the patch after the last 3 commits?

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

PR: https://git.openjdk.java.net/jdk/pull/350


More information about the serviceability-dev mailing list