xml-dev - Re: SAX2 RFD: LexicalHandler draft v.1.1

[Lars Marius Garshol <larsga@ifi.uio.no>]

* 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@ic.ac.uk
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@ic.ac.uk the following message;
(un)subscribe xml-dev
To subscribe to the digests, mailto:majordomo@ic.ac.uk the following message;
subscribe xml-dev-digest
List coordinator, Henry Rzepa (mailto:rzepa@ic.ac.uk)