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

Reply via email to