[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Classpathx-xml] Re: Unwanted SAXParseException

From: Ito Kazumitsu
Subject: [Classpathx-xml] Re: Unwanted SAXParseException
Date: Tue, 21 Oct 2003 01:02:16 +0900
User-agent: EMH/1.10.0 SEMI/1.13.7 (Awazu) FLIM/1.13.2 (Kasanui) Emacs/21.2 (i386-unknown-freebsd4.7) MULE/5.0 (SAKAKI)

>>>>> ":" == David Brownell <address@hidden> writes:

:> You didn't see that warning though, right?  Odd.  And you didn't
:> see it _after_ applying your patch either either.


P1:  The patch to gnu/xml/aelfred2/
     dated Sat Oct 18 08:43:42 2003

P2:  The patch to gnu/xml/aelfred2/
     dated Sun Oct 19 10:07:50 2003

W1:  "URI was not reported to parser for entity [dtd]"
     from gnu/xml/aelfred2/

W2:  "missing system ID, using test.dtd"
     from gnu/xml/aelfred2/

P1  P2     W1  W2
--- ---    --- ---
No  No     Yes Yes
Yes No     No  No
No  Yes    Yes No

:> Guessing ids[1] seems wrong; rather than recording systemIdGuessed
:> as in your patch, I'd just remove the assignment.  And the test,
:> and the warning, since the SAXDriver should handle that (see my
:> response to Nic).

So here is another patch:

bash-2.05b$ diff -u gnu/xml/aelfred2/ 
--- gnu/xml/aelfred2/        Sun Sep 14 07:32:02 2003
+++ gnu/xml/aelfred2/     Tue Oct 21 00:52:17 2003
@@ -3449,10 +3449,12 @@
            // we might be using alternate IDs/encoding
            systemId = source.getSystemId ();
+           /*   systemId may be null but we should not make a guess
            if (systemId == null) {
                handler.warn ("missing system ID, using " + ids [1]);
                systemId = ids [1];
+           */
        } else {
            // "[document]", or "[dtd]" via getExternalSubset()
            scratch.setCharacterStream (reader);

reply via email to

[Prev in Thread] Current Thread [Next in Thread]