[XML-DEV Mailing List Archive Home] [By Thread] [By Date] [Recent Entries] [Reply To This Message]

Re: SAX2 RFD: LexicalHandler draft v.1.1

• From: Lars Marius Garshol <larsga@i...>
• To: "XML Developers' List" <xml-dev@i...>
• Date: 21 Mar 1999 18:14:13 +0100

* David Megginson
|
|     public abstract void xmlDecl (String version,
| 				  String encoding,
| 				  String standalone)
| 	throws SAXException;

Should we perhaps make standalone a boolean instead?  It can only have
two values anyway, and this will spare us a lot of
standalone.equals(this or that).
 
|     public abstract void startDTD (String doctype,
| 				   String publicID,
| 				   String systemID)
| 	throws SAXException;

I think naming doctype docelem or rootelem would be better. It took me
a couple of seconds to figure out what it meant.
 
|     public abstract void startEntity (String name)
| 	throws SAXException;

Is this sufficient? Now we don't even know whether it's internal or
external. I know EntityResolver can be used to get that information
(including sysid and pubid), but I'd much rather see it included here
as well, since IMHO EntityResolver fills a separate role from the data
handlers. Often one would want to plug in a separate component there.

How about this?

      public abstract void startEntity (String name, String publicID,
                                        String systemID)
        throws SAXException;
 
If systemID is null we know it is an external entity. Alternatively,
we could have a separate callback for external entities.

--Lars M.


xml-dev: A list for W3C XML Developers. To post, mailto:xml-dev@i...
Archived as: http://www.lists.ic.ac.uk/hypermail/xml-dev/ and on CD-ROM/ISBN 981-02-3594-1
To (un)subscribe, mailto:majordomo@i... the following message;
(un)subscribe xml-dev
To subscribe to the digests, mailto:majordomo@i... the following message;
subscribe xml-dev-digest
List coordinator, Henry Rzepa (mailto:rzepa@i...)

• Follow-Ups:
  • Re: SAX2 RFD: LexicalHandler draft v.1.1
    • From: David Megginson <david@m...>

• References:
  • SAX2 RFD: LexicalHandler draft v.1.1
    • From: David Megginson <david@m...>

• Prev by Date: IE5.0 does not conform to RFC2376
• Next by Date: Re: DOM Implemetation in C?
• Previous by thread: SAX2 RFD: LexicalHandler draft v.1.1
• Next by thread: Re: SAX2 RFD: LexicalHandler draft v.1.1
• Index(es):
  • Date
  • Thread