+1 On Oct 29, 2013, at 5:13 PM, huizhe wang wrote: > A quick fix to add the missing braces and correct the misspellings. > > webrevs: http://cr.openjdk.java.net/~joehw/jdk8/8027484/webrev/ > > The following is where the braces were missing, others were a global > substitution of the misspelling. > > if (_factory == null) { > - if (useServicesMachnism) > + if (useServicesMechanism) { > _factory = DocumentBuilderFactory.newInstance(); > if (!(_factory instanceof > com.sun.org.apache.xerces.internal.jaxp.DocumentBuilderFactoryImpl)) { > _internal = false; > } > - else > + } else { > _factory = DocumentBuilderFactory.newInstance( > > "com.sun.org.apache.xerces.internal.jaxp.DocumentBuilderFactoryImpl", > SAX2DOM.class.getClassLoader() > ); > } > + } > > > Thanks > Joe >
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com