On Tue, 29 Sep 2020 20:57:02 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:

>> the patch removes `PropertyResolvingWrapper` from `vmTestbase/nsk/jdwp` 
>> tests and cleans up/reformats the touched files.
>> 
>> testing:  ✅ `vmTestbase/nsk/jdwp` on `{linux,windows,macos}-x64`
>
> The changes look good, although I wouldn't have preferred multiple RFRs for 
> these changes. Yes, you have separate
> commits which did make them easier to review, but Skara is going to collapse 
> them all into one commit.

Igor, this update looks good.

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

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

Reply via email to