25 messages in org.ibiblio.lists.xom-interest[XOM-interest] Three cosmetic patches
FromSent OnAttachments
Wolfgang HoschekFeb 9, 2004 6:42 pm 
Elliotte Rusty HaroldFeb 10, 2004 9:25 am 
Wolfgang HoschekFeb 10, 2004 3:23 pm 
jco...@reutershealth.comFeb 10, 2004 3:48 pm 
Elliotte Rusty HaroldFeb 11, 2004 1:21 pm 
Wolfgang HoschekFeb 11, 2004 1:24 pm 
jco...@reutershealth.comFeb 11, 2004 2:15 pm 
Elliotte Rusty HaroldFeb 11, 2004 3:18 pm 
jco...@reutershealth.comFeb 11, 2004 4:20 pm 
Francois BeausoleilFeb 11, 2004 8:08 pm 
Trimmer, ToddFeb 12, 2004 11:57 am 
Elliotte Rusty HaroldFeb 12, 2004 2:53 pm 
Trimmer, ToddFeb 13, 2004 1:26 pm 
Elliotte Rusty HaroldMar 8, 2004 3:17 pm 
jco...@reutershealth.comMar 8, 2004 4:06 pm 
Elliotte Rusty HaroldMar 8, 2004 4:34 pm 
Elliotte Rusty HaroldMar 9, 2004 11:55 am 
Bradley S. HuffmanMar 9, 2004 12:09 pm 
Elliotte Rusty HaroldMar 9, 2004 1:05 pm 
jco...@reutershealth.comMar 9, 2004 2:04 pm 
jco...@reutershealth.comMar 9, 2004 4:32 pm 
Elliotte Rusty HaroldMar 9, 2004 10:43 pm 
John CowanMar 9, 2004 10:58 pm 
Elliotte Rusty HaroldMar 10, 2004 6:12 am 
Dirk BergstromMar 10, 2004 6:25 pm 
Actions with this message:
Paste this link in email or IM:
Paste this link in email or IM:
Atom feed for this thread
Paste this URL into your reader:
Subject:[XOM-interest] Three cosmetic patchesActions...
From:jco...@reutershealth.com (jco@reutershealth.com)
Date:Feb 11, 2004 2:15:06 pm
List:org.ibiblio.lists.xom-interest

Elliotte Rusty Harold scripsit:

OK. Based on this I made that change. Code will be in CVS soon if it isn't already. However, my benchmarks were not able to measure any performance difference between the two cases. If there is one, it's below my resolution.

I note that Serializer is recursive, and should be fixed on the same grounds: see http://lists.ibiblio.org/pipermail/xom-interest/2002-September/000082.html for the non-recursive tree-walking algorithm I used in DOMParser.

Note also that the above message proposes an extension to SAXConverter to make it a full XMLReader; the parse() methods ignore their arguments, and aSaxConverter.setProperty("http://xom.nu/properties/xom-node", aDocument) specifies which document to walk. You never actually rejected this, and it's very simple and allows code that expects to be instantiating a SAX parser to do so with minimal change.