[kaffe] Re: [Classpathx-xml] Re: Unwanted SAXParseException

[Arnaud Vandyck]

I did commit this patch.

Waiting for your comments.

I'll make a new Debian package of this version next week if no comment.

Cheers,

PS: and thanks to Ito and all of you who discuss this point ;)

On Sat, 25 Oct 2003 08:57:05 +0900
Ito Kazumitsu <kaz@maczuka.gcd.org> wrote:

> bash-2.05b$ diff -u gnu/xml/aelfred2/SAXDriver.java.orig gnu/xml/aelfred2/SAXDriver.java
> --- gnu/xml/aelfred2/SAXDriver.java.orig	Sat Oct 18 19:51:10 2003
> +++ gnu/xml/aelfred2/SAXDriver.java	Sat Oct 25 08:41:10 2003
> @@ -596,8 +596,6 @@
>  			in.getSystemId ());
>  	    if (source == null)
>  		source = in;
> -	    else if (source.getSystemId () == null)
> -		source.setSystemId (in.getSystemId ());
>  	}
>  	startExternalEntity (name, source.getSystemId (), true);
>  	return source;
> @@ -638,8 +636,13 @@
>      	boolean stackOnly)
>      throws SAXException
>      {
> +	// The following warning was deleted because the application has the
> +	// option of not setting systemId. Sun's JAXP or Xerces seems to
> +	// ignore this case.
> +	/*
>  	if (systemId == null)
>  	    warn ("URI was not reported to parser for entity " + name);
> +	*/
>  	if (!stackOnly)		// spliced [dtd] needs startEntity
>  	    lexicalHandler.startEntity (name);
>  	entityStack.push (systemId);
> bash-2.05b$ diff -u gnu/xml/aelfred2/XmlParser.java.orig gnu/xml/aelfred2/XmlParser.java


-- 
  .''`.       ** Debian GNU/Linux **
 : :' :          Arnaud   Vandyck
 `. `'   http://people.debian.org/~avdyk/
   `-