Thanks Serguei!

Harold

On 12/13/2019 4:50 PM, [email protected] wrote:
Hi Harold,

+1

Thanks,
Serguei

On 12/13/19 11:45 AM, Daniel D. Daugherty wrote:
On 12/13/19 2:35 PM, Harold Seigel wrote:
Hi,

Please review this trivial fix to prevent java/lang/instrument/... TestRecordAttr.java and TestRecordAttrGenericSig.java from failing.  The fix replaces hard-wired JDK version 14 with mechanisms that get the latest JDK version.

Open Webrev: http://cr.openjdk.java.net/~hseigel/bug_8235922/webrev/index.html

test/jdk/java/lang/instrument/RedefineRecordAttr/TestRecordAttr.java
    No comments.

test/jdk/java/lang/instrument/RedefineRecordAttrGenericSig/TestRecordAttrGenericSig.java
    No comments.

Thumbs up! I agree that this is a trivial fix.

Thanks for fixing this so quickly!

Dan


JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8235922

The fix was tested by running the tests locally on Linux-x64.

Thanks, Harold



Reply via email to