Hi Frank,
Looks good to me too. Thanks for adding tests.
Best,
Joe
On 12/7/16, 9:35 AM, Lance Andersen wrote:
Hi Frank,
Looks fine
Best
Lance
On Dec 6, 2016, at 10:15 PM, Frank Yuan <frank.y...@oracle.com
<mailto:frank.y...@oracle.com>> wrote:
Hi all
Would you like to review
http://cr.openjdk.java.net/~fyuan/8169948/webrev.00/?
<http://cr.openjdk.java.net/%7Efyuan/8169948/webrev.00/?>
Bug: https://bugs.openjdk.java.net/browse/JDK-8169948
This test update is because of JDK-8169778 Add new public methods to
get new instances of the JAXP factories builtin system-default
implementations. DefaultFactoryWrapperTest.java tests a provider that
return a custom factory instance wrapping the built-in
system-default implementation.
Thanks
Frank
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance
Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>