Re: [xml-dev] SAX/Java Proposed Changes

Elliotte Rusty Harold <[email protected]> Mon, 8 Mar 2004 11:02:16 -0500
Newsgroups gmane.text.xml.sax.devel,gmane.text.xml.devel
Message-ID <p0601020dbc724800722a@[192.168.254.4]>
At 10:49 AM -0500 3/8/04, Karl Waclawek wrote:

>Well, obviously, any initialization or cleanup methods would have to
>be exposed, that was the point about adding a parsingDone() method.
>I guess there was a misunderstanding.

I would prefer to avoid adding methods to the API when methods 
already exist that serve this purpose, even if their names aren't 
ideal.

>If I use a Stream, then I should close it at the end.
>Does that mean the Stream class has a bad API?

No, in the stream API the behavior of each method is well-defined 
irrespective of order. If you call don't call close() at the end, 
nothing bad happens. If you call close at the beginning the 
read()/write() methods still behave as documented. No method fails to 
do its job simply because its called before or after another method. 
The larger program that uses the class may depend on a specific order 
of method invocations. However, the class itself exposes only atomic 
operations.


>The contentHandler would still do its own cleanup.
>Just not throught endDocument(), which should serve a different purpose.

I'm not sure we need two methods here.

-- 

   Elliotte Rusty Harold
   [email protected]
   Effective XML (Addison-Wesley, 2003)
   http://www.cafeconleche.org/books/effectivexml
   http://www.amazon.com/exec/obidos/ISBN%3D0321150406/ref%3Dnosim/cafeaulaitA


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
List: sax-devel, [email protected]
See:  http://www.saxproject.org/
https://lists.sourceforge.net/lists/listinfo/sax-devel