Elena Litani wrote:

> Andy Clark wrote:
> >
> > I would
> > suggest a slight change to your idea, Arnaud, and pass
> > the namespace context object to the startDocument method
> > in much the same way that we pass the document locator
> > object.
>
> I like this proposal. Here is my +1 for making this change.

hi,

i am not xerces 2 commiter but anyway would like to voice my concern -1.

if i understand it correctly you will ad a new parameter to startDocument()?
would it not break any code that actually tries overrides startDocument()?
if it is the case then it is even worse than adding new method and deprecating old one 
...

currently it is:

    public void startDocument(XMLLocator locator, String encoding, Augmentations augs)

maybe instead of adding new parameter namespace context could be passed
as part of Augmentations (using some well defined key)?

is there a good way to detect and deal with such changes when using
xerces XNI from outside xercesImpl.jar?

i wrote code that uses Xerces 2 XNI to implement XPP2 and other implementation for
XmlPull API (xni2xmlpull on sourceforge) so you can consider me as one of XNI API 
clients.

making such backward incompatible change will not only break my code
but also introduce lot of possible bugs if people will try to use my code with older 
version
of Xerces 2 (something like incompatible class change exception throw to the user
comes ot mind ...).

so iw ould liek to propose to keep start/enPrefixMapping() methods but deprecated
and use Augmentations in startDocument() to pass namespace contenxt for those
that needs it.

thanks,

alek



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

Reply via email to