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

huizhe wang huizhe.wang at oracle.com
Tue Oct 29 21:13:13 UTC 2013


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




More information about the core-libs-dev mailing list