It is a good idea. Just be aware that it may break Chainsaw being able to load the throwable information. This will most likely be the case for other people who have written tools to load it.
I will fix Chainsaw to work, I would like to be backwards compatible so people can load both the old format, and the new one. Pity there is not version information in the XML file so Chainsaw would know how to handle the throwable information. Oliver > -----Original Message----- > From: Mark Womack [mailto:[EMAIL PROTECTED]] > Sent: Friday, 26 April 2002 06:59 > To: 'Log4J Developers List' > Subject: RE: XMLLayout and throwable element > > > I think it is a good idea. The output will be readable. > -Mark > > > -----Original Message----- > > From: Ceki Gülcü [mailto:[EMAIL PROTECTED]] > > Sent: Thursday, April 25, 2002 1:52 PM > > To: [EMAIL PROTECTED] > > Subject: XMLLayout and throwable element > > > > > > > > > > XMLLayout currently does not output line separators when > > outputting the > > <log4j:throwable> element. Example output: > > > > <log4j:throwable><![CDATA[java.lang.Exception: Just testing at > > org.apache.log4j.xml.XMLLayoutTest.common(XMLLayoutTestCase > > .java:68) at > > org.apache.log4j.xml.XMLLayoutTest.basic(XMLLayoutTestCase.jav > > a:43) at > > java.lang.reflect.Method.invoke > > (Native Method) at > > junit.framework.Test.runTest(TestCase.java:166) at > > junit.framework.TestCase.runBare(TestCase.java:140)]]></log4j: > > throwable> > > > > I am proposing the following instead which includes line separators: > > > > <log4j:throwable><![CDATA[java.lang.Exception: Just testing > > at > > org.apache.log4j.xml.XMLLayoutTest.common(XMLLayoutTestCase.java:68) > > at > > org.apache.log4j.xml.XMLLayoutTest.basic(XMLLayoutTestCase.java:43) > > at java.lang.reflect.Method.invoke(Native Method) > > at junit.framework.TestCase.runTest(TestCase.java:166) > > at junit.framework.TestCase.runBare(TestCase.java:140)]]> > > </log4j:throwable> > > > > Is this OK for everyone? > > > > -- > > Ceki > > > > > > -- > > To unsubscribe, e-mail: > <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> > > -- > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> > -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
