RFR JDK-8170192 [JAXP] [TESTBUG] test/javax/xml/jaxp/libs/jaxp/library/JAXPPolicyManager.java should grant permissions to jtreg, javatest, and testng jars

Langer, Christoph christoph.langer at sap.com
Wed Nov 23 07:48:27 UTC 2016


Hi Frank,

to me this looks fine. Maybe a small cosmetical thing: Around lines 212 and 240 you could remove the additional empty lines to have one blank line between methods as in the rest of the file.

Note that I'm not a reviewer.

Best regards
Christoph

From: Frank Yuan [mailto:frank.yuan at oracle.com]
Sent: Mittwoch, 23. November 2016 05:41
To: core-libs-dev at openjdk.java.net; 'Daniel Fuchs' <daniel.fuchs at oracle.com>; Langer, Christoph <christoph.langer at sap.com>; 'Volker Simonis' <volker.simonis at gmail.com>; 'Joe Wang' <huizhe.wang at oracle.com>
Cc: code-tools-dev at openjdk.java.net; jtreg-use at openjdk.java.net
Subject: RFR JDK-8170192 [JAXP] [TESTBUG] test/javax/xml/jaxp/libs/jaxp/library/JAXPPolicyManager.java should grant permissions to jtreg, javatest, and testng jars

Hi All

Would you like to review http://cr.openjdk.java.net/~fyuan/8170192/webrev.00/?
Bug: https://bugs.openjdk.java.net/browse/JDK-8170192

This patch is fully same as Daniel provided except a few lines of additional cleaning, thanks to Daniel for providing the patch!

Thanks
Frank



More information about the core-libs-dev mailing list