Sorry my previous message went off too quickly.

Hi Mark, Paul, Scott,

This commit was probably not necessary at all. What is the difference
between this XMLLayout and the one in log4j proper? Can we remove this
duplicate file?

Having duplicates lying around makes it quite difficult to maintain
the code. Now that both Scott and you have access to the log4j CVS
repository and not just the sandbox, I ask you please stop maintaining
duplicates in log4j and log4j-sandbox. Given that your enhancements to
chainsaw remain unchallenged, please also migrate chainsaw as well as
the new plugin, receiver code.

Is that OK with everyone? Mark?


At 02:45 AM 6/24/2003 +0000, you wrote:
psmith 2003/06/23 19:45:45

  Modified:    src/java/org/apache/log4j/xml XMLLayout.java
  Log:
  Sandbox version of XMLLayout now conforms to the new Layout contract.

Revision Changes Path
1.3 +43 -51 jakarta-log4j-sandbox/src/java/org/apache/log4j/xml/XMLLayout.java


Index: XMLLayout.java
===================================================================
RCS file: /home/cvs/jakarta-log4j-sandbox/src/java/org/apache/log4j/xml/XMLLayout.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- XMLLayout.java 29 Apr 2003 16:21:30 -0000 1.2
+++ XMLLayout.java 24 Jun 2003 02:45:45 -0000 1.3
@@ -51,6 +51,8 @@
// Contributors: Mathias Bogaert
package org.apache.log4j.xml;


  +import java.io.IOException;
  +import java.io.Writer;
   import java.util.Iterator;
   import java.util.Set;

  @@ -94,9 +96,6 @@
    * @since 0.9.0
    * */
   public class XMLLayout extends Layout {
  -  private final int DEFAULT_SIZE = 256;
  -  private final int UPPER_LIMIT = 2048;
  -  private StringBuffer buf = new StringBuffer(DEFAULT_SIZE);
     private boolean locationInfo = false;

     /**
  @@ -128,108 +127,101 @@
     /**
      * Formats a [EMAIL PROTECTED] LoggingEvent} in conformance with the log4j.dtd.
      * */

--
Ceki For log4j documentation consider "The complete log4j manual"
ISBN: 2970036908 http://www.qos.ch/shop/products/clm_t.jsp



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



Reply via email to