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

Chris Plummer cjplummer at openjdk.java.net
Tue Sep 29 21:09:15 UTC 2020


On Tue, 29 Sep 2020 14:43:30 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

>> 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?

I have to push back on this. Needs to be broken up. You have 3 large changes that are completely unrelated. Seems you
can push any of those 3 independent of the others.

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

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


More information about the serviceability-dev mailing list