You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by ru...@apache.org on 2005/12/03 23:16:45 UTC
svn commit: r348141 -
/webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
Author: ruchithf
Date: Tue Nov 22 02:17:13 2005
New Revision: 348141
URL: http://svn.apache.org/viewcvs?rev=348141&view=rev
Log:
Makeing the saaj imple use the OM-DOM Dom impl
Modified:
webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
Modified: webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java?rev=348141&r1=348140&r2=348141&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java (original)
+++ webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java Tue Nov 22 02:17:13 2005
@@ -21,6 +21,7 @@
import org.apache.axis2.om.OMNode;
import org.apache.axis2.om.OMText;
import org.apache.axis2.om.impl.OMNodeEx;
+import org.apache.axis2.om.impl.dom.jaxp.DocumentBuilderFactoryImpl;
import org.apache.axis2.util.Dom2OmUtils;
import org.w3c.dom.DOMException;
import org.w3c.dom.Document;
@@ -36,7 +37,12 @@
* Class NodeImpl
*/
public class NodeImpl implements Node {
-
+
+ static {
+ System.setProperty("javax.xml.parsers.DocumentBuilderFactory",DocumentBuilderFactoryImpl.class.getName());
+ }
+
+
/**
* Field omNode
*/
Re: svn commit: r348141 - /webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
Posted by Ruchith Fernando <ru...@gmail.com>.
We started on a saaj impl based on DOOM.
Code is here (not complete):
http://svn.apache.org/viewcvs.cgi/webservices/axis2/trunk/archive/java/scratch/ruchith_dims/
Thanks,
Ruchith
On 12/7/05, Davanum Srinivas <da...@gmail.com> wrote:
> i haven't reviewed the code, but this certainly is not the solution.
> will do soon.
>
> -- dims
>
> On 12/6/05, Sanjiva Weerawarana <sa...@opensource.lk> wrote:
> > On Sun, 2005-12-04 at 09:37 +0600, Ruchith Fernando wrote:
> > > Removed - r353805
> > >
> > > Thanks,
> > > ruchith
> > >
> > > On 12/4/05, Davanum Srinivas <da...@gmail.com> wrote:
> > > > Hmm...Please don't do this. If the NodeImpl class gets loaded the
> > > > ENTIRE JVM will start using our Document Builder Factory.
> >
> > What's the solution?
> >
> > Sanjiva.
> >
> >
> >
>
>
> --
> Davanum Srinivas : http://wso2.com/blogs/
>
--
Ruchith
Re: svn commit: r348141 - /webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
Posted by Davanum Srinivas <da...@gmail.com>.
i haven't reviewed the code, but this certainly is not the solution.
will do soon.
-- dims
On 12/6/05, Sanjiva Weerawarana <sa...@opensource.lk> wrote:
> On Sun, 2005-12-04 at 09:37 +0600, Ruchith Fernando wrote:
> > Removed - r353805
> >
> > Thanks,
> > ruchith
> >
> > On 12/4/05, Davanum Srinivas <da...@gmail.com> wrote:
> > > Hmm...Please don't do this. If the NodeImpl class gets loaded the
> > > ENTIRE JVM will start using our Document Builder Factory.
>
> What's the solution?
>
> Sanjiva.
>
>
>
--
Davanum Srinivas : http://wso2.com/blogs/
Re: svn commit: r348141 -
/webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
Posted by Sanjiva Weerawarana <sa...@opensource.lk>.
On Sun, 2005-12-04 at 09:37 +0600, Ruchith Fernando wrote:
> Removed - r353805
>
> Thanks,
> ruchith
>
> On 12/4/05, Davanum Srinivas <da...@gmail.com> wrote:
> > Hmm...Please don't do this. If the NodeImpl class gets loaded the
> > ENTIRE JVM will start using our Document Builder Factory.
What's the solution?
Sanjiva.
Re: svn commit: r348141 - /webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
Posted by Ruchith Fernando <ru...@gmail.com>.
Removed - r353805
Thanks,
ruchith
On 12/4/05, Davanum Srinivas <da...@gmail.com> wrote:
> Hmm...Please don't do this. If the NodeImpl class gets loaded the
> ENTIRE JVM will start using our Document Builder Factory.
>
> thanks,
> -- dims
>
> On 12/3/05, ruchithf@apache.org <ru...@apache.org> wrote:
> > Author: ruchithf
> > Date: Tue Nov 22 02:17:13 2005
> > New Revision: 348141
> >
> > URL: http://svn.apache.org/viewcvs?rev=348141&view=rev
> > Log:
> > Makeing the saaj imple use the OM-DOM Dom impl
> >
> >
> > Modified:
> > webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
> >
> > Modified: webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
> > URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java?rev=348141&r1=348140&r2=348141&view=diff
> > ==============================================================================
> > --- webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java (original)
> > +++ webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java Tue Nov 22 02:17:13 2005
> > @@ -21,6 +21,7 @@
> > import org.apache.axis2.om.OMNode;
> > import org.apache.axis2.om.OMText;
> > import org.apache.axis2.om.impl.OMNodeEx;
> > +import org.apache.axis2.om.impl.dom.jaxp.DocumentBuilderFactoryImpl;
> > import org.apache.axis2.util.Dom2OmUtils;
> > import org.w3c.dom.DOMException;
> > import org.w3c.dom.Document;
> > @@ -36,7 +37,12 @@
> > * Class NodeImpl
> > */
> > public class NodeImpl implements Node {
> > -
> > +
> > + static {
> > + System.setProperty("javax.xml.parsers.DocumentBuilderFactory",DocumentBuilderFactoryImpl.class.getName());
> > + }
> > +
> > +
> > /**
> > * Field omNode
> > */
> >
> >
> >
>
>
> --
> Davanum Srinivas : http://wso2.com/blogs/
>
--
Ruchith
Re: svn commit: r348141 - /webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
Posted by Davanum Srinivas <da...@gmail.com>.
Hmm...Please don't do this. If the NodeImpl class gets loaded the
ENTIRE JVM will start using our Document Builder Factory.
thanks,
-- dims
On 12/3/05, ruchithf@apache.org <ru...@apache.org> wrote:
> Author: ruchithf
> Date: Tue Nov 22 02:17:13 2005
> New Revision: 348141
>
> URL: http://svn.apache.org/viewcvs?rev=348141&view=rev
> Log:
> Makeing the saaj imple use the OM-DOM Dom impl
>
>
> Modified:
> webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
>
> Modified: webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java
> URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java?rev=348141&r1=348140&r2=348141&view=diff
> ==============================================================================
> --- webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java (original)
> +++ webservices/axis2/trunk/java/modules/saaj/src/org/apache/axis2/saaj/NodeImpl.java Tue Nov 22 02:17:13 2005
> @@ -21,6 +21,7 @@
> import org.apache.axis2.om.OMNode;
> import org.apache.axis2.om.OMText;
> import org.apache.axis2.om.impl.OMNodeEx;
> +import org.apache.axis2.om.impl.dom.jaxp.DocumentBuilderFactoryImpl;
> import org.apache.axis2.util.Dom2OmUtils;
> import org.w3c.dom.DOMException;
> import org.w3c.dom.Document;
> @@ -36,7 +37,12 @@
> * Class NodeImpl
> */
> public class NodeImpl implements Node {
> -
> +
> + static {
> + System.setProperty("javax.xml.parsers.DocumentBuilderFactory",DocumentBuilderFactoryImpl.class.getName());
> + }
> +
> +
> /**
> * Field omNode
> */
>
>
>
--
Davanum Srinivas : http://wso2.com/blogs/