Berin Loritsch wrote:
> 
> > -----Original Message-----
> > From: Stefano Mazzocchi [mailto:[EMAIL PROTECTED]]
> >
> > Sylvain Wallez wrote:
> >
> > > So what about a new VersatileTrAXTransformer that would be
> > the "old"
> > > one (i.e. doesn't use XSLTProcessor) with a configurable
> > > TransformerFactory ? This would then avoid a new CS used
> > only by the
> > > TrAXTransformer and the need for "hacky" subroles.
> >
> > Yeah, makes perfect sense.
> >
> > In fact, the use of XSLT 'internally' is way different than
> > the use of XSLT 'externally' (that is, at the application
> > level rather than at core level).
> >
> > I would go even further: I would change the *existing*
> > TrAXTransformer to use TrAX directly and not add yet another
> > transformer that behaviorally does the exact same thing.
> 
> :) In that case we should also decide whether we will officially
> support the XT package.  It has not been maintained since 1999,
> and is behind the times in both SAX conformance (it is level 1 and
> Cocoon2 is level 2), and in XSLT spec conformance.
> 
> If we get rid of it, we can get rid of the cruft that we have to
> wrap SAX1 XMLParsers with SAX2 XMLReaders.  Which also means that
> we can say we require all SAX2 compliant components.  We can get
> rid of some deprecation messages, and simplify the cocoon.xml
> package.

Ok, I vote to remove the support for XT since it's dead meat.

Anybody against this?

> > So, let's see, here are the proposed changes now:
> >
> >  1) decouple TrAXTransformer from the internal Cocoon
> > components and make it use TrAX directly.
> 
> +1
> 
> Actually I would like to see a complete rewrite of the compiled
> XSP/Sitemap section, but since I can't put in the cycles, I can't
> force it to happen.

Yeah, same here :/

> >  2) remove the <xslt-processor-role> parameter
> 
> +100

no kidding :)

> >  3) add a <trax-factory> parameter to indicate which class
> > should be used as a TrAX factory.
> 
> +1
> 
> > That's it. It is *slightly* back compatible, but I'm sure
> > that cocoon users care *only* for the ability to have their
> > own XSLT implementation only at user level (in fact, changing
> > the xslt-processor component implementation just creates
> > problems since we are basing our behavior on xalan-only
> > features so far).
> 
> The people it affects most are ones who used the <xslt-processor-role>
> which are few in number.

Exactly, probably Sylvain is the only one that uses it :)

> >
> > If I don't receive any -1, I'll go ahead and patch it.
> 
> Sounds good!

Great, consider it coming.

-- 
Stefano Mazzocchi      One must still have chaos in oneself to be
                          able to give birth to a dancing star.
<[EMAIL PROTECTED]>                             Friedrich Nietzsche
--------------------------------------------------------------------


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

Reply via email to