RFR (JAXP): 8027484 : Implementation error in SAX2DOM.java

[Lance Andersen - Oracle]

+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
> 

-------------- next part --------------

Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com