RFR: javax.xml.transform: Using ServiceLoader to load JAXP stream factories (7169894: JAXP Plugability Layer: using service loader)

Mandy Chung mandy.chung at oracle.com
Wed Dec 19 18:43:21 UTC 2012


On 12/19/2012 6:34 AM, Daniel Fuchs wrote:
>
> <http://cr.openjdk.java.net/~dfuchs/JDK-7169894/javax.xml.transform/webrev.01/> 
>

Thanks for the update and it looks much better.  FactoryFinder L214-215: 
this will swallow the new TransformerFactoryConfigurationError thrown at 
L210.  Maybe you can explicitly catch CCE that rethrow TFCE.

Nit:  L189-190 needs some indentation and probably better to merge them 
in one line.

Mandy



More information about the core-libs-dev mailing list