Looks good and trivial.

Thanks,
David

On 28/04/2020 3:02 pm, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8243946/webrev.00
2 lines changed: 0 ins; 0 del; 2 mod;

Hi all,

(so now it's my time to apology for inconvenience)

could you please review this small follow-up fix/partial revert of 8243928[1]? 
serviceability/sa/TestCpoolForInvokeDynamic.java  and TestDefaultMethods.java 
test use non-exported API so they can't be run in driver mode (b/c jtreg use 
vanilla JVM for driver code, meaning even exports from @modules tags are 
ignored)

JBS: https://bugs.openjdk.java.net/browse/JDK-8243946
testing: test/hotspot/jtreg/serviceability (in progress)
webrev: http://cr.openjdk.java.net/~iignatyev//8243946/webrev.00

[1]  https://bugs.openjdk.java.net/browse/JDK-8243928

Thanks,
-- Igor

Reply via email to