On Fri, 21 Dec 2001, Carsten Ziegeler wrote: > Sylvain Wallez wrote: > > > > Carsten Ziegeler a écrit : > > > > > > Vadim Gritsenko wrote: > > > > > > > > I'm not a namespace guru... ;) > > > > If this does not break anything, count me in: +1. > > > Same applies here: +1 > > > > > > What about making the parser configurable for this setting like > > > it is configurable regarding validation? > > > > It is already configurable in PooledJaxpParser that I wrote to reuse > > parsers instead of just factories. > > > > I've been using it here for several weeks without problems. So I propose > > to move PooledJaxpParser code to JaxpPaser and not set > > "namespace-prefixes" by default. > > > This would remove the PooledJaxpParser, right? (Or better deprecate?).
Hmm.. if we follow Berins proposal I think we should deprecate PooledJaxpParser first. +1 Giacomo > +1 > > Carsten > > > > Did you post this as a bug to SAXON? > > > > I'll do it. > > > > The Xalan bug is already identified, but marked as low priority : > > http://nagoya.apache.org/bugzilla/show_bug.cgi?id=1831 > > > > Sylvain. > > > > > Carsten > > > > > > > > > > > Vadim > > > > > > > > > -----Original Message----- > > > > > From: Sylvain Wallez [mailto:[EMAIL PROTECTED]] > > > > > Sent: Thursday, December 20, 2001 12:05 PM > > > > > To: cocoon-dev > > > > > Subject: [vote] Removing namespace-prefixes feature on parsers > > > > > > > > > > Hi team, > > > > > > > > > > The default behaviour of a SAX parser when encountering an xmlns:xxx > > > > > namespace declaration is to call > > ContentHandler.startPrefixMapping(), > > > > > and to remove this attribute from the Attributes object passed to > > > > > startElemnt(). > > > > > > > > > > But Cocoon code also sets the "namespace-prefixes" feature each time > > > > it > > > > > creates a Parser. This has the effect of also passing > > xmlns:xxx in the > > > > > Attributes (see ContentHandler.startElement() javadocs for more > > > > > details). > > > > > > > > > > What is the reason for enabling this feature ? I guess this > > was a try > > > > to > > > > > circumvent a long-standing (still present) bug in Xalan's serializer > > > > > which doesn't output properly namespace declarations if they're not > > > > > present as attributes in the incoming SAX stream. However, > > this wasn't > > > > > the solution and I patched several months ago AbstractTextSerializer > > > > to > > > > > make Xalan happy. > > > > > > > > > > Now I'm using Saxon for some parts of our apps and this one barks on > > > > > each and every xmlns:xxx attribute it encounters when reading > > > > > stylesheets. So I locally removed the namespace-prefixes feature : > > > > this > > > > > fixes Saxon problems and doesn't hurt Xalan (it needs them only for > > > > > proper serialization). > > > > > > > > > > So, team, what about not enabling the "namespace-prefix" feature and > > > > > thus keep the default behaviour of SAX parsers ? > > > > > > > > > > +1 from me. > > > > > > > > > > Sylvain. > > > > -- > > Sylvain Wallez > > Anyware Technologies - http://www.anyware-tech.com > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, email: [EMAIL PROTECTED] > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, email: [EMAIL PROTECTED] > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, email: [EMAIL PROTECTED]
