You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@xalan.apache.org by Sebastien Sahuc <ss...@imediation.com> on 2000/12/18 17:21:51 UTC
RE: [Xalan2][PATCH] SAXSource(XMLReader , InputSource ); doesn't
make use of the XMreader object provided
> Looks reasonable. I'll apply the patches later today.
>
> -scott
Sorry to bother you with that Scott, but it doesn't look like this has been
committed in the current CVS branch.
Thanks for considering this,
Sebastien
PS: IMHO Xalan2 need to be more stable before it goes XLST 1.1. But this is
just IMHO only.
>
>
>
>
>
>
> Sebastien
>
> Sahuc To:
> xalan-dev@xml.apache.org
> <ssahuc@imedi cc: (bcc: Scott
> Boag/CAM/Lotus)
> ation.com> Subject:
> [Xalan2][PATCH] SAXSource(XMLReader , InputSource ); doesn't
> make use of the
> XMreader object provided
> 12/11/00
>
> 06:27 AM
>
> Please
>
> respond to
>
> xalan-dev
>
>
>
>
>
>
>
>
>
> [Xalan2][PATCH] SAXSource ( XMLReader , InputSource );
> doesn't make use of
> the XMreader object provided. The patch below (or attached)
> correct this.
>
>
>
>
>
> Best regards,
>
>
> Sebastien
>
>
> Index: ./src/org/apache/xalan/transformer/TransformerImpl.java
> ===================================================================
> RCS file:
> /home/cvspublic/xml-xalan/java/src/org/apache/xalan/transforme
> r/TransformerImpl.java,v
>
> retrieving revision 1.58
> diff -r1.58 TransformerImpl.java
> 473,486c473,490
> < // Use JAXP1.1 ( if possible )
> < try {
> < javax.xml.parsers.SAXParserFactory factory=
> < javax.xml.parsers.SAXParserFactory.newInstance();
> < factory.setNamespaceAware( true );
> < javax.xml.parsers.SAXParser jaxpParser=
> < factory.newSAXParser();
> < reader=jaxpParser.getXMLReader();
> <
> < } catch( javax.xml.parsers.ParserConfigurationException ex ) {
> < throw new org.xml.sax.SAXException( ex );
> < } catch( javax.xml.parsers.FactoryConfigurationError ex1 ) {
> < throw new org.xml.sax.SAXException( ex1.toString() );
> < } catch( NoSuchMethodError ex2 ) {
> ---
> > if (null == reader)
> > {
> >
> > // Use JAXP1.1 ( if possible )
> > try {
> > javax.xml.parsers.SAXParserFactory factory=
> > javax.xml.parsers.SAXParserFactory.newInstance();
> > factory.setNamespaceAware( true );
> > javax.xml.parsers.SAXParser jaxpParser=
> > factory.newSAXParser();
> > reader=jaxpParser.getXMLReader();
> >
> > } catch(
> javax.xml.parsers.ParserConfigurationException ex ) {
> > throw new org.xml.sax.SAXException( ex );
> > } catch(
> javax.xml.parsers.FactoryConfigurationError ex1 ) {
> > throw new org.xml.sax.SAXException( ex1.toString() );
> > } catch( NoSuchMethodError ex2 ) {
> > }
>
>
> (See attached file: diff.txt)
>
>
>
>
>
>