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