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

Re: SAX/Java Proposed Changes

• To: Kirk Allen Evans <kaevans@x...>
• Subject: Re: SAX/Java Proposed Changes
• From: Dennis Sosnoski <dms@s...>
• Date: Sun, 07 Mar 2004 11:45:55 -0800
• Cc: xml-dev@l...
• In-reply-to: <200403071358418.SM05492@evanslt>
• References: <200403071358418.SM05492@evanslt>
• User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3) Gecko/20030312

Kirk Allen Evans wrote:

>endDocument should only be raised after exception if the rest of the
>document continues to be parsed after exception.  
>
><?xml version="1.0" encoding="UTF-8"?>
><root>
>   <foo/>
>   <foo>&bar;</foo>
>   <foo/>
>   <foo/>
></root>
>
>The entity "bar" is not declared here.  If you are parsing this document,
>how many times do you expect startElement to be invoked?  If you argue it
>raised three times, then endDocument should not be invoked because you imply
>that you reached the end of the document when you did not.  You should only
>invoke endDocument when you actually parse the end of the document.
>
>Kirk
> 
>  
>
I'll vote with Kirk, Jochen, Karl, and others on this one. From a 
principle of least surprise standpoint, I don't see the sense in 
endDocument() being called when the document has not been fully 
processed. I think it should only be called upon successful completion 
of the parse.

As for filters and handlers, they should always initialize their state 
on startDocument(). That will prevent any errors due to reuse after 
attempting to parse a bad document.

  - Dennis

• Follow-Ups:
  • Re: SAX/Java Proposed Changes
    • From: "Karl Waclawek" <karl@w...>
  • Re: SAX/Java Proposed Changes
    • From: Elliotte Rusty Harold <elharo@m...>

• References:
  • RE: SAX/Java Proposed Changes
    • From: "Kirk Allen Evans" <kaevans@x...>

• Prev by Date: RE: SAX/Java Proposed Changes
• Next by Date: Re: SAX/Java Proposed Changes
• Previous by thread: RE: SAX/Java Proposed Changes
• Next by thread: Re: SAX/Java Proposed Changes
• Index(es):
  • Date
  • Thread