This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Patch: FYI: fix SAX fallback
- From: Tom Tromey <tromey at redhat dot com>
- To: GCJ-patches <java-patches at gcc dot gnu dot org>
- Date: 27 Mar 2007 13:10:36 -0600
- Subject: Patch: FYI: fix SAX fallback
- Reply-to: tromey at redhat dot com
I'm checking this in on the trunk and the RH 4.1 branch.
Removing the service files broke SAX. It turns out we have to modify
SAX -- it even says so -- to make this work properly.
I tested this behavior on the Sun 1.5 JDK and we now follow it. In
particular it will load this from Xerces even if it is only on the
classpath, not just if it is endorsed.
I'll push this into Classpath as well.
Tom
Index: ChangeLog
from Tom Tromey <tromey@redhat.com>
PR classpath/31303:
* external/sax/org/xml/sax/helpers/XMLReaderFactory.java
(createXMLReader): Code in Classpath default.
Index: external/sax/org/xml/sax/helpers/XMLReaderFactory.java
===================================================================
--- external/sax/org/xml/sax/helpers/XMLReaderFactory.java (revision 123217)
+++ external/sax/org/xml/sax/helpers/XMLReaderFactory.java (working copy)
@@ -137,6 +137,11 @@
if (className == null) {
// BEGIN DISTRIBUTION-SPECIFIC
+ // CLASSPATH LOCAL: have to code in the backup.
+ // Among other things, see PR 31303, and this thread:
+ // http://gcc.gnu.org/ml/java-patches/2007-q1/msg00661.html
+ className = "gnu.xml.stream.SAXParser";
+
// EXAMPLE:
// className = "com.example.sax.XmlReader";
// or a $JAVA_HOME/jre/lib/*properties setting...