You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@excalibur.apache.org by Leo Simons <ma...@leosimons.com> on 2007/01/23 12:34:35 UTC

Re: svn commit: r492453 - in /excalibur/trunk/components/xmlutil/src/java/org/apache/excalibur/xml/dom: DefaultDOMHandlerFactory.java DefaultDOMSerializer.java

This is a backwards-incompatible change, not just cleanup. I don't  
have a problem with it, but it should be justified. Why did you make  
the change?

- LSD

On Jan 4, 2007, at 9:38 AM, cziegeler@apache.org wrote:
> Author: cziegeler
> Date: Thu Jan  4 00:38:26 2007
> New Revision: 492453
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=492453
> Log:
> Cleanup and add new header
>
> Modified:
>     excalibur/trunk/components/xmlutil/src/java/org/apache/ 
> excalibur/xml/dom/DefaultDOMHandlerFactory.java
>     excalibur/trunk/components/xmlutil/src/java/org/apache/ 
> excalibur/xml/dom/DefaultDOMSerializer.java
>
> Modified: excalibur/trunk/components/xmlutil/src/java/org/apache/ 
> excalibur/xml/dom/DefaultDOMHandlerFactory.java
> URL: http://svn.apache.org/viewvc/excalibur/trunk/components/ 
> xmlutil/src/java/org/apache/excalibur/xml/dom/ 
> DefaultDOMHandlerFactory.java?view=diff&rev=492453&r1=492452&r2=492453
> ====================================================================== 
> ========
> --- excalibur/trunk/components/xmlutil/src/java/org/apache/ 
> excalibur/xml/dom/DefaultDOMHandlerFactory.java (original)
> +++ excalibur/trunk/components/xmlutil/src/java/org/apache/ 
> excalibur/xml/dom/DefaultDOMHandlerFactory.java Thu Jan  4 00:38:26  
> 2007
> @@ -20,7 +20,6 @@
>  import javax.xml.transform.sax.SAXTransformerFactory;
>  import javax.xml.transform.sax.TransformerHandler;
>
> -import org.apache.avalon.framework.component.Component;
>  import org.apache.avalon.framework.logger.AbstractLogEnabled;
>  import org.apache.avalon.framework.thread.ThreadSafe;
>  import org.w3c.dom.Document;
> @@ -32,7 +31,7 @@
>   */
>  public class DefaultDOMHandlerFactory
>      extends AbstractLogEnabled
> -    implements DOMHandlerFactory, Component, ThreadSafe
> +    implements DOMHandlerFactory, ThreadSafe
>  {
>      private final SAXTransformerFactory m_transformerFactory =  
> (SAXTransformerFactory) TransformerFactory.newInstance();
>
>
> Modified: excalibur/trunk/components/xmlutil/src/java/org/apache/ 
> excalibur/xml/dom/DefaultDOMSerializer.java
> URL: http://svn.apache.org/viewvc/excalibur/trunk/components/ 
> xmlutil/src/java/org/apache/excalibur/xml/dom/ 
> DefaultDOMSerializer.java?view=diff&rev=492453&r1=492452&r2=492453
> ====================================================================== 
> ========
> --- excalibur/trunk/components/xmlutil/src/java/org/apache/ 
> excalibur/xml/dom/DefaultDOMSerializer.java (original)
> +++ excalibur/trunk/components/xmlutil/src/java/org/apache/ 
> excalibur/xml/dom/DefaultDOMSerializer.java Thu Jan  4 00:38:26 2007
> @@ -23,7 +23,6 @@
>  import javax.xml.transform.dom.DOMSource;
>  import javax.xml.transform.sax.SAXResult;
>
> -import org.apache.avalon.framework.component.Component;
>  import org.apache.avalon.framework.logger.AbstractLogEnabled;
>  import org.w3c.dom.Document;
>  import org.xml.sax.ContentHandler;
> @@ -36,7 +35,7 @@
>   */
>  public class DefaultDOMSerializer
>      extends AbstractLogEnabled
> -    implements DOMSerializer, Component
> +    implements DOMSerializer
>  {
>      private final TransformerFactory m_factory =  
> TransformerFactory.newInstance();
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: scm-unsubscribe@excalibur.apache.org
> For additional commands, e-mail: scm-help@excalibur.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@excalibur.apache.org
For additional commands, e-mail: dev-help@excalibur.apache.org