Re: [SrcML] XHTML-ViewPlugin
Status: Beta
Brought to you by:
crashchaos
From: Frank R. <fra...@un...> - 2005-11-18 09:36:16
|
On Fri, Nov 18, 2005 at 10:21:47AM +0100, Leif Bladt wrote: > >What about extending your NextElementDoneEvent into a more generic > >DoneEvent? Shouldn't require that much of a change. > Done :) It's in the CVS although the Unittests don't compile (the > JavaHelper.java doesn't compile anymore). Ops. I'll fix that when I'm back from uni tonight. > >Only problem might be the performance when every single whitespace > >results in a whole bunch of events being fired through the whole > >machinery. But we can still add some event filtering later if it > >proves > >to be a real problem and only send the events when requested. > This is a real problem, so at the moment I will fire DoneEvents only > after StringEvents are processed. That's acceptable, but also > noticeable slower. What about adding a boolean variable then for the different events specifying whether they should fire DoneEvents and an accessor method to set this behavior? -- Raiser, Frank Student @ University of Ulm (www.uni-ulm.de) This report, by its very length, defends itself against the risk of being read. (Winston Churchill) |