> Torsten Curdt a écrit :
> >
> > Are there reasons or objections not to change
> > the access to the contenthandler in the xsp.xsl
> > from "this.contentHandler" to only "contentHandler"?
> >
> > Then we could do a local redirect of SAX events:
> >
> >   ...
> >   contentHandler.startElement("", "item", "item", xspAttr);
> >   {
> >     AttributesImpl xspAttr = new AttributesImpl();
> >     XMLConsumer contentHandler = ...;
> >
> >     contentHandler.startElement("", "item", "item", xspAttr);
> >   }
> >
> > It gives us the feature of redirecting SAX events
> > to other XMLConsumers. (This would be a real seamless
> > integration for the XForm and DOMObject implementation)
> >
> > There should be no undesired side effects...
>
> Yes, there are side effects with logicsheets that use
> this.contentHandler.

Then we make them not to use it, too ;)

> I use the following construct to change the contentHandler :
> <xsp:logic>
>   ContentHandler oldHandler = this.contentHandler;
>   try {
>     this.contentHandler = otherHandler;
>     ...
>   } finally {
>     this.contentHandler = oldHandler;
>   }
> </xsp:logic>

...I was just not sure if changing the contentHandler in
a multi thread environment is a good idea...

..but if this is really ok - I'm also fine with that!
(Although a simple search and replace over logicsheets
 should be a bit cleaner - don't you think?)

regards
--
Torsten


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, email: [EMAIL PROTECTED]

Reply via email to