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 <address@hidden> 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/
   `-